[Buildroot] [PATCH v3] Added lesstif support

[Buildroot] [PATCH v3] Added lesstif support

[Thierry Bultel]

This patch adds support for lesstif.
This is a quite old thing, latest version is from 2009
but it is useful to applications based on xmotif.
It has been tested with buildroot on ARM/iMX6.

Signed-off-by Thierry Bultel <thierry.bultel@wanadoo.fr>
---
Changes v2 -> v3
  - Text alignment changes merged into a single patch
Changes v1 -> v2
  - Fixed the way the aclocal file is handled: The lesstif makefile is patched in order not to install it.
  - Changed the location in xconfig, lesstif is now under 'Libraries' -> 'Graphics' (suggested by Thomas).
  - Fixes the indentation
  - Added dependency to freetype
  - Match dependencies in Config.in and lesstif.mk (Suggested by Peter) 

[Buildroot] [PATCH v3] Added lesstif support

[Thierry Bultel]

---
 package/Config.in          |    1 +
 package/lesstif/Config.in  |   10 ++++++++++
 package/lesstif/lesstif.mk |   41 +++++++++++++++++++++++++++++++++++++++++
 3 files changed, 52 insertions(+), 0 deletions(-)
 create mode 100644 package/lesstif/Config.in
 create mode 100644 package/lesstif/lesstif.mk

diff --git a/package/Config.in b/package/Config.in
index 1dd60d1..dc44609 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -498,6 +498,7 @@ source "package/harfbuzz/Config.in"
 source "package/imlib2/Config.in"
 source "package/jpeg/Config.in"
 source "package/lcms2/Config.in"
+source "package/lesstif/Config.in"
 source "package/libart/Config.in"
 source "package/libdmtx/Config.in"
 source "package/libdrm/Config.in"
diff --git a/package/lesstif/Config.in b/package/lesstif/Config.in
new file mode 100644
index 0000000..3a999ea
--- /dev/null
+++ b/package/lesstif/Config.in
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_LESSTIF
+	bool "lesstif"
+	depends on BR2_PACKAGE_XORG7
+	select BR2_PACKAGE_XLIB_LIBXT
+	select BR2_PACKAGE_XLIB_LIBXEXT
+	select BR2_PACKAGE_FREETYPE
+	help
+	  lesstif is the Hungry Programmers' version of OSF/Motif
+
+	  http://lesstif.sourceforge.net/
diff --git a/package/lesstif/lesstif.mk b/package/lesstif/lesstif.mk
new file mode 100644
index 0000000..e912554
--- /dev/null
+++ b/package/lesstif/lesstif.mk
@@ -0,0 +1,41 @@
+################################################################################
+#
+# lesstif
+#
+################################################################################
+LESSTIF_VERSION = 0.95.2
+LESSTIF_SOURCE  = lesstif-$(LESSTIF_VERSION).tar.bz2
+LESSTIF_SITE    = http://downloads.sourceforge.net/project/lesstif/lesstif/$(LESSTIF_VERSION)
+LESSTIF_INSTALL_STAGING = YES
+LESSTIF_DEPENDENCIES = \
+	xlib_libXt \
+	xlib_libXext \
+	freetype
+
+LESSTIF_CONF_OPT = \
+	--with-gnu-ld \
+	--with-freetype-config=$(STAGING_DIR)/usr/bin/freetype-config \
+	--enable-debug=no \
+	--enable-production=yes	\
+	--enable-build-tests=no
+
+# Reduces the buggy makefile to the smallest possible (and working) thing
+define LESSTIF_NOMAN2HTML
+	echo "all:" 	> $(@D)/doc/Makefile
+	echo "" 		>> $(@D)/doc/Makefile
+	echo "install:" >> $(@D)/doc/Makefile
+	echo "" 		>> $(@D)/doc/Makefile
+	echo "clean:" 	>> $(@D)/doc/Makefile
+endef
+
+# Prevents to copy ac_find_motif.m4 on target, it would else
+# be created at $(TARGET_DIR)/$(TOPDIR)/output/host/usr/share/aclocal/ac_find_motif.m4
+
+define LESSTIF_FIXACLOCAL
+	sed -i -e "/install-data-am: install-aclocalDATA/d" $(@D)/scripts/autoconf/Makefile
+endef
+
+LESSTIF_POST_CONFIGURE_HOOKS += LESSTIF_NOMAN2HTML
+LESSTIF_POST_CONFIGURE_HOOKS += LESSTIF_FIXACLOCAL
+
+$(eval $(autotools-package))
-- 
1.7.1

[Buildroot] [PATCH v3] Added lesstif support

[Peter Korsgaard]

>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:

The commit message should have been in this email.

I've added it and committed, thanks.

I had to pass --no-recursion to not have it error out in test/configure
though.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH v3] Added lesstif support

[Thierry Bultel]

Le 01/10/2013 23:24, Peter Korsgaard a ?crit :
>>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:
> 
> The commit message should have been in this email.
> 
> I've added it and committed, thanks.
> 
> I had to pass --no-recursion to not have it error out in test/configure
> though.
> 

Thanks Peter,

could you please be more specific about that --no-recustion issue that I
have missed ?

Regards
Thierry

[Buildroot] [PATCH v3] Added lesstif support

[Peter Korsgaard]

>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:

 Thierry> Le 01/10/2013 23:24, Peter Korsgaard a ?crit :
 >>>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:
 >> 
 >> The commit message should have been in this email.
 >> 
 >> I've added it and committed, thanks.
 >> 
 >> I had to pass --no-recursion to not have it error out in test/configure
 >> though.
 >> 

 Thierry> Thanks Peter,

 Thierry> could you please be more specific about that --no-recustion
 Thierry> issue that I have missed ?

Without --no-recursion, ./configure executes test/configure at the end,
which error'ed out with missing XPrint headers (as far as I
remember). Perhaps you have a bunch of X development packages installed
on your build machine?

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH v3] Added lesstif support

[Thierry Bultel]

Le 02/10/2013 12:37, Peter Korsgaard a ?crit :
>>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:
> 
>  Thierry> Le 01/10/2013 23:24, Peter Korsgaard a ?crit :
>  >>>>>>> "Thierry" == Thierry Bultel <thierry.bultel@wanadoo.fr> writes:
>  >> 
>  >> The commit message should have been in this email.
>  >> 
>  >> I've added it and committed, thanks.
>  >> 
>  >> I had to pass --no-recursion to not have it error out in test/configure
>  >> though.
>  >> 
> 
>  Thierry> Thanks Peter,
> 
>  Thierry> could you please be more specific about that --no-recustion
>  Thierry> issue that I have missed ?
> 
> Without --no-recursion, ./configure executes test/configure at the end,
> which error'ed out with missing XPrint headers (as far as I
> remember). Perhaps you have a bunch of X development packages installed
> on your build machine?
> 

Yes indeed.
Thanks for having noticed it.