[RFC,00/37] efl bump to 1.14.2

Hello Romain,
I've tested you efl update
https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
It did not work, :(
But applying the two patches below works again!
The first one was already commented in a previous e-mail.
The second is related to a regression in efl-1.15.0.
The testing I've done has been with the musl libc and with
BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
Basically the same config as in the previous e-mail.

A minor issue: check the spelling of recommanded.

Regards,
  Vicente.

--

In-Reply-To=<55C31E99.8010002@openwide.fr>
Tested-by: vicencb <vicencb@gmail.com>
--

Hi Vicente,

Le 21/08/2015 02:30, vj a écrit :
> Hello Romain,
> I've tested you efl update
> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
> It did not work, :(
> But applying the two patches below works again!
> The first one was already commented in a previous e-mail.

Yes, sorry I haven't looked at your issue yet.

libmount seems to be optional and can be disabled with --disable-libmount:
https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35

But since it's not recommended to disable it, I'll apply your patch :)

> The second is related to a regression in efl-1.15.0.

Can you report your issue to the efl mailing list ?

> The testing I've done has been with the musl libc and with
> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
> Basically the same config as in the previous e-mail.
> 
> A minor issue: check the spelling of recommanded.

Ha indeed recommanded is used in Config.in prompt
(my French was turned on sorry ;-) )

Thanks for testing!

Best regards,
Romain
> 
> Regards,
>   Vicente.
> 
> --
> 
> In-Reply-To=<55C31E99.8010002@openwide.fr>
> Tested-by: vicencb <vicencb@gmail.com>
> --
> 
> diff --git a/package/efl/Config.in b/package/efl/Config.in
> index 3de717f..8cc0f91 100644
> --- a/package/efl/Config.in
> +++ b/package/efl/Config.in
> @@ -11,6 +11,8 @@ config BR2_PACKAGE_EFL
>      select BR2_PACKAGE_JPEG # Emile needs libjpeg
>      select BR2_PACKAGE_LIBCURL # eina_module, runtime dependency
>      select BR2_PACKAGE_LUA # Evas lua 5.1 or 5.2
> +    select BR2_PACKAGE_UTIL_LINUX
> +    select BR2_PACKAGE_UTIL_LINUX_LIBMOUNT
>      select BR2_PACKAGE_LIBXKBCOMMON if BR2_PACKAGE_WAYLAND
>      help
>        Enlightenment Foundation Libraries
> 
> 
> 
> diff --git a/package/efl/efl.mk b/package/efl/efl.mk
> index 2d29375..d785efc 100644
> --- a/package/efl/efl.mk
> +++ b/package/efl/efl.mk
> @@ -15,6 +15,11 @@ EFL_INSTALL_STAGING = YES
>  EFL_DEPENDENCIES = host-pkgconf host-efl dbus freetype jpeg libcurl lua udev \
>      zlib
> 
> +# In 1.15.0 there is a regression with missing symbols,
> +# at least when trying to load the fb module.
> +# Making all symbols visible by default fixes the problem.
> +EFL_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -fvisibility=default"
> +
>  # Configure options:
>  # --disable-cxx-bindings: disable C++11 bindings.
>  # --enable-lua-old: disable Elua and remove luajit dependency.
>

Hi Vicente,

>> Tested-by: vicencb <vicencb@gmail.com>

Can you use your real name here.

Thanks!

Best regards,
Romain

Hello Romain,

On Fri, Aug 21, 2015 at 10:51 AM, Romain Naour <romain.naour@openwide.fr> wrote:
> Hi Vicente,
>
> Le 21/08/2015 02:30, vj a écrit :
>> Hello Romain,
>> I've tested you efl update
>> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
>> It did not work, :(
>> But applying the two patches below works again!
>> The first one was already commented in a previous e-mail.
>
> Yes, sorry I haven't looked at your issue yet.
>
> libmount seems to be optional and can be disabled with --disable-libmount:
> https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35

Without that dependency it failed with 1.14.2.
When I saw the dependency was not there in 1.15.0, I just added it
again without checking.
Just now I've tried it again without that patch and it's fine, so, you
can forget it.
Sorry for the hassle.

>
> But since it's not recommended to disable it, I'll apply your patch :)
>
>> The second is related to a regression in efl-1.15.0.
>
> Can you report your issue to the efl mailing list ?

