[Buildroot] ParaGUI package?

[Buildroot] ParaGUI package?

[Hartley Sweeten]

Hello all,

Has anyone tried to create a package for ParaGUI?

I'm trying to find a simple GUI library that can be used with SDL and this one appears
to cover my needs. Unfortunately the ParaGUI library appears to be a bit dead. The
latest release at http://download.savannah.gnu.org/releases/paragui/ is

paragui-1.1.8.tar.gz	19-Oct-2004

I found a newer branch on https://github.com/pipelka/paragui that appears to be
version 1.1.11.

I tried creating a .mk for it but have some problems getting the auto reconf to work
correctly. There is an autogen.sh in the package that correctly does the auto reconf
when I try it local on my system but I can't get it to build with Buildroot.

Here's the .mk I have so far:


################################################################################
#
# paragui
#
################################################################################

PARAGUI_VERSION = 26365b27d184ec118057bbd9bc5ef65bc6b6f5b7
PARAGUI_SITE = $(call github,pipelka,paragui,$(PARAGUI_VERSION))
PARAGUI_LICENSE = GPLv2+
PARAGUI_LICENSE_FILES = COPYING

PARAGUI_AUTORECONF = YES

PARAGUI_INSTALL_STAGING = YES
PARAGUI_DEPENDENCIES = sdl
PARAGUI_CONF_OPT = \
	--with-sdl-prefix=$(STAGING_DIR)/usr \
	--disable-debug \
	--enable-internalphysfs

ifeq ($(BR2_PACKAGE_SDL_IMAGE),y)
PARAGUI_CONF_OPT += --enable-sdl_image
PARAGUI_DEPENDENCIES += sdl_image
else
PARAGUI_CONF_OPT += --disable-sdl_image
endif

ifeq ($(BR2_PACKAGE_EXPAT),y)
PARAGUI_CONF_OPT += --disable-internalexpat
PARAGUI_DEPENDENCIES += expat
else
PARAGUI_CONF_OPT += --enable-internalexpat
endif

$(eval $(autotools-package))


The Autoreconfiguring is failing like this:

