Bugzilla – Bug 156175
Leading "- " in author in change log entry
Last modified: 2008-06-25 09:53:49 UTC
zypp::ChangeLogEntry::author() always returns a string with a leading "- " for zypp::Package data read from the RPM DB. I suspect this is the remainder of the line split off after the date of each change log entry as returned by RPM: % rpm -q --changelog yast2-core | head -n 10 * Mo Sep 12 2005 - kkaempf@suse.de - ensure string reference does not go out of scope too early (#116432) - V 2.12.27 * Fr Sep 09 2005 - sh@suse.de - (visnov) Fixed bug #115298: Install on x86_64 horribly slow due to excessive memory requirements
Check with the "change log" view of yast2-qt-2.13.28 or later. The problem is probably somewhere in zypp/target/rpm/RpmHeader.cc , but I don't know enough of the parsers there to change anything.
Created attachment 71941 [details] screen shot showing the problem
Sample code: QString YQPkgChangeLogView::changeLogTable( const zypp::Changelog & changeLog ) const { QString html; for ( zypp::Changelog::const_iterator it = changeLog.begin(); it != changeLog.end(); ++it ) { QString changes = htmlEscape( fromUTF8( (*it).text() ) ); changes.replace( "\n", "<br>" ); changes.replace( " ", " " ); html += row( cell( (*it).date() ) + cell( (*it).author() ) + "<td valign=top>" + changes + "</td>" // cell() calls htmlEscape() ! ); } return html.isEmpty() ? "" : table( html ); }
It's still there (just checked), but it's not that important. I had a look into that already when I reported it, but I don't know enough about that underlying parser to change it, much less this late in the release phase. -> LATER
mass reopening all SuSE Linux bugs that are set to REMIND+LATER to change the resolution to WONTFIX (adapting to new policy)
Closing old LATER+REMIND bugs as WONTFIX - if you still plan to work on it, feel free to reopen and set to ASSIGNED. In case the report saw repeated reopen comments, it's due to bugzilla timing out on the huge request ;(