I know it would be better to have it fixed upstream, but have no time
for that now.
If you would like to, feel free to report it.
To debug it, it helps enabling the WRN and DGB macros in eina_module.c.

>
>> The testing I've done has been with the musl libc and with
>> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
>> Basically the same config as in the previous e-mail.
>>
>> A minor issue: check the spelling of recommanded.
>
> Ha indeed recommanded is used in Config.in prompt
> (my French was turned on sorry ;-) )
>
> Thanks for testing!

Thanks for adding musl and maintaining efl!

>
> Best regards,
> Romain
>>
>> Regards,
>>   Vicente.
>>
>> --
>>
>> In-Reply-To=<55C31E99.8010002@openwide.fr>
Tested-by: Vicente Bergas <vicencb@gmail.com>
>> --
>>
>> diff --git a/package/efl/Config.in b/package/efl/Config.in
>> index 3de717f..8cc0f91 100644
>> --- a/package/efl/Config.in
>> +++ b/package/efl/Config.in
>> @@ -11,6 +11,8 @@ config BR2_PACKAGE_EFL
>>      select BR2_PACKAGE_JPEG # Emile needs libjpeg
>>      select BR2_PACKAGE_LIBCURL # eina_module, runtime dependency
>>      select BR2_PACKAGE_LUA # Evas lua 5.1 or 5.2
>> +    select BR2_PACKAGE_UTIL_LINUX
>> +    select BR2_PACKAGE_UTIL_LINUX_LIBMOUNT
>>      select BR2_PACKAGE_LIBXKBCOMMON if BR2_PACKAGE_WAYLAND
>>      help
>>        Enlightenment Foundation Libraries
>>
>>
>>
>> diff --git a/package/efl/efl.mk b/package/efl/efl.mk
>> index 2d29375..d785efc 100644
>> --- a/package/efl/efl.mk
>> +++ b/package/efl/efl.mk
>> @@ -15,6 +15,11 @@ EFL_INSTALL_STAGING = YES
>>  EFL_DEPENDENCIES = host-pkgconf host-efl dbus freetype jpeg libcurl lua udev \
>>      zlib
>>
>> +# In 1.15.0 there is a regression with missing symbols,
>> +# at least when trying to load the fb module.
>> +# Making all symbols visible by default fixes the problem.
>> +EFL_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -fvisibility=default"
>> +
>>  # Configure options:
>>  # --disable-cxx-bindings: disable C++11 bindings.
>>  # --enable-lua-old: disable Elua and remove luajit dependency.
>>
>

Hi Vicente

Le 21/08/2015 19:56, vj a écrit :
> Hello Romain,
> 
> On Fri, Aug 21, 2015 at 10:51 AM, Romain Naour <romain.naour@openwide.fr> wrote:
>> Hi Vicente,
>>
>> Le 21/08/2015 02:30, vj a écrit :
>>> Hello Romain,
>>> I've tested you efl update
>>> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
>>> It did not work, :(
>>> But applying the two patches below works again!
>>> The first one was already commented in a previous e-mail.
>>
>> Yes, sorry I haven't looked at your issue yet.
>>
>> libmount seems to be optional and can be disabled with --disable-libmount:
>> https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35
> 
> Without that dependency it failed with 1.14.2.
> When I saw the dependency was not there in 1.15.0, I just added it
> again without checking.
> Just now I've tried it again without that patch and it's fine, so, you
> can forget it.
> Sorry for the hassle.

It's ok ;-) I haven't heavily tested with a minimal config.
Maybe next time can you post the last ~100 build log lines, it not always easy
to reproduce a build issue...

> 
>>
>> But since it's not recommended to disable it, I'll apply your patch :)
>>
>>> The second is related to a regression in efl-1.15.0.
>>
>> Can you report your issue to the efl mailing list ?
> 
> I know it would be better to have it fixed upstream, but have no time
> for that now.
> If you would like to, feel free to report it.
> To debug it, it helps enabling the WRN and DGB macros in eina_module.c.

Well, I can't really report an issue that I can't reproduce.

See, I reported the issue about SDL2 dependency and it's now fixed in the efl
1.15.1. So, I can drop the last patch and avoid to autoreconf the package.

Also, I'm probably not the good person to report an efl issue.
Honestly, I don't know what to do with -fvisibility=default.
It would be better if you can send just an email to the enlightenment mailing
list to report your issue.

