[Bug 990425] New: calibre's package has a broken description/name
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 Bug ID: 990425 Summary: calibre's package has a broken description/name Classification: openSUSE Product: openSUSE Distribution Version: Leap 42.1 Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Other Assignee: bnc-team-screening@forge.provo.novell.com Reporter: dwaas@suse.com QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- Created attachment 685389 --> http://bugzilla.opensuse.org/attachment.cgi?id=685389&action=edit zypper search attached you can find the output of: zypper se calibre > calibre.txt -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 Andreas Stieger <astieger@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |astieger@suse.com, | |cornelis@solcon.nl, | |jnweiger@gmail.com, | |saigkill@opensuse.org Assignee|bnc-team-screening@forge.pr |cornelis@solcon.nl |ovo.novell.com | -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c1 Andreas Stieger <astieger@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CONFIRMED --- Comment #1 from Andreas Stieger <astieger@suse.com> --- This is not in the spec but the appdata.xml:
$ zypper search -t package calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+---------+------------------------------+-------- | calibre | EBook Management Application | package
but
$ zypper search -t application calibre [...many translations..]
-- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c2 Andreas Stieger <astieger@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |badshah400@gmail.com Assignee|cornelis@solcon.nl |badshah400@gmail.com --- Comment #2 from Andreas Stieger <astieger@suse.com> --- I think that was your most recent change. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c3 --- Comment #3 from Atri Bhattacharya <badshah400@gmail.com> --- I Don't understand the problem, really, but here are a few comments that might help explain what I don't understand: 1. "zypper se -t application" is a broken feature, it only shows applications corresponding to installed appdata on a given system and randomly a few not installed ones. So if calibre doesn't show up when it isn't installed on zypper's application search, that may not be surprising. On my TW system :~> zypper se -t application calibre Loading repository data... Reading installed packages... No matching items found. 2. zypper se calibre Loading repository data... Reading installed packages... S | Name | Summary | Type --+---------+------------------------------+-------- | calibre | EBook Management Application | package So not really clear where the problem is. Fwiw, my changes are here: https://build.opensuse.org/package/rdiff/Documentation:Tools/calibre?linkrev=base&rev=352 -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c4 --- Comment #4 from Cor Blom <cornelis@solcon.nl> --- I can confirm the bug was introduced with the last Leap update. With update repo disabled and the previous version installed, zypper se is working as with other applications. (In reply to Atri Bhattacharya from comment #3)
2. zypper se calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+---------+------------------------------+-------- | calibre | EBook Management Application | package
Do I understand this correctly that zypper se on TW works correclty with calibre? If so, we can revert the latest change and submit 2.62 again without the app store change. That is, if we consider this bug serious enough. I'm prepared to do the work then. It would be interesting to know if it works correctly on 42.2, but at the moment I don't have the time to test that. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c5 --- Comment #5 from Atri Bhattacharya <badshah400@gmail.com> --- (In reply to Cor Blom from comment #4)
I can confirm the bug was introduced with the last Leap update. With update repo disabled and the previous version installed, zypper se is working as with other applications.
I still don't get why. The appdata changes should not affect the specfile descriptions of calibre, which I did not touch! If it does that's a zypper problem. zypper se is supposed to show the specfile description corresponding to the package, not confuse it with appdata description. Two distinct things.
(In reply to Atri Bhattacharya from comment #3)
2. zypper se calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+---------+------------------------------+-------- | calibre | EBook Management Application | package
Do I understand this correctly that zypper se on TW works correclty with calibre? If so, we can revert the latest change and submit 2.62 again without the app store change. That is, if we consider this bug serious enough. I'm prepared to do the work then.
Yes it works perfectly, but please don't revert this change in Leap either. The "fix" should not be to ship broken appstream metadata, and the previous appdata was broken. Instead, what I can do is try to fix the descr txt in appdata (which is currently blank), but my commit did not change that either. It has always been so. I think it might be a rather major bug for the zypper guys to look into. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c6 --- Comment #6 from Andreas Stieger <astieger@suse.com> --- Look at the /usr/share/appdata/calibre-gui.appdata.xml It contains the description paragraphs in the name node. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c7 --- Comment #7 from Atri Bhattacharya <badshah400@gmail.com> --- Yes, I noticed that. I am trying to fix this appdata issue. But zypper se calibre should not be referencing this data. That is the main problem here. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c8 --- Comment #8 from Atri Bhattacharya <badshah400@gmail.com> --- The appdata issue is fixed here: https://build.opensuse.org/package/show/home:badshah400:branches:Documentati... I will make an sr as soon as I have inspected a built package. Feel free to test packages from here. Thanks! -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c9 --- Comment #9 from Atri Bhattacharya <badshah400@gmail.com> --- https://build.opensuse.org/request/show/415129 should help with the appdata issue. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 Martin Pluskal <mpluskal@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mpluskal@suse.com -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c10 --- Comment #10 from Cor Blom <cornelis@solcon.nl> --- I've accepted the request so it can be tested. I do not see any difference on Leap. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c11 Atri Bhattacharya <badshah400@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(cornelis@solcon.n | |l) --- Comment #11 from Atri Bhattacharya <badshah400@gmail.com> --- (In reply to Cor Blom from comment #10)
I've accepted the request so it can be tested. I do not see any difference on Leap.
You mean the appdata isn't fixed yet? Or "zypper se calibre" does not show the right result? -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c12 Cor Blom <cornelis@solcon.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(cornelis@solcon.n | |l) | --- Comment #12 from Cor Blom <cornelis@solcon.nl> --- "zypper se calibre" still give the same result as the reported initially. So, no. not the right result. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c13 --- Comment #13 from Andreas Stieger <astieger@suse.com> --- If you restrict the search to the Documentation:Tools project, it is shown correctly:
$ zypper ar --refresh --check obs://Documentation:Tools doc $ zypper search -r doc -t application calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+----------------------+------------------------------------------------+------------ | Calibre Ebook Reader | The one stop solution to all your e-book needs | application
Also:
zypper search -r doc calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+----------------------+------------------------------------------------+------------ | Calibre Ebook Reader | The one stop solution to all your e-book needs | application | calibre | EBook Management Application | srcpackage | calibre | EBook Management Application | package | calibre-debuginfo | Debug information for package calibre | package | calibre-debugsource | Debug sources for package calibre | package | calibre-test | Calibre Test Package | package | calibre-test | Calibre Test Package | srcpackage
As such SR#415147 / r354 seems to fix the issue. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c14 Atri Bhattacharya <badshah400@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zypp-maintainers@forge.prov | |o.novell.com Flags| |needinfo?(zypp-maintainers@ | |forge.provo.novell.com) --- Comment #14 from Atri Bhattacharya <badshah400@gmail.com> --- Thanks, Andreas! I see the appdata situation as fixed too. However, it is still worrying that a simple "zypper se calibre" confuses names and summaries between the specfile derived ones and those from the appstream metadata. This should not happen. While I suggest that the current revision of calibre with its fixed appstream metadata be pushed to Leap and Factory, this is something the zypper guys still need to look at for Leap. @zypp-maintainers Could you please take a look at why doing zypper se calibre on openSUSE:Leap:42.1 gives a broken result? For sure it currently has a broken appstream metadata file, but shouldn't the above command return summary and name deduced from the specfile rather than from appdata? As a package management tool, one would expect zypper to prioritise package results in favour of applications, no? -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c15 Michael Andres <ma@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dimstar@opensuse.org Flags|needinfo?(zypp-maintainers@ |needinfo?(dimstar@opensuse. |forge.provo.novell.com) |org) --- Comment #15 from Michael Andres <ma@suse.com> ---
http://download.opensuse.org/update/leap/42.1/oss/repodata/ d5ad30410c4be3bc3b4e55d34a20dd375417aa9ce216ad7468836eef256448bc-appdata.xml.gz
The file contains broken metadata. The application name is defined as multiline string including the translated summaries:
<component> <name>Calibre Ebook Reader<p>calibre is the one stop solu...</p> <p xml:lang="ca">El calibre és la solució completa per ...</p> <p xml:lang="it">calibre è la soluzione definitiva ai ...</p> <p xml:lang="cs">Calibre je jediné řešení pro všechny p...</p> ... </name>
That's why you get a mess if the application is included in the list:
$ zypper se calibre| cut -c -30 Loading repository data... Reading installed packages...
S | Name --+--------------------------- | Calibre Ebook Readercalibr El calibre és la solució calibre è la soluzione d Calibre je jediné řešení calibre on vahepeatustet calibre es la solución ú calibre это комплексное calibre is de complete o O Calibre é a solução ce calibre tüm e-kitap ihti calibre ir vienas pietur Calibre er fullnaðarlaus calibre to jedna odpowie calibre est la solution Calibre ist die Komplett calibre — програма, за д O calibre é a solução de Calibreは電子書籍のすべてのニーズに応える calibre är en samlad lös calibre adalah penyelesa You can use calibre to c Podeu utilitzar el calib Calibre můžete používat Puede usar calibre para Вы можете использовать c U kunt gebruik maken van Pode usar o Calibre para Calibre'yi kitaplarınızı Calibre ir izmantojama J Hægt er að nota calibre Możesz użyć calibre do k Vous pouvez utiliser cal Mit Calibre können Sie I Ви можете скористатися c Você pode usar o calibre Calibreは、本のカタログ化、本の書誌情報の Du kan använda calibre f Anda boleh guna calibre | The one stop solution t | calibre | calibre | calibre
Remember that 'zypper se' default is to match substrings. If you use an exact match to omit the brokenmatch, it looks fine:
$ zypper se -x calibre Loading repository data... Reading installed packages...
S | Name | Summary | Type --+---------+------------------------------+------------ | calibre | EBook Management Application | package | calibre | Ebook reader | application | calibre | EBook Management Application | srcpackage
@DominiqUe: This looks like a broken metadata generator? -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c16 Atri Bhattacharya <badshah400@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(dimstar@opensuse. | |org) | --- Comment #16 from Atri Bhattacharya <badshah400@gmail.com> --- Hi Michael! Thanks for your response. The broken metadata was actually the result of a patch. Fixed now and should be coming to an update near you soon... -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c18 Andreas Stieger <astieger@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #18 from Andreas Stieger <astieger@suse.com> --- Update running, setting as resolved. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c19 --- Comment #19 from Dominique Leuenberger <dimstar@opensuse.org> --- (In reply to Bernhard Wiedemann from comment #17)
This is an autogenerated message for OBS integration: This bug (990425) was mentioned in https://build.opensuse.org/request/show/415209 42.1 / calibre
Is this also an issue in Tumbleweed? If so, can we have the fix there too? (and also 42.2 if the issue exists there) -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=990425 http://bugzilla.opensuse.org/show_bug.cgi?id=990425#c20 --- Comment #20 from Atri Bhattacharya <badshah400@gmail.com> --- SR for factory: https://build.opensuse.org/request/show/415231 -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com