>>> paragui 26365b27d184ec118057bbd9bc5ef65bc6b6f5b7 Autoreconfiguring
aclocal: warning: autoconf input should be named 'configure.ac', not 'configure.in'
aclocal: warning: autoconf input should be named 'configure.ac', not 'configure.in'
libtoolize: putting auxiliary files in `.'.
libtoolize: copying file `./ltmain.sh'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
aclocal: warning: autoconf input should be named 'configure.ac', not 'configure.in'
automake: warning: autoconf input should be named 'configure.ac', not 'configure.in'
configure.in:48: warning: AM_INIT_AUTOMAKE: two- and three-arguments forms are deprecated.  For more info, see:
configure.in:48: http://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation
configure.in:55: installing './compile'
configure.in:42: installing './config.guess'
configure.in:42: installing './config.sub'
configure.in:48: installing './install-sh'
configure.in:48: installing './missing'
archivers/Makefile.am:4: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
automake: warning: autoconf input should be named 'configure.ac', not 'configure.in'
archivers/Makefile.am: installing './depcomp'
platform/Makefile.am:3: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
test/Makefile.am:5: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
libtoolize: putting auxiliary files in `.'.
libtoolize: copying file `./ltmain.sh'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
aclocal: warning: autoconf input should be named 'configure.ac', not 'configure.in'
automake: warning: autoconf input should be named 'configure.ac', not 'configure.in'
configure.in:55: warning: AM_INIT_AUTOMAKE: two- and three-arguments forms are deprecated.  For more info, see:
configure.in:55: http://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation
configure.in:63: installing './compile'
configure.in:51: installing './config.guess'
configure.in:51: installing './config.sub'
configure.in:55: installing './install-sh'
configure.in:55: installing './missing'
Makefile.am: error: required file './NEWS' not found
Makefile.am: error: required file './ChangeLog' not found
automake: warning: autoconf input should be named 'configure.ac', not 'configure.in'
bindings/python/Makefile.am:5: warning: shell ls $(top_srcdir: non-POSIX variable name
bindings/python/Makefile.am:5: (probably a GNU make extension)
bindings/python/Makefile.am:16: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
bindings/ruby/Makefile.am:8: warning: shell ls $(top_srcdir: non-POSIX variable name
bindings/ruby/Makefile.am:8: (probably a GNU make extension)
bindings/ruby/Makefile.am:15: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/Makefile.am:28: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/Makefile.am: installing './depcomp'
src/core/Makefile.am:30: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/draw/Makefile.am:16: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/expat/Makefile.am:9: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/font/Makefile.am:10: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/gp2x/Makefile.am:7: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/themes/Makefile.am:7: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/widgets/Makefile.am:43: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
src/ystring/Makefile.am:9: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
test/Makefile.am:107: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
test/colorselector/Makefile.am:16: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
test/fireworks/Makefile.am:16: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
test/keyboard/Makefile.am:16: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
autoreconf: /home/bigguiness/rpc353/buildroot/output/host/usr/bin/automake failed with exit status: 1
make: *** [/home/bigguiness/rpc353/buildroot/output/build/paragui-26365b27d184ec118057bbd9bc5ef65bc6b6f5b7/.stamp_configured] Error 1


Does anyone have any ideas or suggestions to get this library to build?

Thanks,
Hartley

[Buildroot] ParaGUI package?

[Thomas Petazzoni]

Dear Hartley Sweeten,

On Wed, 8 Jan 2014 00:10:29 +0000, Hartley Sweeten wrote:

> I'm trying to find a simple GUI library that can be used with SDL and this one appears
> to cover my needs. Unfortunately the ParaGUI library appears to be a bit dead. The
> latest release at http://download.savannah.gnu.org/releases/paragui/ is
> 
> paragui-1.1.8.tar.gz	19-Oct-2004
> 
> I found a newer branch on https://github.com/pipelka/paragui that appears to be
> version 1.1.11.

My first reaction is: do you really want to start using something that
appears to be dead/deprecated?

> Makefile.am: error: required file './NEWS' not found
> Makefile.am: error: required file './ChangeLog' not found

These seems to be the real problems. Easiest solution is to add a post
patch hook with "touch $(@D)/NEWS $(@D)/ChangeLog"

A cleaner solution is to add the "foreign" option in the
AM_INIT_AUTOMAKE call in configure.in, like:

AM_INIT_AUTOMAKE([1.10 no-define foreign])

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] ParaGUI package?

[Hartley Sweeten]

On Tuesday, January 07, 2014 8:34 PM, Thomas Petazzoni wrote:
> On Wed, 8 Jan 2014 00:10:29 +0000, Hartley Sweeten wrote:
>> I'm trying to find a simple GUI library that can be used with SDL and this one appears
>> to cover my needs. Unfortunately the ParaGUI library appears to be a bit dead. The
>> latest release at http://download.savannah.gnu.org/releases/paragui/ is
>> 
>> paragui-1.1.8.tar.gz	19-Oct-2004
>> 
>> I found a newer branch on https://github.com/pipelka/paragui that appears to be
>> version 1.1.11.
>
> My first reaction is: do you really want to start using something that
> appears to be dead/deprecated?

Not really.. But, I have not been able to find anything else that fits my needs.

>> Makefile.am: error: required file './NEWS' not found
>> Makefile.am: error: required file './ChangeLog' not found
>
> These seems to be the real problems. Easiest solution is to add a post
> patch hook with "touch $(@D)/NEWS $(@D)/ChangeLog"
>
> A cleaner solution is to add the "foreign" option in the
> AM_INIT_AUTOMAKE call in configure.in, like:
>
> AM_INIT_AUTOMAKE([1.10 no-define foreign])

I was able to get it to compile under Buildroot yesterday. Now I just need to
figure out what I did to make it work...

One issue I had was:

aclocal: warning: autoconf input should be named 'configure.ac', not 'configure.in'

The only way I could make this go away was by renaming the configure.in files.
Is this warning normal? I don't remember seeing it with any other packages in
Buildroot.

Thanks,
Hartley

[Buildroot] ParaGUI package?

[Arnout Vandecappelle]

On 08/01/14 04:34, Thomas Petazzoni wrote:
> Dear Hartley Sweeten,
>
> On Wed, 8 Jan 2014 00:10:29 +0000, Hartley Sweeten wrote:
>
>> I'm trying to find a simple GUI library that can be used with SDL and this one appears
>> to cover my needs. Unfortunately the ParaGUI library appears to be a bit dead. The
>> latest release at http://download.savannah.gnu.org/releases/paragui/ is
>>
>> paragui-1.1.8.tar.gz	19-Oct-2004
>>
>> I found a newer branch on https://github.com/pipelka/paragui that appears to be
>> version 1.1.11.
>
> My first reaction is: do you really want to start using something that
> appears to be dead/deprecated?
>
>> Makefile.am: error: required file './NEWS' not found
>> Makefile.am: error: required file './ChangeLog' not found
>
> These seems to be the real problems. Easiest solution is to add a post
> patch hook with "touch $(@D)/NEWS $(@D)/ChangeLog"

  Actually, perhaps we should always to this when PKG_AUTORECONF is YES? 
There are quite a lot of packages that have this touch trick.

  Regards,
  Arnout

>
> A cleaner solution is to add the "foreign" option in the
> AM_INIT_AUTOMAKE call in configure.in, like:
>
> AM_INIT_AUTOMAKE([1.10 no-define foreign])
>
> Thomas
>


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] ParaGUI package?

