https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c1 --- Comment #1 from Vincent Untz 2012-02-15 17:50:37 UTC --- Created an attachment (id=476311) --> (http://bugzilla.novell.com/attachment.cgi?id=476311) Test script Just call the script with a package name. See that it works for installed packages, but not for non-installed packages. I tried on Fedora, and it works fine in all cases. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c3 --- Comment #3 from Vincent Untz 2012-02-15 17:52:55 UTC --- Also, it seems that in backend_resolve_thread(), we call zypp_filter_solvable() on each item, while this will already be called later on in zypp_emit_filtered_packages_in_list(), so I don't think this is needed there... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c5 --- Comment #5 from Vincent Untz 2012-02-16 08:30:23 UTC --- Reverting cf65ba285d546e4bbe981d364b0b111fc142bc89 fixes the bug. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c7 --- Comment #7 from Vincent Untz 2012-02-16 08:46:43 UTC --- Duncan, after digging in the code, I have a few questions: - zypp_refresh_cache() seems to be quite slow, even if force is set to false. It's unclear to me why it's slow, since if there's no need to refresh anything on libzypp side, things shouldn't be slow (at least, it's not slow with zypper). - it's unclear when to call zypp_refresh_cache(). It makes sense that it's called on installing packages, but we call it also in backend_find_packages_thread(), while it's not called in backend_resolve_thread(). (I first added it there, and that's when I found it's slow) - I wonder if the change you did in cf65ba285d546e4bbe981d364b0b111fc142bc89 should also be applied to zypp_get_packages_by_file(). Which interestingly isn't used at all (?). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c9 --- Comment #9 from Vincent Untz 2012-02-16 08:57:10 UTC --- (In reply to comment #6)

Created an attachment (id=476437) --> (http://bugzilla.novell.com/attachment.cgi?id=476437) [details] Build pool on resolve

This fixes the issue for me.

We might need a similar fix in backend_get_update_detail_thread(). I see the pool isn't built there, and if it turns out to be the fist function called, it will fail. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c11 --- Comment #11 from Vincent Untz 2012-03-19 16:24:18 UTC --- (In reply to comment #2)

Created an attachment (id=476312) --> (http://bugzilla.novell.com/attachment.cgi?id=476312) [details] Small fix, not completely related

While working on this, I noticed that we raise a "package not found" error on resolve. Looking at the zif backend (which is likely a good reference, since Richard is working on it ;-)), it's not done there. And I don't think it's needed.

For reference, we really want something like this, otherwise resolve just fails when trying to resolve 10 names when 1 of them doesn't exist. And I think failing is wrong in that case. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c Vincent Untz changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|PackageKit cannot resolve |Several PackageKit zypp |non-installed packages |things to fix (comments 7, | |8 & 9) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=747193 https://bugzilla.novell.com/show_bug.cgi?id=747193#c14 --- Comment #14 from Duncan Mac-Vicar 2012-07-20 08:15:39 UTC --- Comment #13. I don't think this is something that can be known for sure. PackageKit offers a refresh cache method, giving the impression that the client is responsible for this. But we got bug reports because updates were not shown up-to-date because the clients were not refreshing the cache. Something to clarify upstream. I am pretty sure there are places to optimize. In the past we fixed pool loads in loops which made everything slow. But we would like to rewrite the backend. There are mistakes in the code like access to low level APIs (like the KByte conversion) which would be much simpler if they used the libzypp front APIs. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.