> 
>>
>>> The testing I've done has been with the musl libc and with
>>> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
>>> Basically the same config as in the previous e-mail.
>>>
>>> A minor issue: check the spelling of recommanded.
>>
>> Ha indeed recommanded is used in Config.in prompt
>> (my French was turned on sorry ;-) )
>>
>> Thanks for testing!
> 
> Thanks for adding musl and maintaining efl!

You're welcome!

Best regards,
Romain

Hello Romain,

On Mon, Aug 24, 2015 at 10:42 PM, Romain Naour <romain.naour@openwide.fr> wrote:
> Hi Vicente
>
> Le 21/08/2015 19:56, vj a écrit :
>> Hello Romain,
>>
>> On Fri, Aug 21, 2015 at 10:51 AM, Romain Naour <romain.naour@openwide.fr> wrote:
>>> Hi Vicente,
>>>
>>> Le 21/08/2015 02:30, vj a écrit :
>>>> Hello Romain,
>>>> I've tested you efl update
>>>> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
>>>> It did not work, :(
>>>> But applying the two patches below works again!
>>>> The first one was already commented in a previous e-mail.
>>>
>>> Yes, sorry I haven't looked at your issue yet.
>>>
>>> libmount seems to be optional and can be disabled with --disable-libmount:
>>> https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35
>>
>> Without that dependency it failed with 1.14.2.
>> When I saw the dependency was not there in 1.15.0, I just added it
>> again without checking.
>> Just now I've tried it again without that patch and it's fine, so, you
>> can forget it.
>> Sorry for the hassle.
>
> It's ok ;-) I haven't heavily tested with a minimal config.
> Maybe next time can you post the last ~100 build log lines, it not always easy
> to reproduce a build issue...
>
>>
>>>
>>> But since it's not recommended to disable it, I'll apply your patch :)
>>>
>>>> The second is related to a regression in efl-1.15.0.
>>>
>>> Can you report your issue to the efl mailing list ?
>>
>> I know it would be better to have it fixed upstream, but have no time
>> for that now.
>> If you would like to, feel free to report it.
>> To debug it, it helps enabling the WRN and DGB macros in eina_module.c.
>
> Well, I can't really report an issue that I can't reproduce.

Do you mean that the fb backend is working for you?
Are you crosscompiling with upstream vanilla gcc>=4 for arm?
It might be a compiler bug too.
I'm using musl gcc 4.9.3 build with buildroot itself.
gcc 5 seems broken for arm crosscompilation.
Just for completeness, the compiler I'm using was build with:
buildroot.version=2ebbb7fe355c18a0be3d0fb8e50997142113c46b
BR2_arm=y
BR2_cortex_a8=y
BR2_ARM_EABIHF=y
BR2_ARM_FPU_NEON=y
BR2_HOST_DIR="/opt/arm-buildroot-linux-musleabihf"
BR2_OPTIMIZE_3=y
BR2_TOOLCHAIN_BUILDROOT_MUSL=y
BR2_BINUTILS_VERSION_2_25_X=y
BR2_TOOLCHAIN_BUILDROOT_CXX=y
BR2_GCC_ENABLE_LTO=y
BR2_GCC_ENABLE_GRAPHITE=y
BR2_TARGET_OPTIMIZATION="-march=armv7-a -mtune=cortex-a8
-mcpu=cortex-a8 -mfpu=neon -mfloat-abi=hard"

> See, I reported the issue about SDL2 dependency and it's now fixed in the efl
> 1.15.1. So, I can drop the last patch and avoid to autoreconf the package.
>
> Also, I'm probably not the good person to report an efl issue.
> Honestly, I don't know what to do with -fvisibility=default.
> It would be better if you can send just an email to the enlightenment mailing
> list to report your issue.

Is "enlightenment-devel@lists.sourceforge.net" the correct mailing
list for an efl bug?

>>
>>>
>>>> The testing I've done has been with the musl libc and with
>>>> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
>>>> Basically the same config as in the previous e-mail.
>>>>
>>>> A minor issue: check the spelling of recommanded.
>>>
>>> Ha indeed recommanded is used in Config.in prompt
>>> (my French was turned on sorry ;-) )
>>>
>>> Thanks for testing!
>>
>> Thanks for adding musl and maintaining efl!
>
> You're welcome!
>
> Best regards,
> Romain