[Thomas Petazzoni]

Dear Arnout Vandecappelle,

On Thu, 09 Jan 2014 18:01:24 +0100, Arnout Vandecappelle wrote:

> >> Makefile.am: error: required file './NEWS' not found
> >> Makefile.am: error: required file './ChangeLog' not found
> >
> > These seems to be the real problems. Easiest solution is to add a post
> > patch hook with "touch $(@D)/NEWS $(@D)/ChangeLog"
> 
>   Actually, perhaps we should always to this when PKG_AUTORECONF is YES? 
> There are quite a lot of packages that have this touch trick.

I think I'd prefer these packages to be fixed upstream by adding the
"foreign" argument to AM_INIT_AUTOMAKE. It's not like a patch doing
this should be hard to get merged upstream, no?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] ParaGUI package?

[Hartley Sweeten]

On Thursday, January 09, 2014 10:01 AM, Arnout Vandecappelle wrote:
> On 08/01/14 04:34, Thomas Petazzoni wrote:
>> On Wed, 8 Jan 2014 00:10:29 +0000, Hartley Sweeten wrote:
>>> I'm trying to find a simple GUI library that can be used with SDL and this one appears
>>> to cover my needs. Unfortunately the ParaGUI library appears to be a bit dead. The
>>> latest release at http://download.savannah.gnu.org/releases/paragui/ is
>>>
>>> paragui-1.1.8.tar.gz	19-Oct-2004
>>>
>>> I found a newer branch on https://github.com/pipelka/paragui that appears to be
>>> version 1.1.11.
>>
>> My first reaction is: do you really want to start using something that
>> appears to be dead/deprecated?
>>
>>> Makefile.am: error: required file './NEWS' not found
>>> Makefile.am: error: required file './ChangeLog' not found
>>
>> These seems to be the real problems. Easiest solution is to add a post
>> patch hook with "touch $(@D)/NEWS $(@D)/ChangeLog"
>
>  Actually, perhaps we should always to this when PKG_AUTORECONF is YES? 
> There are quite a lot of packages that have this touch trick.

I was finally able to get paragui to compile with Buildroot using a POST_PATCH_HOOKS
like was just done for the sdl package:

define PARAGUI_RUN_AUTOGEN
	cd $(@D) && ./autogen.sh
endef

PARAGUI_POST_PATCH_HOOKS += PARAGUI_RUN_AUTOGEN

Doing this I only needed 2 patches.
1) to use the correct sdl-config
2) to include <ostream> in a source file (I think this is needed because of a gcc 4.3.3 bug)

I was also able to get the package to compile using AUTORECONF = YES but that
needed a lot more patches.

I can post the patch to add the paragui package but there is one issue. ParaGUI requires
libsigc++ 1.2 (>= 1.2.5) but the current libsigc package in Buildroot is for libsigc++ 2.2 (2.2.11).
The API is quite different between the 1.2 and 2.2 versions. So to get paragui to compile I
had to also add a libsigc1 package (leaving the current libsigc as-is).

Is this acceptable? If so I can post 2 patches:

1) add package libsigc1
2) add package paragui

Thanks,
Hartley

[Buildroot] ParaGUI package?

[Thomas Petazzoni]

Dear Hartley Sweeten,

On Thu, 9 Jan 2014 22:37:40 +0000, Hartley Sweeten wrote:

> I was finally able to get paragui to compile with Buildroot using a POST_PATCH_HOOKS
> like was just done for the sdl package:
> 
> define PARAGUI_RUN_AUTOGEN
> 	cd $(@D) && ./autogen.sh
> endef
> 
> PARAGUI_POST_PATCH_HOOKS += PARAGUI_RUN_AUTOGEN

Humm, ok. This is not the nicest thing in the world, but oh well.

> Doing this I only needed 2 patches.
> 1) to use the correct sdl-config
> 2) to include <ostream> in a source file (I think this is needed because of a gcc 4.3.3 bug)
> 
> I was also able to get the package to compile using AUTORECONF = YES but that
> needed a lot more patches.

Which patches?

> 
> I can post the patch to add the paragui package but there is one issue. ParaGUI requires
> libsigc++ 1.2 (>= 1.2.5) but the current libsigc package in Buildroot is for libsigc++ 2.2 (2.2.11).
> The API is quite different between the 1.2 and 2.2 versions. So to get paragui to compile I
> had to also add a libsigc1 package (leaving the current libsigc as-is).
> 
> Is this acceptable? If so I can post 2 patches:

Yes, packaging libsigc1 separately is OK, since it has a different API
than libsigc.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com