Description of problem: I hope this is not already fixed in fedora. gpk GUI can't parse rpm changelog entries with header lines that don't have a dash (minus: ie " - ") after the email address. Some developers and publican are still generating changelog entries without a minus and causing gpk to display a parse warning instead of changelog/update info. gpk should really be kind enough to also parse changelogs without the modern " - " included they still exist in the wild. Steps to Reproduce: 1. gpk with an updated rpm with " - " in changelog (eg publican book). Actual results: Could not parse header: 'Ryan Lerch <rlerch> 1', expected: 'Firstname Lastname <email> - version-release' Expected results: package update info/changelog etc to display normally Additional info: I filed bug 628464 to rfe publican use the right format.
(In reply to comment #0) > gpk should really be kind enough to also parse > changelogs without the modern " - " included > they still exist in the wild. Sure, it should work fine. > Steps to Reproduce: > 1. gpk with an updated rpm with " - " in changelog (eg publican book). > > Actual results: > Could not parse header: 'Ryan Lerch <rlerch> 1', expected: > 'Firstname Lastname <email> - version-release' Surely it should be: Ryan Lerch <rlerch> 1-1 or Ryan Lerch <rlerch> - 1-1 i.e., version-release, not just the release _or_ version? There's no way we can parse a version-release from just the version or the release on their own. Richard.
Oops indeed... Does PK parse the ver-rel in the changelog headers to decide how many entries to show. Could it treat unparsible ver-rel as none?
(In reply to comment #2) > Does PK parse the ver-rel in the changelog headers > to decide how many entries to show. Could it treat > unparsible ver-rel as none? It could, I guess. Care to submit a patch please?