[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