[Buildroot] [PATCH 3/5 v5] core/pkg-infra: allow meson packages to specify custom build rules

James Hilliard james.hilliard1 at gmail.com
Thu Dec 27 04:21:25 UTC 2018


On Wed, Dec 26, 2018 at 6:06 AM Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
>
> James, Éric, All,
>
> On 2018-12-26 05:36 -0600, James Hilliard spake thusly:
> > On Wed, Dec 26, 2018 at 2:52 AM Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> > > Some packages may want to build only specific targets, instead of the
> > > default.
> [--SNIP--]
> > > diff --git a/package/pkg-meson.mk b/package/pkg-meson.mk
> > > index 507e686068..0588660dbf 100644
> > > --- a/package/pkg-meson.mk
> > > +++ b/package/pkg-meson.mk
> > > @@ -103,12 +103,12 @@ ifndef $(2)_BUILD_CMDS
> > >  ifeq ($(4),target)
> > >  define $(2)_BUILD_CMDS
> > >         $$(TARGET_MAKE_ENV) $$($$(PKG)_NINJA_ENV) \
> > > -               $$(NINJA) $$(NINJA_OPTS) -C $$($$(PKG)_SRCDIR)/build
> > > +               $$(NINJA) $$(NINJA_OPTS) $$($$(PKG)_NINJA_OPTS) -C $$($$(PKG)_SRCDIR)/build
> > >  endef
> > >  else
> > >  define $(2)_BUILD_CMDS
> > >         $$(HOST_MAKE_ENV) $$($$(PKG)_NINJA_ENV) \
> > > -               $$(NINJA) $$(NINJA_OPTS) -C $$($$(PKG)_SRCDIR)/build
> > > +               $$(NINJA) $$(NINJA_OPTS) $$($$(PKG)_NINJA_OPTS) -C $$($$(PKG)_SRCDIR)/build
> > shouldn't $$($$(PKG)_NINJA_OPTS) come after $$($$(PKG)_SRCDIR)/build?
>
> I don't know. It worked that way for me, and I wanted to group together
> the two _OPTS variables, and also like we do for the autotools infra. But
> your suggestion also makes sense, and is indeed more in-line with the
> ninja's short-help:
Maybe add a $$($$(PKG)_NINJA_TARGETS) option as well?
>
>     $ ./output/host/bin/ninja -h
>     usage: ninja [options] [targets...]
>     [...]
>
> Éric, your opinion?
>
> Regards,
> Yann E. MORIN.
>
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list