Regards,
  Vicente.

Hello Vicente,

Sorry for the delay.

Le 25/08/2015 00:09, vj a écrit :
> Hello Romain,
> 
> On Mon, Aug 24, 2015 at 10:42 PM, Romain Naour <romain.naour@openwide.fr> wrote:
>> Hi Vicente
>>
>> Le 21/08/2015 19:56, vj a écrit :
>>> Hello Romain,
>>>
>>> On Fri, Aug 21, 2015 at 10:51 AM, Romain Naour <romain.naour@openwide.fr> wrote:
>>>> Hi Vicente,
>>>>
>>>> Le 21/08/2015 02:30, vj a écrit :
>>>>> Hello Romain,
>>>>> I've tested you efl update
>>>>> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
>>>>> It did not work, :(
>>>>> But applying the two patches below works again!
>>>>> The first one was already commented in a previous e-mail.
>>>>
>>>> Yes, sorry I haven't looked at your issue yet.
>>>>
>>>> libmount seems to be optional and can be disabled with --disable-libmount:
>>>> https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35
>>>
>>> Without that dependency it failed with 1.14.2.
>>> When I saw the dependency was not there in 1.15.0, I just added it
>>> again without checking.
>>> Just now I've tried it again without that patch and it's fine, so, you
>>> can forget it.
>>> Sorry for the hassle.
>>
>> It's ok ;-) I haven't heavily tested with a minimal config.
>> Maybe next time can you post the last ~100 build log lines, it not always easy
>> to reproduce a build issue...
>>
>>>
>>>>
>>>> But since it's not recommended to disable it, I'll apply your patch :)
>>>>
>>>>> The second is related to a regression in efl-1.15.0.
>>>>
>>>> Can you report your issue to the efl mailing list ?
>>>
>>> I know it would be better to have it fixed upstream, but have no time
>>> for that now.
>>> If you would like to, feel free to report it.
>>> To debug it, it helps enabling the WRN and DGB macros in eina_module.c.
>>
>> Well, I can't really report an issue that I can't reproduce.
> 
> Do you mean that the fb backend is working for you?

I mean I don't have this issue because I don't use the fb backend during my test.

> Are you crosscompiling with upstream vanilla gcc>=4 for arm?

I'm using a toolchain for ARM build by buildroot with a uClibc-ng.

> It might be a compiler bug too.
> I'm using musl gcc 4.9.3 build with buildroot itself.
> gcc 5 seems broken for arm crosscompilation.
> Just for completeness, the compiler I'm using was build with:
> buildroot.version=2ebbb7fe355c18a0be3d0fb8e50997142113c46b
> BR2_arm=y
> BR2_cortex_a8=y
> BR2_ARM_EABIHF=y
> BR2_ARM_FPU_NEON=y
> BR2_HOST_DIR="/opt/arm-buildroot-linux-musleabihf"
> BR2_OPTIMIZE_3=y
> BR2_TOOLCHAIN_BUILDROOT_MUSL=y
> BR2_BINUTILS_VERSION_2_25_X=y
> BR2_TOOLCHAIN_BUILDROOT_CXX=y
> BR2_GCC_ENABLE_LTO=y
> BR2_GCC_ENABLE_GRAPHITE=y
> BR2_TARGET_OPTIMIZATION="-march=armv7-a -mtune=cortex-a8
> -mcpu=cortex-a8 -mfpu=neon -mfloat-abi=hard"

Theses flags are handled directly by your target configuration in "Target
options" menu. You don't need to define them in BR2_TARGET_OPTIMIZATION.

> 
>> See, I reported the issue about SDL2 dependency and it's now fixed in the efl
>> 1.15.1. So, I can drop the last patch and avoid to autoreconf the package.
>>
>> Also, I'm probably not the good person to report an efl issue.
>> Honestly, I don't know what to do with -fvisibility=default.
>> It would be better if you can send just an email to the enlightenment mailing
>> list to report your issue.
> 
> Is "enlightenment-devel@lists.sourceforge.net" the correct mailing
> list for an efl bug?

Yes it is.

Best regards,
Romain

