[Buildroot] [PATCH] [PATCH v2] p7zip: New package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon May 30 13:20:42 UTC 2016
Hello,
On Sun, 29 May 2016 19:22:50 +0200, André Hentschel wrote:
> >> +config BR2_PACKAGE_P7ZIP
> >> + bool "p7zip"
> >> + depends on BR2_INSTALL_LIBSTDCPP
> >> + depends on BR2_i386 || BR2_x86_64 || \
> >> + BR2_aarch64 || BR2_arm || BR2_armeb
> >
> > Please explain why this is limited to those architectures.
> >
> > You can put it in the commit log, or you may add a terse comment just
> > above the depends-on line. If the explanations is not straightforward,
> > prefer the commit log.
> >
> > (Yes, I read the previous reply that it was broken on NIOSII. Limiting
> > to a small set of architectures wihtout an explanation is not enough.)
> >
> > Thanks! :-)
>
> Can I start with a small set of archs until it is tested on more?
> (with a comment then anyway of course)
I will disagree with Yann here. We typically do not limit the
architectures based simply on what has been tested or not.
A dependency on specific architectures should only be added if the
package is either 1/ contains some architecture-specific code or 2/ is
known to not build/work properly on some architectures.
In any case, the fact that you could only test on i386/AArch64/ARM is
not a proper reason to restrict the package to those architectures only.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list