[Buildroot] [PATCH v7 3/4] infra: Introduce BR2_HOST_NEEDS_JAVA
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Feb 18 16:44:56 UTC 2014
Dear Maxime Hadjinlian,
On Tue, 18 Feb 2014 17:41:17 +0100, Maxime Hadjinlian wrote:
> >> +# Hidden boolean selected by packages in need of Java in order to build
> >> +# (example: xbmc)
> >> +config BR2_HOST_NEEDS_JAVA
> >> + bool
> >
> > Do we need a kconfig bool for this? We have the same situation for
> > classpath, and we haven't introduced a specific kconfig bool for it.
> Yes, I had two patches:
> - One to introduce kconfig bool option for JAVAC and JAR
> - The second to make classpath uses them and remove the specific in
> dependencies.sh
>
> It seems cleaner and if another package has the same needs, it's already there.
Right, but when it's needed for just one package, it's maybe not needed
to have a generic solution. When it starts to be needed by a few
packages, then it is time to refactor things and have a generic
solution such as the one you propose.
But oh well, since you already wrote the code, I'm fine with it.
> > Moreover, I am not sure BR2_HOST_NEEDS_JAVA is the right name. It's not
> > really the host that needs Java, it's the compilation process that
> > requires a host Java. So BR2_NEEDS_HOST_JAVA is probably more
> > technically correct.
> True, I'll fix that.
Cool, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list