> 
>>>
>>>>
>>>>> The testing I've done has been with the musl libc and with
>>>>> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
>>>>> Basically the same config as in the previous e-mail.
>>>>>
>>>>> A minor issue: check the spelling of recommanded.
>>>>
>>>> Ha indeed recommanded is used in Config.in prompt
>>>> (my French was turned on sorry ;-) )
>>>>
>>>> Thanks for testing!
>>>
>>> Thanks for adding musl and maintaining efl!
>>
>> You're welcome!
>>
>> Best regards,
>> Romain
> 
> Regards,
>   Vicente.
>

Hello Romain,
this weekend I had some time to spend on this, sorry for the delay.

I've posted the EFL issue:
http://sourceforge.net/p/enlightenment/mailman/message/34493376

And tested your efl-1.15.1-v1 branch from
git://github.com/RomainNaour/buildroot.git
Aside from the EFL workaround (-fvisibility=default) the only
remaining issue is that when audio is disabled it fails to build
because of bae9cf8354208bdad7780e0d11e5b4c8cf1b1f3e.
I think it should be safe to just revert it because multisense is
already enabled by default and (as an audio-related feature) it only
makes sense when audio is also enabled.

Regards,
  Vicente.

Tested-by: Vicente Bergas <vicencb@gmail.com>

On Sat, Sep 5, 2015 at 3:55 PM, Romain Naour <romain.naour@openwide.fr> wrote:
> Hello Vicente,
>
> Sorry for the delay.
>
> Le 25/08/2015 00:09, vj a écrit :
>> Hello Romain,
>>
>> On Mon, Aug 24, 2015 at 10:42 PM, Romain Naour <romain.naour@openwide.fr> wrote:
>>> Hi Vicente
>>>
>>> Le 21/08/2015 19:56, vj a écrit :
>>>> Hello Romain,
>>>>
>>>> On Fri, Aug 21, 2015 at 10:51 AM, Romain Naour <romain.naour@openwide.fr> wrote:
>>>>> Hi Vicente,
>>>>>
>>>>> Le 21/08/2015 02:30, vj a écrit :
>>>>>> Hello Romain,
>>>>>> I've tested you efl update
>>>>>> https://github.com/RomainNaour/buildroot/tree/efl-1.15.0-v1
>>>>>> It did not work, :(
>>>>>> But applying the two patches below works again!
>>>>>> The first one was already commented in a previous e-mail.
>>>>>
>>>>> Yes, sorry I haven't looked at your issue yet.
>>>>>
>>>>> libmount seems to be optional and can be disabled with --disable-libmount:
>>>>> https://github.com/RomainNaour/buildroot/blob/efl-1.15.0-v1/package/efl/efl.mk#L35
>>>>
>>>> Without that dependency it failed with 1.14.2.
>>>> When I saw the dependency was not there in 1.15.0, I just added it
>>>> again without checking.
>>>> Just now I've tried it again without that patch and it's fine, so, you
>>>> can forget it.
>>>> Sorry for the hassle.
>>>
>>> It's ok ;-) I haven't heavily tested with a minimal config.
>>> Maybe next time can you post the last ~100 build log lines, it not always easy
>>> to reproduce a build issue...
>>>
>>>>
>>>>>
>>>>> But since it's not recommended to disable it, I'll apply your patch :)
>>>>>
>>>>>> The second is related to a regression in efl-1.15.0.
>>>>>
>>>>> Can you report your issue to the efl mailing list ?
>>>>
>>>> I know it would be better to have it fixed upstream, but have no time
>>>> for that now.
>>>> If you would like to, feel free to report it.
>>>> To debug it, it helps enabling the WRN and DGB macros in eina_module.c.
>>>
>>> Well, I can't really report an issue that I can't reproduce.
>>
>> Do you mean that the fb backend is working for you?
>
> I mean I don't have this issue because I don't use the fb backend during my test.
>
>> Are you crosscompiling with upstream vanilla gcc>=4 for arm?
>
> I'm using a toolchain for ARM build by buildroot with a uClibc-ng.
>
>> It might be a compiler bug too.
>> I'm using musl gcc 4.9.3 build with buildroot itself.
>> gcc 5 seems broken for arm crosscompilation.
>> Just for completeness, the compiler I'm using was build with:
>> buildroot.version=2ebbb7fe355c18a0be3d0fb8e50997142113c46b
>> BR2_arm=y
>> BR2_cortex_a8=y
>> BR2_ARM_EABIHF=y
>> BR2_ARM_FPU_NEON=y
>> BR2_HOST_DIR="/opt/arm-buildroot-linux-musleabihf"
>> BR2_OPTIMIZE_3=y
>> BR2_TOOLCHAIN_BUILDROOT_MUSL=y
>> BR2_BINUTILS_VERSION_2_25_X=y
>> BR2_TOOLCHAIN_BUILDROOT_CXX=y
>> BR2_GCC_ENABLE_LTO=y
>> BR2_GCC_ENABLE_GRAPHITE=y
>> BR2_TARGET_OPTIMIZATION="-march=armv7-a -mtune=cortex-a8
>> -mcpu=cortex-a8 -mfpu=neon -mfloat-abi=hard"
>
> Theses flags are handled directly by your target configuration in "Target
> options" menu. You don't need to define them in BR2_TARGET_OPTIMIZATION.
>
>>
>>> See, I reported the issue about SDL2 dependency and it's now fixed in the efl
>>> 1.15.1. So, I can drop the last patch and avoid to autoreconf the package.
>>>
>>> Also, I'm probably not the good person to report an efl issue.
>>> Honestly, I don't know what to do with -fvisibility=default.
>>> It would be better if you can send just an email to the enlightenment mailing
>>> list to report your issue.
>>
>> Is "enlightenment-devel@lists.sourceforge.net" the correct mailing
>> list for an efl bug?
>
> Yes it is.
>
> Best regards,
> Romain
>
>>
>>>>
>>>>>
>>>>>> The testing I've done has been with the musl libc and with
>>>>>> BR2_PACKAGE_EFL_RECOMMENDED_CONFIG unset.
>>>>>> Basically the same config as in the previous e-mail.
>>>>>>
>>>>>> A minor issue: check the spelling of recommanded.
>>>>>
>>>>> Ha indeed recommanded is used in Config.in prompt
>>>>> (my French was turned on sorry ;-) )
>>>>>
>>>>> Thanks for testing!
>>>>
>>>> Thanks for adding musl and maintaining efl!
>>>
>>> You're welcome!
>>>
>>> Best regards,
>>> Romain
>>
>> Regards,
>>   Vicente.
>>

