[Buildroot] [PATCH v3] dieharder: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sun May 17 13:55:22 UTC 2015
Dear Julien Viard de Galbert,
On Sun, 17 May 2015 14:18:15 +0200, Julien Viard de Galbert wrote:
> > Those help lines look fairly too long. Make sure they're not longer
> > than 72 columns.
> >
> Ok, also I just sent another patch to document that point.
Sure, good idea.
> You are right this should be made more explicit, there is an
> extra "beverage" clause in the licence file.
> I could either use the syntax the software uses:
> DIEHARDER_LICENSE = GPLv2b (b for beverage)
>
> Or make it simply:
> DIEHARDER_LICENSE = GPLv2 with beverage clause
>
> What do you think ?
The second solution you propose looks better to me.
> > > +DIEHARDER_LICENSE_FILES = $(DIEHARDER_SUBDIR)/COPYING
> > > +DIEHARDER_DEPENDENCIES = gsl
> > > +
> > > +DIEHARDER_CONF_OPTS = --includedir=$(STAGING_DIR)/usr/include
> >
> > Why do you need this? This looks wrong, as it would install the
> > headers in a completely wrong location, if dieharder would be
> > installing headers. --includedir is not used to specify where a program
> > should look for headers of libraries, but to tell where it should
> > install its own headers.
> >
> That was to try to fix the build using "/usr/include/" but it's wrong.
> I overlooked the normal --includedir function. So I guess the upstream
> scripts are wrong as they use the --includedir value to also specify
> some include search path...
>
> Also it _really_ do install some headers there. We probably don't want
> those on the target rootfs. So I guess I need to find how to tell it not
> to do that...
As long as headers are installed in $(TARGET_DIR)/usr/include, you
shouldn't do anything: Buildroot automatically deletes this directory
at the end of the build process.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list