[Buildroot] [PATCH v3] package/zile: add Zile is Lossy Emacs editor

[Alex Bennée]

Thomas Petazzoni <thomas.petazzoni at free-electrons.com> writes:

> Dear Alex Bennée,
<snip>
>> +++ b/package/zile/Config.in
>> @@ -0,0 +1,13 @@
>> +config BR2_PACKAGE_ZILE
>> +	bool "zile"
>> +	select BR2_PACKAGE_LIBGC
>> +	depends on BR2_USE_MMU # fork()
>
> You need to replicate the dependencies of BR2_PACKAGE_LIBGC here.

Surely the whole point of dependencies is the LIBGC ones get pulled in
when you select LIBGC, or do you mean I need:

     select BR2_PACKAGE_LIBGC
     depends on BR2_PACKAGE_LIBGC
     depends on BR2_USE_MMU # fork()

> Also, when testing your package, it failed because ncurses was not
> available, so I had to add ncurses as a dependency here.
>
> This actually leads me to wonder if you actually tested this stuff. How
> would it be possible to miss the ncurses dependency?

I have a bunch of benchmarking tools enabled, one of them must of pulled
it in. I have been testing in a qemu-system aarch64 initrd rootfs and it
was working!

>
> Also, some of the configure messages need some investigation:
>
> configure: WARNING: libacl development library was not found or not
> usable. configure: WARNING: Zile will be built without ACL support.
>
>  => this probably means that an optional dependency on BR2_PACKAGE_ACL,
>  with a --enable-acl / --disable-acl as appropriate

I'll look at that.

>
> checking whether btowc is declared without a macro... yes
> checking whether wctob is declared without a macro... yes
> checking whether mbsinit is declared without a macro... yes
>
>  => this smells like a dependency on wide-char support. Can you test
>  your package with a toolchain that does not have wide char support,
>  such as
>  http://autobuild.buildroot.org/toolchains/configs/free-electrons/br-arm-basic.config.

I've got to build some armv7 images so I'll do that then.

>
> checking for emacs... /usr/bin/emacs
> checking Emacs version 24.3.50.1 >= 23.1... yes
>
>  => is it normal that it references my host emacs? what is this used
>  for?

No idea. Possibly too much autoconf boilerplate. I shall have a look.

>
>
>> diff --git a/package/zile/zile-0001-remove-help2man.patch b/package/zile/zile-0001-remove-help2man.patch
>> new file mode 100644
>> index 0000000..d397543
>> --- /dev/null
>> +++ b/package/zile/zile-0001-remove-help2man.patch
>> @@ -0,0 +1,30 @@
>> +Upstream-Status: Inappropriate [configuration]
>> +
>> +Signed-off-by: Alex Bennée <alex.bennee at linaro.org>
>
> Since you're passing --without-help2man, why is this patch useful?

IIRC without that patch it still attempted to call the help2man tool
even though it wasn't generating the final output. 

>
>> diff --git a/package/zile/zile.mk b/package/zile/zile.mk
>> new file mode 100644
>> index 0000000..07226c6
>> --- /dev/null
>> +++ b/package/zile/zile.mk
>> @@ -0,0 +1,15 @@
>> +################################################################################
>> +#
>> +# Zile
>> +#
>> +################################################################################
>> +
>> +ZILE_VERSION = 2.4.11
>> +ZILE_SITE = $(BR2_GNU_MIRROR)/zile
>> +ZILE_LICENSE = GPLv1+
>
> Have you actually read the COPYING file?

Ahh ok - I shall fix that up.

>
>> +ZILE_LICENSE_FILES = COPYING
>> +ZILE_AUTORECONF = YES
>> +ZILE_CONF_OPT = --without-help2man
>> +ZILE_DEPENDENCIES = libgc
>
> Missing ncurses library.
>
> Also, when I build your package with the toolchain at
> http://autobuild.buildroot.org/toolchains/configs/free-electrons/br-arm-full.config,
> it fails to build with:
>
> /home/thomas/projets/buildroot/output/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-uclibcgnueabi/4.7.3/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld: /home/thomas/projets/buildroot/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libgc.so:
> undefined reference to symbol 'pthread_cancel'
> /home/thomas/projets/buildroot/output/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-uclibcgnueabi/4.7.3/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld:
> note: 'pthread_cancel' is defined in DSO /home/thomas/projets/buildroot/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/lib/libpthread.so.0 so try adding it to the linker command line
> /home/thomas/projets/buildroot/output/host/usr/arm-buildroot-linux-uclibcgnueabi/sysroot/lib/libpthread.so.0:
> could not read symbols: Invalid operation
>
> Could you work on these remaining issues, and resubmit an updated
> version?
>
> Thanks!
>
> Thomas

-- 
Alex Bennée