Bug 4153 - Packages add by locales does not have preinst run
Summary: Packages add by locales does not have preinst run
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 1.3.2
Hardware: x86 Multiple
: Medium major
Target Milestone: 1.3.2
Assignee: Ross Burton
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-03-29 18:48 UTC by Otavio Salvador
Modified: 2013-04-30 12:02 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Otavio Salvador 2013-03-29 18:48:31 UTC
It took me sometime to figure why preinst were not being run by avahi-daemon and it seems it was because it has not being explicitly installed. System has included it late due locale package and in this case it was not run.
Comment 1 Paul Eggleton 2013-04-03 17:00:10 UTC
So this behaviour is definitely a bug; however is it legitimate to bring in avahi as a dependency for locale installation? How is that getting brought in, can you tell?
Comment 2 Otavio Salvador 2013-04-03 17:50:06 UTC
I think both behaviours are buggy:

 * locale packages shouldn't depends on PN packages

 * Packages included during locale install ought to have postinst respected.
Comment 3 Paul Eggleton 2013-04-05 07:40:27 UTC
Which package backend are you using?
Comment 4 Otavio Salvador 2013-04-05 11:49:25 UTC
ipk
Comment 5 Paul Eggleton 2013-04-08 12:27:27 UTC
FYI, locale package dependencies on the main package have already been removed in master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=b71a16e8ac18b5633ac3a52c60ad78dde37ede7b

With this patch reverted on top of master, I can't reproduce the issue. I'll try with danny.
Comment 6 Paul Eggleton 2013-04-08 13:15:31 UTC
I've reproduced this with danny; the following commit is the one that fixed it in master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=f931aeb72e4bcc2c8138d9f1b55b1aea86e13b67

This applies cleanly on top of danny and I've verified it fixes the issue.
Comment 7 Paul Eggleton 2013-04-08 16:29:22 UTC
Ross, can you cherry-pick f931aeb72e4bcc2c8138d9f1b55b1aea86e13b67 for the next danny release?
Comment 8 Ross Burton 2013-04-08 16:34:28 UTC
Merged to my danny branch.
Comment 9 Ross Burton 2013-04-30 12:02:27 UTC
Merged into oe-core/danny.