On 12/10/14 06:17, Paul Barker wrote:
opkg_remove_pkg is never called with from_upgrade set to a non-zero value. The removal of an old package during an upgrade is handled by directly calling remove_data_files_and_list and remove_maintainer_scripts from opkg_install_pkg. The desired behaviour is given in the comments in prerm_upgrade_old_pkg in opkg_install.c, the behaviour given by the comment in that function just needs implementing.
Indeed, I got fooled by looking at the call to opkg_install_pkg(pkg, 1)
This is already entered as issue 104 in the opkg issue tracker: https://code.google.com/p/opkg/issues/detail?id=104
So it is. Slightly more work than my trivial opkg patch, but then again, my trivial opkg patch is not much use ;-) Please disregard it.
Have you tested this patch before submitting it?
To be honest, I have not tested the opkg "upgrade" patch in isolation. I have tested the opkg patch for multiple package upgrade case together with my changes to update-rc.d.bbclass changes (i.e. the entire patch I posted to the oe-core list) and this has solved the problems that I encountered initially. The "fix" for my use case is due to the changes in the way the prerm, postrm and postinst scripts work, but reviewing those changes, that's not good enough either.
Thanks for the review. I'll try again later... Peter -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
