[Buildroot] [PATCH] rpm: really take DEPENDENCIES into account

Peter Korsgaard peter at korsgaard.com
Mon Nov 19 15:06:30 UTC 2018


>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at bootlin.com> writes:

 > Commit e7af4033c32560594ddbd457b68f6d3713662a26 ("rpm: use the new
 > gettext logic") introduced a really nasty bug: by adding
 > $(TARGET_NLS_DEPENDENCIES) to RPM_DEPENDENCIES, it completely
 > overwrote the existing value of RPM_DEPENDENCIES, entirely masking all
 > mandatory RPM dependencies.

 > rpm is fairly towards the end of the alphabet, and most other
 > mandatory dependencies (berkeleydb, host-pkgconf, file and popt)
 > appear earlier by alphabetic ordering. Only zlib was afterwards, but
 > since file depends on zlib, it was always built before. This probably
 > explains why our autobuilders haven't encountered a single build
 > failure.

 > However, a simple "make rpm" clearly exhibits the failure, and
 > obviously the upcoming per-package folder mechanism makes such bugs
 > even more obvious.

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
 > ---
 > I don't know who was this guy @free-electrons.com who did this silly
 > change, but something wasn't quite right with him :-)

:P

Committed, thanks.

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list