[Buildroot] [PATCH 3/7] core: add generic support for lz archives
Peter Seiderer
ps.report at gmx.net
Thu Feb 9 21:16:35 UTC 2017
Hello Baruch,
On Thu, 9 Feb 2017 23:11:28 +0200, Baruch Siach <baruch at tkos.co.il> wrote:
> Hi Peter,
>
> On Thu, Feb 09, 2017 at 11:00:37PM +0200, Baruch Siach wrote:
> > On Thu, Feb 09, 2017 at 09:26:12PM +0100, Peter Seiderer wrote:
> > > On Thu, 9 Feb 2017 21:50:14 +0200, Baruch Siach <baruch at tkos.co.il> wrote:
> > > > This commit teaches the generic code how to extract .tar.lz archives. When
> > > > lzip is not installed on the host, host-lzip gets built automatically.
> > > >
> > >
> > > On my host there is no lzip installed, with your patches applied I get the following
> > > error message:
> > >
> > > which: no lzip in (/home/seiderer/Work/build_arm_qt5_stdcpp11_001/host/usr/bin)
> > > You must install '/home/seiderer/Work/build_arm_qt5_stdcpp11_001/host/usr/bin/lzip' on your build machine
> > > support/dependencies/dependencies.mk:22: recipe for target 'core-dependencies' failed
> > > make[1]: *** [core-dependencies] Error 1
> > > Makefile:16: recipe for target '_all' failed
> > > make: *** [_all] Error 2
> >
> > Thanks for testing.
> >
> > Does this fixes your problem?
> >
> > diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> > index 303dda090daa..eb01c2fa0e2f 100644
> > --- a/package/pkg-generic.mk
> > +++ b/package/pkg-generic.mk
> > @@ -930,7 +930,7 @@ endif # SITE_METHOD
> > # ZCAT="gzip -d -c", and to check for the dependency we only want 'gzip'.
> > # Remove xzcat/lzip from the list of required dependencies, as they get built
> > # automatically if they're not found.
> > -DL_TOOLS_DEPENDENCIES += $$(filter-out xzcat lzip,$$(firstword $$(call suitable-extractor,$$($(2)_SOURCE))))
> > +DL_TOOLS_DEPENDENCIES += $$(filter-out xzcat lzip,$$(notdir $$(firstword $$(call suitable-extractor,$$($(2)_SOURCE)))))
Fixes my test case too, I tried:
DL_TOOLS_DEPENDENCIES += $$(filter-out $(HOST_DIR)/usr/bin/xzcat $(HOST_DIR)/usr/bin/lzip,$$(firstword $$(call suitable-extractor,$$($(2)_SOURCE))))
Regards,
Peter
> >
> > # Ensure all virtual targets are PHONY. Listed alphabetically.
> > .PHONY: $(1) \
>
> Thinking about it again, this is clearly not the right solution. It breaks
> custom BR2_LZCAT. I'll try to come up with something better.
>
> baruch
>
More information about the buildroot
mailing list