Pull request for branch for-2011.11/pkg-infra-urgent
Hello,
Here is an urgent fix that follows the changes in the package infrastructure. Unfortunately, those changes break the build of bootloaders (they are ignored by the build process).
Thomas
The following changes since commit 8511728696a298ccdb8b902b60ab398d8f67e745:
gnutls: bump to version 2.10.5 (2011-09-30 08:26:08 +0200)
are available in the git repository at:
http://free-electrons.com/~thomas/buildroot.git for-2011.11/pkg-infra-urgent
Thomas Petazzoni (1):
package: fix bootloader building
package/Makefile.package.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
Thanks,
Signature · 1 line
Thomas Petazzoni
PATCH 1/1 package: fix bootloader building
The changes in e055aea216cff4d12cf55eab2e35ca1459edaa91 and 0849e8193ef429f29945b8c2a54eafc8635e0969 have broken the bootloader support.
This is because the 'pkgparentdir' macro returns the string 'boot/' for bootloaders, while the GENTARGETS_INNER macro was testing this against 'boot' to know if the configuration option to check for this package is BR2_PACKAGE_* or BR2_TARGET_*.
So, instead of testing against 'boot', we test against 'boot/'.
--- package/Makefile.package.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
package/Makefile.package.in +1 -1
@@ -637,7 +637,7 @@ $$($(2)_TARGET_DIRCLEAN): PKG=$(2) # kernel case, the bootloaders case, and the normal packages case. ifeq ($(1),linux) $(2)_KCONFIG_VAR = BR2_LINUX_KERNEL -else ifeq ($(4),boot) +else ifeq ($(4),boot/) $(2)_KCONFIG_VAR = BR2_TARGET_$(2) else $(2)_KCONFIG_VAR = BR2_PACKAGE_$(2)
Signature · 1 line
1.7.4.1
PATCH 1/1 package: fix bootloader building
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Thomas" == Thomas Petazzoni <[email protected]> writes:
Thomas> The changes in e055aea216cff4d12cf55eab2e35ca1459edaa91 and Thomas> 0849e8193ef429f29945b8c2a54eafc8635e0969 have broken the bootloader Thomas> support.
Thomas> This is because the 'pkgparentdir' macro returns the string 'boot/' Thomas> for bootloaders, while the GENTARGETS_INNER macro was testing this Thomas> against 'boot' to know if the configuration option to check for this Thomas> package is BR2_PACKAGE_* or BR2_TARGET_*.
Thomas> So, instead of testing against 'boot', we test against 'boot/'.
Committed, thanks.
Signature · 1 line
Bye, Peter Korsgaard