Hi Vicente,

Le 27/09/2015 02:37, vj a écrit :
> Hello Romain,
> this weekend I had some time to spend on this, sorry for the delay.

That's ok, I guess you're working on your free time :)

> 
> I've posted the EFL issue:
> http://sourceforge.net/p/enlightenment/mailman/message/34493376

Glad to see you had some help on your error on framebuffer support.
If you send me a git formated patch that fixes your issue, be sure it'll be part
of the next efl series. (I hope, in the meantime, a patch will be applied
upstream and backported to 1.15 stable branch before the 1.15.2 release).

> 
> And tested your efl-1.15.1-v1 branch from
> git://github.com/RomainNaour/buildroot.git
> Aside from the EFL workaround (-fvisibility=default) the only
> remaining issue is that when audio is disabled it fails to build
> because of bae9cf8354208bdad7780e0d11e5b4c8cf1b1f3e.
> I think it should be safe to just revert it because multisense is
> already enabled by default and (as an audio-related feature) it only
> makes sense when audio is also enabled.

Thanks for the testing and the feedback!
The last patches from efl-1.15.1-v1 branch are WIP and won't be included yet in
the next Buildroot series. I'll take a closer look to multisense stuff.

> 
> Regards,
>   Vicente.
> 
> Tested-by: Vicente Bergas <vicencb@gmail.com>
> 

I already added your tested-by tag in the commit that bump efl for the target.
https://github.com/RomainNaour/buildroot/commit/04b24bc4e9b9c21209cbdb1110d0ae41991f838d

Also, tags are not taken into account by patchwork when you reply to a cover
letter. You can send several tags on a specific patch on the series by replying
to the email.

Best regards,
Romain

Hello Romain,

> Hi Vicente,
> 
> Le 27/09/2015 02:37, vj a =C3=A9crit :
>> Hello Romain,
>> this weekend I had some time to spend on this, sorry for the delay.
> 
> That's ok, I guess you're working on your free time :)
> 
>>=20
>> I've posted the EFL issue:
>> http://sourceforge.net/p/enlightenment/mailman/message/34493376
> 
> Glad to see you had some help on your error on framebuffer support.
> If you send me a git formated patch that fixes your issue, be sure it'll =
> be part
> of the next efl series. (I hope, in the meantime, a patch will be applied
> upstream and backported to 1.15 stable branch before the 1.15.2 release).

