[Buildroot] [PATCH v5 24/36] package/enlightenment: bump to version 0.19.12
Romain Naour
romain.naour at openwide.fr
Sat Oct 31 14:10:38 UTC 2015
Hi Yann, All,
Le 25/10/2015 17:28, Yann E. MORIN a écrit :
> Romain, All,
>
> On 2015-10-24 23:01 +0200, Romain Naour spake thusly:
>> Use the newly added libefl package wich provide a new version
>> of efl libraries.
>> This new version needs libelementary dependency.
>>
>> Update upstream url, add hash file and use xz archive.
>>
>> Select png and jpeg image loader since enlightenment_start
>> fail at runtime if png and jpeg image loader are not
>> available in elf libraries.
>>
>> Signed-off-by: Romain Naour <romain.naour at openwide.fr>
>> ---
>> v5: Use depends on BR2_PACKAGE_EFL instead of select (ThomasP)
>> remove comment on dependencies
>> v4: update Config.in to use libefl
>> improve commit log
>> keep the dependency on threads
>> v3: bymp to 0.19.12
>> remove trailing slash (Vicente Bergas)
>> v2: bump to 0.19.9
>> remove glibc only dependency
>> ---
>> package/enlightenment/Config.in | 36 +++++++-------------------------
>> package/enlightenment/enlightenment.hash | 2 ++
>> package/enlightenment/enlightenment.mk | 22 +++++++------------
>> 3 files changed, 17 insertions(+), 43 deletions(-)
>> create mode 100644 package/enlightenment/enlightenment.hash
>>
>> diff --git a/package/enlightenment/Config.in b/package/enlightenment/Config.in
>> index 58f8f39..4a349fb 100644
>> --- a/package/enlightenment/Config.in
>> +++ b/package/enlightenment/Config.in
>> @@ -1,33 +1,16 @@
>> config BR2_PACKAGE_ENLIGHTENMENT
>> bool "enlightenment"
>> - select BR2_PACKAGE_EFL
>> - select BR2_PACKAGE_LIBECORE
>> - select BR2_PACKAGE_LIBECORE_X
>> - select BR2_PACKAGE_LIBEINA
>> - select BR2_PACKAGE_LIBEET
>> - select BR2_PACKAGE_LIBEVAS
>> - select BR2_PACKAGE_LIBEVAS_JPEG
>> - select BR2_PACKAGE_LIBEVAS_PNG
>> - select BR2_PACKAGE_LIBEVAS_EET
>> - select BR2_PACKAGE_LIBECORE_EVAS
>> - select BR2_PACKAGE_LIBEDJE
>> - select BR2_PACKAGE_LIBEFREET
>> - select BR2_PACKAGE_LIBEDBUS
>> - select BR2_PACKAGE_LIBEIO
>> - select BR2_PACKAGE_XCB_UTIL_KEYSYMS
>> + select BR2_PACKAGE_LIBEFL_X
>> + select BR2_PACKAGE_LIBEFL_JPEG # needed at runtime by enlightenment_start
>> + select BR2_PACKAGE_LIBEFL_PNG # needed at runtime by enlightenment_start
>> + select BR2_PACKAGE_LIBELEMENTARY
>> select BR2_PACKAGE_LIBEVAS_GENERIC_LOADERS
>> select BR2_PACKAGE_LIBEVAS_GENERIC_LOADERS_SVG
>> - # libevas-generic-loaders-svg -> librsvg -> glib2 / pango
>> - depends on BR2_USE_WCHAR
>> - depends on BR2_INSTALL_LIBSTDCPP
>> - # libedbus -> dbus
>> - # libedbus -> glib2, libevas
>> - depends on BR2_TOOLCHAIN_HAS_THREADS
>> - # libedbus -> dbus, efl libraries
>> - depends on BR2_USE_MMU
>> - depends on BR2_PACKAGE_XORG7
>> + select BR2_PACKAGE_XCB_UTIL_KEYSYMS
>> # libevas-generic-loaders-svg -> librsvg -> pango -> cairo
>> depends on BR2_ARCH_HAS_ATOMICS
>> + depends on BR2_PACKAGE_EFL
>> + depends on BR2_PACKAGE_XORG7
>
> I really do not like those depends that go after the selects. It is
> more complex to read.
>
> Even though the ordering has absolutely no consequences in Kconfig, it
> is not logical (when reading). It is more logical and far easier to
> understand when depends come first, followed by the selects.
>
> (i.e it runs as thus in my head: this stuff needs foo and bar, so it
> depends on them. Then, once dependenecies are fullfilled, I can select
> other stuff.)
>
> Note: it is not your fault, the code was already laid out thusly before
> your patch. Still, this is looking ugly to me.
Ok, I'll add some clean up patches then.
>
>> diff --git a/package/enlightenment/enlightenment.mk b/package/enlightenment/enlightenment.mk
>> index ddebe06..9127099 100644
>> --- a/package/enlightenment/enlightenment.mk
>> +++ b/package/enlightenment/enlightenment.mk
>> @@ -4,24 +4,18 @@
>> #
>> ################################################################################
>>
>> -ENLIGHTENMENT_VERSION = 0.17.6
>> -ENLIGHTENMENT_SITE = http://download.enlightenment.org/releases
>> +ENLIGHTENMENT_VERSION = 0.19.12
>> +ENLIGHTENMENT_SOURCE = enlightenment-$(ENLIGHTENMENT_VERSION).tar.xz
>> +ENLIGHTENMENT_SITE = http://download.enlightenment.org/rel/apps/enlightenment
>> ENLIGHTENMENT_LICENSE = BSD-2c
>> ENLIGHTENMENT_LICENSE_FILES = COPYING
>>
>> -ENLIGHTENMENT_DEPENDENCIES = \
>> - host-pkgconf \
>> - libecore \
>> - libeet \
>> - libeina \
>> - libevas \
>> +ENLIGHTENMENT_DEPENDENCIES = \
>> + host-pkgconf \
>> + host-libefl \
>> + libefl \
>> + libelementary \
>> libevas-generic-loaders \
>> - libedje \
>> - libefreet \
>> - libedbus \
>> - libeio \
>> - host-libedje \
>> - host-libeet \
>> xcb-util-keysyms
>>
>> ENLIGHTENMENT_CONF_OPTS = \
>
> A bit later in that file, we have:
>
> # alsa backend needs mixer support
> ifeq ($(BR2_PACKAGE_ALSA_LIB)$(BR2_PACKAGE_ALSA_LIB_MIXER),yy)
> ENLIGHTENMENT_DEPENDENCIES += alsa-lib
Probably I should add :
ENLIGHTENMENT_CONF_ENV += enable_alsa=yes
> else
> ENLIGHTENMENT_CONF_ENV += enable_alsa=no
> endif
>
> However, you previously said, when introducing libefl, that alsa support
> was not working at all (even when alsa-lib was found).
>
> How does that affect the enlightment package?
It seems that there is a optional module mixer which use alsa.
As far I understand this support doesn't come from efl libraries.
Also I added --disable-pam and --disable-systemd to ENLIGHTENMENT_CONF_OPTS
since the support is auto detected.
Best regards,
Romain
>
> Regards,
> Yann E. MORIN.
>
More information about the buildroot
mailing list