[Buildroot] [PATCH 1/2] New package: giblib

[Buildroot] [PATCH 1/2] New package: giblib

[yegorslists at googlemail.com]

From: Yegor Yefremov <yegorslists@googlemail.com>

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/Config.in        |    1 +
 package/giblib/Config.in |    8 ++++++++
 package/giblib/giblib.mk |   15 +++++++++++++++
 3 files changed, 24 insertions(+), 0 deletions(-)
 create mode 100644 package/giblib/Config.in
 create mode 100644 package/giblib/giblib.mk

diff --git a/package/Config.in b/package/Config.in
index 7a7b4bf..7f82934 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -280,6 +280,7 @@ source "package/cairo/Config.in"
 source "package/fltk/Config.in"
 source "package/fontconfig/Config.in"
 source "package/freetype/Config.in"
+source "package/giblib/Config.in"
 source "package/gtk2-engines/Config.in"
 source "package/gtk2-themes/Config.in"
 source "package/imlib2/Config.in"
diff --git a/package/giblib/Config.in b/package/giblib/Config.in
new file mode 100644
index 0000000..0c704e1
--- /dev/null
+++ b/package/giblib/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_GIBLIB
+	bool "giblib"
+	select BR2_PACKAGE_IMLIB2
+	help
+	  Giblib is a simple library which wraps imlib2.
+
+	  http://linuxbrit.co.uk/giblib/
+
diff --git a/package/giblib/giblib.mk b/package/giblib/giblib.mk
new file mode 100644
index 0000000..8a868c5
--- /dev/null
+++ b/package/giblib/giblib.mk
@@ -0,0 +1,15 @@
+#############################################################
+#
+# giblib
+#
+#############################################################
+GIBLIB_VERSION = 1.2.4
+GIBLIB_SOURCE = giblib-$(GIBLIB_VERSION).tar.gz
+GIBLIB_SITE = http://linuxbrit.co.uk/downloads/
+GIBLIB_INSTALL_STAGING = YES
+GIBLIB_DEPENDENCIES = imlib2
+GIBLIB_CONF_OPT = --with-imlib2-prefix=$(STAGING)/usr/lib \
+		  --with-imlib2-exec-prefix=$(STAGING)/usr/bin
+
+$(eval $(call AUTOTARGETS))
+
-- 
1.7.1.1

[Buildroot] [PATCH 2/2] New package: feh

[yegorslists at googlemail.com]

From: Yegor Yefremov <yegorslists@googlemail.com>

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/Config.in     |    1 +
 package/feh/Config.in |   12 ++++++++++++
 package/feh/feh.mk    |   22 ++++++++++++++++++++++
 3 files changed, 35 insertions(+), 0 deletions(-)
 create mode 100644 package/feh/Config.in
 create mode 100644 package/feh/feh.mk

diff --git a/package/Config.in b/package/Config.in
index 7f82934..b39f41b 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -133,6 +133,7 @@ source "package/fluxbox/Config.in"
 comment "X applications"
 source "package/alsamixergui/Config.in"
 source "package/docker/Config.in"
+source "package/feh/Config.in"
 source "package/gqview/Config.in"
 source "package/gmpc/Config.in"
 source "package/gob2/Config.in"
diff --git a/package/feh/Config.in b/package/feh/Config.in
new file mode 100644
index 0000000..d462cba
--- /dev/null
+++ b/package/feh/Config.in
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_FEH
+	bool "feh"
+	depends on BR2_PACKAGE_XLIB_LIBXINERAMA
+	depends on BR2_PACKAGE_XLIB_LIBXT
+	select BR2_PACKAGE_IMLIB2
+	select BR2_PACKAGE_IMLIB2_PNG
+	select BR2_PACKAGE_GIBLIB
+	select BR2_PACKAGE_LIBCURL
+	help
+	  feh is an X11 image viewer aimed mostly at console users.
+
+	  http://feh.finalrewind.org/
diff --git a/package/feh/feh.mk b/package/feh/feh.mk
new file mode 100644
index 0000000..b59bb2f
--- /dev/null
+++ b/package/feh/feh.mk
@@ -0,0 +1,22 @@
+FEH_VERSION = 2.0
+FEH_SOURCE = feh-$(FEH_VERSION).tar.bz2
+FEH_SITE = http://feh.finalrewind.org/
+FEH_DEPENDENCIES = libcurl giblib imlib2 libpng xlib_libXinerama xlib_libXt
+
+define FEH_BUILD_CMDS
+	$(MAKE1) CC="$(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS)" \
+		-C $(@D) all
+endef
+
+define FEH_INSTALL_TARGET_CMDS
+	$(MAKE1) CC="$(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS) " \
+		DESTDIR=$(TARGET_DIR) -C $(@D) install
+endef
+
+define FEH_UNINSTALL_TARGET_CMDS
+	$(MAKE1) CC="$(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS) " \
+		DESTDIR=$(TARGET_DIR) -C $(@D) uninstall
+endef
+
+$(eval $(call GENTARGETS))
+
-- 
1.7.1.1

