[Buildroot] [PATCH 1/1] eja: new package
Ubaldo Porcheddu
ubaldo at eja.it
Mon May 2 10:36:36 UTC 2016
Hello Thomas,
thank you for your comments and sorry for the misunderstanding about the
cover letter, I didn't know that the "head" would be committed with the
patch.
>> +
>> +define EJA_BUILD_CMDS
>> + $(MAKE) CC="$(TARGET_CC)" -C $(@D) static
>
> Can you try:
>
> $(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) static
>
> Also, why do you build the static variant?
I tried to compile with this build syntax but I couldn't get it working
fine so I had a look at the Lua package and after rewriting a little bit
eja.mk it is now working both for static and shared libraries.
> Could you rework your patch to take into account those comments and
> send an updated version?
I reworked the rest of the points and I am going to submit a new patch
right now.
More information about the buildroot
mailing list