It seems that the issue was not posted soon enough to make it for 1.15.2,
so, here is the patch required for it to work.

> 
>>=20
>> And tested your efl-1.15.1-v1 branch from
>> git://github.com/RomainNaour/buildroot.git
>> Aside from the EFL workaround (-fvisibility=3Ddefault) the only
>> remaining issue is that when audio is disabled it fails to build
>> because of bae9cf8354208bdad7780e0d11e5b4c8cf1b1f3e.
>> I think it should be safe to just revert it because multisense is
>> already enabled by default and (as an audio-related feature) it only
>> makes sense when audio is also enabled.
> 
> Thanks for the testing and the feedback!
> The last patches from efl-1.15.1-v1 branch are WIP and won't be included =
> yet in
> the next Buildroot series. I'll take a closer look to multisense stuff.

I've spotted something extra:
package/efl/efl.mk:312: *** EFL_SITE (http://download.enlightenment.org/rel/libs/efl/) cannot have a trailing slash.  Stop.
Affected packages are:
econnman efl elemines empc enlightenment libcanberra libelementary libemotion-generic-players libetrophy libevas-generic-loaders python-efl rage terminology x11r7
This fixes it:
sed -i 's@\(_SITE = .*\)/$@\1@' $(grep -rl '_SITE = .*/$')

Regards,
  Vicente.

> 
>>=20
>> Regards,
>>   Vicente.
>>=20
>> Tested-by: Vicente Bergas <vicencb@gmail.com>
>>=20
> 
> I already added your tested-by tag in the commit that bump efl for the ta=
> rget.
> https://github.com/RomainNaour/buildroot/commit/04b24bc4e9b9c21209cbdb111=
> 0d0ae41991f838d
> 
> Also, tags are not taken into account by patchwork when you reply to a co=
> ver
> letter. You can send several tags on a specific patch on the series by re=
> plying
> to the email.
> 
> Best regards,
> Romain

Hi Vicente,

Le 05/10/2015 21:41, Vicente Bergas a écrit :
> Hello Romain,
> 
>> Hi Vicente,
>>
>> Le 27/09/2015 02:37, vj a =C3=A9crit :
>>> Hello Romain,
>>> this weekend I had some time to spend on this, sorry for the delay.
>>
>> That's ok, I guess you're working on your free time :)
>>
>>> =20
>>> I've posted the EFL issue:
>>> http://sourceforge.net/p/enlightenment/mailman/message/34493376
>>
>> Glad to see you had some help on your error on framebuffer support.
>> If you send me a git formated patch that fixes your issue, be sure it'll =
>> be part
>> of the next efl series. (I hope, in the meantime, a patch will be applied
>> upstream and backported to 1.15 stable branch before the 1.15.2 release).
> 
> It seems that the issue was not posted soon enough to make it for 1.15.2,
> so, here is the patch required for it to work.

Ok no problem, I'll add you patch in the next series.

> 
>>
>>> =20
>>> And tested your efl-1.15.1-v1 branch from
>>> git://github.com/RomainNaour/buildroot.git
>>> Aside from the EFL workaround (-fvisibility=3Ddefault) the only
>>> remaining issue is that when audio is disabled it fails to build
>>> because of bae9cf8354208bdad7780e0d11e5b4c8cf1b1f3e.
>>> I think it should be safe to just revert it because multisense is
>>> already enabled by default and (as an audio-related feature) it only
>>> makes sense when audio is also enabled.
>>
>> Thanks for the testing and the feedback!
>> The last patches from efl-1.15.1-v1 branch are WIP and won't be included =
>> yet in
>> the next Buildroot series. I'll take a closer look to multisense stuff.
> 
> I've spotted something extra:
> package/efl/efl.mk:312: *** EFL_SITE (http://download.enlightenment.org/rel/libs/efl/) cannot have a trailing slash.  Stop.
> Affected packages are:
> econnman efl elemines empc enlightenment libcanberra libelementary libemotion-generic-players libetrophy libevas-generic-loaders python-efl rage terminology x11r7
> This fixes it:
> sed -i 's@\(_SITE = .*\)/$@\1@' $(grep -rl '_SITE = .*/$')

Thanks for noticing it. Will fix.

Best regards,
Romain

> 
> Regards,
>   Vicente.
>