[Buildroot] [PATCH 2/2] New package: feh

[Arnout Vandecappelle]

On Tuesday 11 October 2011 12:32:31, yegorslists at googlemail.com wrote:
> +config BR2_PACKAGE_FEH
> +       bool "feh"
> +       depends on BR2_PACKAGE_XLIB_LIBXINERAMA
> +       depends on BR2_PACKAGE_XLIB_LIBXT

 depends on BR2_PACKAGE_XORG7 and select these two?

> +       select BR2_PACKAGE_IMLIB2

 Redundant, giblib already selects imlib2.

> +       select BR2_PACKAGE_IMLIB2_PNG

 Here we have a problem that probably occurs in other places as well, and I 
don't see a good solution.

 The reason to select BR2_PACKAGE_IMLIB2_PNG is probably because ImLib2 is 
completely useless if you don't select any backend.  However, the choice for 
png is arbitrary.

 You could put a depends on instead of a select, but then the user has to go 
to two places to get it configured.

 You could repeat the backend options here, but then you get a lot of 
duplication.

 You could select all backends but then ImLib2 is overconfigured.

 You could make png default y for ImLib2, but then the user still has to go to 
two places if he needs gif instead of png.

 At least, the user should be directed to the configuration menu for ImLib2 so 
he can select the required backend there.

 One final option is to create a new top-level menu called 'Features' or 
something, which contains items like BR2_FEATURE_PNG, BR2_FEATURE_GIF.  
imlib2/Config.in could then set BR2_PACKAGE_IMLIB2_PNG default y if 
BR2_FEATURE_PNG.  But this feels a bit heavy-handed to me.


 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111011/cd876558/attachment.html>

[Buildroot] [PATCH 2/2] New package: feh

[Yegor Yefremov]

On Tue, Oct 11, 2011 at 7:29 PM, Arnout Vandecappelle <arnout@mind.be> wrote:
>
> On Tuesday 11 October 2011 12:32:31, yegorslists at googlemail.com wrote:
>
>> +config BR2_PACKAGE_FEH
>
>> +???????bool "feh"
>
>> +???????depends on BR2_PACKAGE_XLIB_LIBXINERAMA
>
>> +???????depends on BR2_PACKAGE_XLIB_LIBXT
>
> depends on BR2_PACKAGE_XORG7 and select these two?
>
>> +???????select BR2_PACKAGE_IMLIB2
>
> Redundant, giblib already selects imlib2.
>
>> +???????select BR2_PACKAGE_IMLIB2_PNG
>
> Here we have a problem that probably occurs in other places as well, and I
> don't see a good solution.
>
> The reason to select BR2_PACKAGE_IMLIB2_PNG is probably because ImLib2 is
> completely useless if you don't select any backend. However, the choice for
> png is arbitrary.
>
> You could put a depends on instead of a select, but then the user has to go
> to two places to get it configured.
>
> You could repeat the backend options here, but then you get a lot of
> duplication.
>
> You could select all backends but then ImLib2 is overconfigured.
>
> You could make png default y for ImLib2, but then the user still has to go
> to two places if he needs gif instead of png.
>
> At least, the user should be directed to the configuration menu for ImLib2
> so he can select the required backend there.
>
> One final option is to create a new top-level menu called 'Features' or
> something, which contains items like BR2_FEATURE_PNG, BR2_FEATURE_GIF.
> imlib2/Config.in could then set BR2_PACKAGE_IMLIB2_PNG default y if
> BR2_FEATURE_PNG. But this feels a bit heavy-handed to me.

Yes. Such cases are rather tricky. Let's wait for comments from Peter
and Thomas.

Yegor