* [Buildroot] [PATCH] libiconv: bump to 1.14
From: Peter Korsgaard @ 2011-10-05 19:36 UTC (permalink / raw)
To: buildroot
In-Reply-To: <4E79BB4A.1040608@visionsystems.de>
>>>>> "Yegor" == Yegor Yefremov <yegor_sub1@visionsystems.de> writes:
Yegor> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [git commit] Add the picocom package
From: Peter Korsgaard @ 2011-10-05 19:45 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=08540f43b63abccfc243834c637a5624ef69952f
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
[Peter: fix Config.in help text indentation]
Signed-off-by: Maxime Ripard <maxime@free-electrons.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
package/Config.in | 1 +
package/picocom/Config.in | 8 ++++++++
package/picocom/picocom.mk | 18 ++++++++++++++++++
3 files changed, 27 insertions(+), 0 deletions(-)
diff --git a/package/Config.in b/package/Config.in
index 8b9c816..7dfcc55 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -193,6 +193,7 @@ source "package/ntfs-3g/Config.in"
source "package/ntfsprogs/Config.in"
source "package/openocd/Config.in"
source "package/pciutils/Config.in"
+source "package/picocom/Config.in"
source "package/rng-tools/Config.in"
source "package/sdparm/Config.in"
source "package/setserial/Config.in"
diff --git a/package/picocom/Config.in b/package/picocom/Config.in
new file mode 100644
index 0000000..bd584e2
--- /dev/null
+++ b/package/picocom/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_PICOCOM
+ bool "picocom"
+ help
+ picocom is a minimal dumb-terminal emulation program. It
+ is, in principle, very much like minicom, only it's pico
+ instead of mini!
+
+ http://code.google.com/p/picocom/
diff --git a/package/picocom/picocom.mk b/package/picocom/picocom.mk
new file mode 100644
index 0000000..fa02594
--- /dev/null
+++ b/package/picocom/picocom.mk
@@ -0,0 +1,18 @@
+#############################################################
+#
+# picocom
+#
+#############################################################
+
+PICOCOM_VERSION = 1.6
+PICOCOM_SITE = http://picocom.googlecode.com/files/
+
+define PICOCOM_BUILD_CMDS
+ $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D)
+endef
+
+define PICOCOM_INSTALL_TARGET_CMDS
+ install -D -m 0755 $(@D)/picocom $(TARGET_DIR)/usr/bin/picocom
+endef
+
+$(eval $(call GENTARGETS,package,picocom))
^ permalink raw reply related
* [Buildroot] [PATCH 1/4] Add the picocom package
From: Peter Korsgaard @ 2011-10-05 19:51 UTC (permalink / raw)
To: buildroot
In-Reply-To: <b1bcdc61348588f371f00063e571c4d78cd8f4ae.1317636624.git.maxime.ripard@free-electrons.com>
>>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:
Maxime> From: Maxime Ripard <maxime@free-electrons.com>
Maxime> Signed-off-by: Maxime Ripard <maxime@free-electrons.com>
Maxime> ---
Maxime> +++ b/package/picocom/Config.in
Maxime> @@ -0,0 +1,9 @@
Maxime> +config BR2_PACKAGE_PICOCOM
Maxime> + bool "picocom"
Maxime> + help
Maxime> + picocom is a minimal dumb-terminal emulation program. It
Maxime> + is, in principle, very much like minicom, only it's pico
Maxime> + instead of mini!
Maxime> +
nitpick, help text is indented with <tab><space><space>, not 3 spaces.
Maxime> + http://code.google.com/p/picocom/
Maxime> +
You have an extra blank line here and in the .mk.
Committed with those small fixes, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH 2/4] Add Transmission package
From: Peter Korsgaard @ 2011-10-05 20:01 UTC (permalink / raw)
To: buildroot
In-Reply-To: <d6de2d00988b709d01b373cae8f326f889f67904.1317636624.git.maxime.ripard@free-electrons.com>
>>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:
Maxime> From: Maxime Ripard <maxime@free-electrons.com>
Maxime> Signed-off-by: Maxime Ripard <maxime@free-electrons.com>
Isn't this based on some of the earlier transmission patches? If so, it
would be good to give those authors some credits.
Maxime> +++ b/package/transmission/transmission.mk
Maxime> @@ -0,0 +1,47 @@
Maxime> +#############################################################
Maxime> +#
Maxime> +# transmission
Maxime> +#
Maxime> +#############################################################
Maxime> +TRANSMISSION_VERSION = 2.33
Maxime> +TRANSMISSION_SITE = http://download.transmissionbt.com/files/
Maxime> +TRANSMISSION_SOURCE = transmission-$(TRANSMISSION_VERSION).tar.bz2
Maxime> +TRANSMISSION_DEPENDENCIES = \
Maxime> + host-pkg-config \
Maxime> + libcurl \
Maxime> + libevent \
Maxime> + openssl \
Maxime> + zlib
Maxime> +
Maxime> +TRANSMISSION_CONF_OPT = \
Maxime> + --disable-gtk \
Maxime> + --disable-libnotify \
Maxime> + --disable-nls \
--disable-nls is normally handled automatically (depending on if the
toolchain has locale support or not - See package/Makefile.in).
Maxime> + --enable-lightweight
Maxime> +
Maxime> +define TRANSMISSION_INIT_SCRIPT_INSTALL
Maxime> + $(INSTALL) -m 0755 -D package/transmission/S92transmission \
Maxime> + $(TARGET_DIR)/etc/init.d/S92transmission
The init script seems to have some configuration settings which might
not suit everyone. For those cases we typically only install the file if
it isn't already present in the rootfs skeleton - E.G. something like:
[ -f $(TARGET_DIR)/etc/init.d/S92transmission ] || $(INSTALL) -m 0755 -D ..
Maxime> +ifeq ($(BR2_PACKAGE_TRANSMISSION_REMOTE),y)
Maxime> + TRANSMISSION_CONF_OPT += --enable-remote
Maxime> +else
Maxime> + TRANSMISSION_CONF_OPT += --disable-remote
Maxime> + TRANSMISSION_POST_INSTALL_TARGET_HOOKS += TRANSMISSION_INIT_SCRIPT_INSTALL
I don't really know transmission, but why is the init script only
installed if this option isn't enabled? Transmission is a server
(daemon), right? Presumably we should either have an option to not
install the server (so you can use the -remote program to talk to a
server on another machine). If we do that, the CLI option should depend
on the server one.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [git commit] Add lame support in gstreamer
From: Peter Korsgaard @ 2011-10-05 20:02 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=63b7450072c72a1b2f4af6c893e70dd9658e9fb3
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
Signed-off-by: Maxime Ripard <maxime@free-electrons.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
package/multimedia/gst-plugins-ugly/Config.in | 4 ++++
.../gst-plugins-ugly/gst-plugins-ugly.mk | 7 +++++++
2 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/package/multimedia/gst-plugins-ugly/Config.in b/package/multimedia/gst-plugins-ugly/Config.in
index 94e2b9d..c0046e4 100644
--- a/package/multimedia/gst-plugins-ugly/Config.in
+++ b/package/multimedia/gst-plugins-ugly/Config.in
@@ -38,6 +38,10 @@ config BR2_PACKAGE_GST_PLUGINS_UGLY_PLUGIN_SYNAESTHESIA
comment "plugins with external dependencies (there may be more available)"
+config BR2_PACKAGE_GST_PLUGINS_UGLY_PLUGIN_LAME
+ bool "lame (*.mp3 audio encoder)"
+ select BR2_PACKAGE_LAME
+
config BR2_PACKAGE_GST_PLUGINS_UGLY_PLUGIN_MAD
bool "mad (*.mp3 audio)"
select BR2_PACKAGE_LIBID3TAG
diff --git a/package/multimedia/gst-plugins-ugly/gst-plugins-ugly.mk b/package/multimedia/gst-plugins-ugly/gst-plugins-ugly.mk
index 5237bc9..e46b515 100644
--- a/package/multimedia/gst-plugins-ugly/gst-plugins-ugly.mk
+++ b/package/multimedia/gst-plugins-ugly/gst-plugins-ugly.mk
@@ -60,6 +60,13 @@ else
GST_PLUGINS_UGLY_CONF_OPT += --disable-synaesthesia
endif
+ifeq ($(BR2_PACKAGE_GST_PLUGINS_UGLY_PLUGIN_LAME),y)
+GST_PLUGINS_UGLY_CONF_OPT += --enable-lame
+GST_PLUGINS_UGLY_DEPENDENCIES += lame
+else
+GST_PLUGINS_UGLY_CONF_OPT += --disable-lame
+endif
+
ifeq ($(BR2_PACKAGE_GST_PLUGINS_UGLY_PLUGIN_MAD),y)
GST_PLUGINS_UGLY_CONF_OPT += --enable-mad
GST_PLUGINS_UGLY_DEPENDENCIES += libid3tag libmad
^ permalink raw reply related
* [Buildroot] [PATCH 3/4] Add lame support in gstreamer
From: Peter Korsgaard @ 2011-10-05 20:02 UTC (permalink / raw)
To: buildroot
In-Reply-To: <e2016538064470f088a919a8dd8d30925317c24c.1317636624.git.maxime.ripard@free-electrons.com>
>>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:
Maxime> From: Maxime Ripard <maxime@free-electrons.com>
Maxime> Signed-off-by: Maxime Ripard <maxime@free-electrons.com>
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH 4/4] Add the libplayer package
From: Peter Korsgaard @ 2011-10-05 20:16 UTC (permalink / raw)
To: buildroot
In-Reply-To: <d0ce7b4624a0a4fc886c8fcbc9c6730e7a931981.1317636624.git.maxime.ripard@free-electrons.com>
>>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:
Hi,
Maxime> From: Maxime Ripard <maxime@free-electrons.com>
Maxime> +config BR2_PACKAGE_LIBPLAYER
Maxime> + depends on BR2_LARGEFILE
Maxime> + bool "libplayer"
Maxime> + help
Maxime> + libplayer provides a generic A/V API that relies on various multimedia
Maxime> + player for Linux systems. It currently supports MPlayer, xine VLC and
Maxime> + GStreamer only
Maxime> +
Maxime> + http://libplayer.geexbox.org/
Maxime> +
Maxime> +comment "Libplayer requires a toolchain with LARGEFILE support"
Maxime> + depends on !BR2_LARGEFILE
Maxime> +
This should go under the 'if BR2_PACKAGE_LIBPLAYER' conditional.
Maxime> +if BR2_PACKAGE_LIBPLAYER
Maxime> +config BR2_PACKAGE_LIBPLAYER_MPLAYER
Maxime> + depends on BR2_PACKAGE_MPLAYER
Maxime> + bool "Libplayer mplayer backend"
Maxime> +
Maxime> +config BR2_PACKAGE_LIBPLAYER_GSTREAMER
Maxime> + depends on BR2_PACKAGE_GSTREAMER
Maxime> + bool "Libplayer GStreamer backend"
Maxime> +
Maxime> +config BR2_PACKAGE_LIBPLAYER_PYTHON
Maxime> + depends on BR2_PACKAGE_PYTHON
Maxime> + bool "Libplayer python bindings"
Maxime> +endif
Maxime> +
Empty line.
Does libplayer do anything sensible with both backends disabled or do we
need to depend on BR2_PACKAGE_MPLAYER || BR2_PACKAGE_GSTREAMER?
Does it need any specific gstreamer plugins (like playbin2)?
Maxime> diff --git a/package/multimedia/libplayer/libplayer.mk b/package/multimedia/libplayer/libplayer.mk
Maxime> new file mode 100644
Maxime> index 0000000..1552431
Maxime> --- /dev/null
Maxime> +++ b/package/multimedia/libplayer/libplayer.mk
Maxime> @@ -0,0 +1,44 @@
Maxime> +#############################################################
Maxime> +#
Maxime> +# libplayer
Maxime> +#
Maxime> +#############################################################
Maxime> +LIBPLAYER_VERSION = 2.0.1
Maxime> +LIBPLAYER_SITE = http://libplayer.geexbox.org/releases/
Maxime> +LIBPLAYER_SOURCE = libplayer-$(LIBPLAYER_VERSION).tar.bz2
Maxime> +
Maxime> +# When passing the standard buildroot configure arguments, the configure script
Maxime> +# breaks on --target and --host options. Thus we need to define a configure cmd
Maxime> +# ourselves.
Maxime> +define LIBPLAYER_CONFIGURE_CMDS
Maxime> + (cd $(@D) && rm -rf config.cache && \
Maxime> + $(TARGET_CONFIGURE_OPTS) \
Maxime> + $(TARGET_CONFIGURE_ARGS) \
Maxime> + ./configure \
Maxime> + --prefix=/usr \
Maxime> + --cross-compile \
Maxime> + $(SHARED_STATIC_LIBS_OPTS) \
Maxime> + $(QUIET) $(LIBPLAYER_CONF_OPT) \
Is that $(QUIET) correct?
./configure -q ~/source/buildroot/test/build/libplayer-2.0.1
Unknown option "-q".
See ./configure --help for available options.
It doesn't look like it.
I did a test build, but configure dies right away:
Checking for compiler available...
Error, can't find pthread_mutex_init !
See file "config.log" produced by configure for more details.
config.log contains:
/home/peko/source/buildroot/test/host/usr/bin/arm-unknown-linux-uclibcgnueabi-gc
c -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_REENTRANT -pipe -Os -W -Wall -E
-o /tmp/libplayer-6047-14742-29987.o /tmp/libplayer-6150-14742-15899.c
In file included from /home/peko/source/buildroot/test/host/usr/arm-unknown-linu
x-uclibcgnueabi/sysroot/usr/include/pthread.h:23,
from /tmp/libplayer-6150-14742-15899.c:1:
/home/peko/source/buildroot/test/host/usr/arm-unknown-linux-uclibcgnueabi/sysroot/usr/include/features.h:219:5: error: #error It appears you have defined _FILE_OFFSET_BITS=64. Unfortunately, uClibc was built without large file support enabled.
So it seems you need to depend on BR2_LARGEFILE.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH] gmp: work around ARM thumb build breakage
From: Gustavo Zacarias @ 2011-10-05 20:23 UTC (permalink / raw)
To: buildroot
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/gmp/gmp.mk | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/package/gmp/gmp.mk b/package/gmp/gmp.mk
index 5eb6ecc..2222465 100644
--- a/package/gmp/gmp.mk
+++ b/package/gmp/gmp.mk
@@ -9,5 +9,10 @@ GMP_SITE = $(BR2_GNU_MIRROR)/gmp
GMP_SOURCE = gmp-$(GMP_VERSION).tar.bz2
GMP_INSTALL_STAGING = YES
+# Bad ARM assembly breaks on pure thumb
+ifeq ($(ARCH),arm)
+GMP_MAKE_OPT += CFLAGS="$(TARGET_CFLAGS) -marm"
+endif
+
$(eval $(call AUTOTARGETS))
$(eval $(call AUTOTARGETS,host))
--
1.7.3.4
^ permalink raw reply related
* [Buildroot] [PATCH] pkg-stats: use correct variable names for convert_to_*autotools
From: H Hartley Sweeten @ 2011-10-05 20:34 UTC (permalink / raw)
To: buildroot
The variable convert_to_autotools is not used in the script. The correct
variables are convert_to_target_autotools and convert_to_host_autotools.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
diff --git a/support/scripts/pkg-stats b/support/scripts/pkg-stats
index 3e89bd1..30a21f2 100755
--- a/support/scripts/pkg-stats
+++ b/support/scripts/pkg-stats
@@ -71,7 +71,8 @@ tr.correct td {
convert_to_generic_target=0
convert_to_generic_host=0
-convert_to_autotools=0
+convert_to_target_autotools=0
+convert_to_host_autotools=0
total_patch_count=0
cnt=0
for i in $(find package/ -name '*.mk') ; do
^ permalink raw reply related
* [Buildroot] Question: := vs = in package *.mk files
From: Peter Korsgaard @ 2011-10-05 20:40 UTC (permalink / raw)
To: buildroot
In-Reply-To: <20111005090428.34c8f565@skate>
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
Hi,
Thomas> In package definition, Peter has said that he preferred the usage of
Thomas> '=' except when it would create a noticeable performance problem. So
Thomas> for all package definitions such as _SITE, _VERSION, _SOURCE,
Thomas> _AUTORECONF, _INSTALL_STAGING, etc. the '=' sign is the one to choose.
Indeed. And the reason for this is that it fixes ordering issues because
of the late bindings, so you don't need to be careful about using
"outside" variables within a package .mk
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [git commit] openocd: remove fixed ldflags breaking build
From: Peter Korsgaard @ 2011-10-05 20:50 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=a5094a056ad99da7763d93a97b8e61630f60d170
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
The openocd configure script would forcibly add -L/usr/lib to LDFLAGS,
causing the cross compiler to use host libraries, breaking the build.
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
.../openocd/openocd-0005-dont-force-ldflags.patch | 37 ++++++++++++++++++++
1 files changed, 37 insertions(+), 0 deletions(-)
diff --git a/package/openocd/openocd-0005-dont-force-ldflags.patch b/package/openocd/openocd-0005-dont-force-ldflags.patch
new file mode 100644
index 0000000..1880e00
--- /dev/null
+++ b/package/openocd/openocd-0005-dont-force-ldflags.patch
@@ -0,0 +1,37 @@
+[PATCH] don't force library search path / rpath settings
+
+openocd adds -L$exec_prefix/lib -Wl,rpath,$exec_prefix/lib to the compile
+LDFLAGS if it isn't installed into /usr/local, which breaks cross compilation
+as the compiler ends up using host libraries.
+---
+ configure.in | 17 -----------------
+ 1 file changed, 17 deletions(-)
+
+Index: openocd-0.5.0/configure.in
+===================================================================
+--- openocd-0.5.0.orig/configure.in
++++ openocd-0.5.0/configure.in
+@@ -174,23 +174,6 @@
+ # Let make expand exec_prefix.
+ test x"$OCDxprefix" = xNONE && OCDxprefix="$OCDprefix"
+
+-# what matters is the "exec-prefix"
+-if test "$OCDxprefix" != "$ac_default_prefix"
+-then
+- # We are installing in a non-standard place
+- # Nonstandard --prefix and/or --exec-prefix
+- # We have an override of some sort.
+- # use build specific install library dir
+-
+- LDFLAGS="$LDFLAGS -L$OCDxprefix/lib"
+- # RPATH becomes an issue on Linux only
+- if test $host_os = linux-gnu || test $host_os = linux ; then
+- LDFLAGS="$LDFLAGS -Wl,-rpath,$OCDxprefix/lib"
+- fi
+- # The "INCDIR" is also usable
+- CFLAGS="$CFLAGS -I$includedir"
+-fi
+-
+ AC_ARG_WITH(ftd2xx,
+ AS_HELP_STRING([--with-ftd2xx=<PATH>],[This option has been removed.]),
+ [
^ permalink raw reply related
* [Buildroot] [git commit] pkg-stats: use correct variable names for convert_to_*autotools
From: Peter Korsgaard @ 2011-10-05 21:07 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=e684cecbfa9c01ed8b7494e4b294c8b6b303747e
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
The variable convert_to_autotools is not used in the script. The correct
variables are convert_to_target_autotools and convert_to_host_autotools.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
support/scripts/pkg-stats | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/support/scripts/pkg-stats b/support/scripts/pkg-stats
index 3e89bd1..30a21f2 100755
--- a/support/scripts/pkg-stats
+++ b/support/scripts/pkg-stats
@@ -71,7 +71,8 @@ tr.correct td {
convert_to_generic_target=0
convert_to_generic_host=0
-convert_to_autotools=0
+convert_to_target_autotools=0
+convert_to_host_autotools=0
total_patch_count=0
cnt=0
for i in $(find package/ -name '*.mk') ; do
^ permalink raw reply related
* [Buildroot] [PATCH] pkg-stats: use correct variable names for convert_to_*autotools
From: Peter Korsgaard @ 2011-10-05 21:07 UTC (permalink / raw)
To: buildroot
In-Reply-To: <ADE657CA350FB648AAC2C43247A983F001F39DCFE980@AUSP01VMBX24.collaborationhost.net>
>>>>> "H" == H Hartley Sweeten <hartleys@visionengravers.com> writes:
H> The variable convert_to_autotools is not used in the script. The correct
H> variables are convert_to_target_autotools and convert_to_host_autotools.
H> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH] less: bump version to 444
From: H Hartley Sweeten @ 2011-10-05 22:28 UTC (permalink / raw)
To: buildroot
less-444 was released on 9 Jun 2011.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
diff --git a/package/less/less.mk b/package/less/less.mk
index d20c232..5f74f5c 100644
--- a/package/less/less.mk
+++ b/package/less/less.mk
@@ -4,7 +4,7 @@
#
#############################################################
-LESS_VERSION = 443
+LESS_VERSION = 444
LESS_SITE = http://www.greenwoodsoftware.com/less
LESS_DEPENDENCIES = ncurses
^ permalink raw reply related
* [Buildroot] [PATCH] pkg-stats: update list of .mk to ignore
From: H Hartley Sweeten @ 2011-10-05 23:57 UTC (permalink / raw)
To: buildroot
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
---
diff --git a/support/scripts/pkg-stats b/support/scripts/pkg-stats
index 30a21f2..0a030c1 100755
--- a/support/scripts/pkg-stats
+++ b/support/scripts/pkg-stats
@@ -80,6 +80,7 @@ for i in $(find package/ -name '*.mk') ; do
if test \
$i = "package/java/java.mk" -o \
$i = "package/games/games.mk" -o \
+ $i = "package/gtk2-themes/gtk2-themes.mk" -o \
$i = "package/multimedia/multimedia.mk" -o \
$i = "package/customize/customize.mk" -o \
$i = "package/matchbox/matchbox.mk" -o \
^ permalink raw reply related
* [Buildroot] [git commit] less: bump version to 444
From: Peter Korsgaard @ 2011-10-06 5:20 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=b426e34dbc6908774e311023e6b2e0e4c6817f84
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
less-444 was released on 9 Jun 2011.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
package/less/less.mk | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/package/less/less.mk b/package/less/less.mk
index d20c232..5f74f5c 100644
--- a/package/less/less.mk
+++ b/package/less/less.mk
@@ -4,7 +4,7 @@
#
#############################################################
-LESS_VERSION = 443
+LESS_VERSION = 444
LESS_SITE = http://www.greenwoodsoftware.com/less
LESS_DEPENDENCIES = ncurses
^ permalink raw reply related
* [Buildroot] [PATCH] less: bump version to 444
From: Peter Korsgaard @ 2011-10-06 5:20 UTC (permalink / raw)
To: buildroot
In-Reply-To: <ADE657CA350FB648AAC2C43247A983F001F39DCFE9E4@AUSP01VMBX24.collaborationhost.net>
>>>>> "H" == H Hartley Sweeten <hartleys@visionengravers.com> writes:
H> less-444 was released on 9 Jun 2011.
H> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [git commit] pkg-stats: update list of .mk to ignore
From: Peter Korsgaard @ 2011-10-06 5:20 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=b706e4d0e46a047d84554fe03bf4c2f2c5d1cf66
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
support/scripts/pkg-stats | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/support/scripts/pkg-stats b/support/scripts/pkg-stats
index 30a21f2..0a030c1 100755
--- a/support/scripts/pkg-stats
+++ b/support/scripts/pkg-stats
@@ -80,6 +80,7 @@ for i in $(find package/ -name '*.mk') ; do
if test \
$i = "package/java/java.mk" -o \
$i = "package/games/games.mk" -o \
+ $i = "package/gtk2-themes/gtk2-themes.mk" -o \
$i = "package/multimedia/multimedia.mk" -o \
$i = "package/customize/customize.mk" -o \
$i = "package/matchbox/matchbox.mk" -o \
^ permalink raw reply related
* [Buildroot] [PATCH] pkg-stats: update list of .mk to ignore
From: Peter Korsgaard @ 2011-10-06 5:21 UTC (permalink / raw)
To: buildroot
In-Reply-To: <ADE657CA350FB648AAC2C43247A983F001F39DCFEA1B@AUSP01VMBX24.collaborationhost.net>
>>>>> "H" == H Hartley Sweeten <hartleys@visionengravers.com> writes:
H> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH 2/2] add libfcgi to buildroot
From: Rico Bachmann @ 2011-10-06 6:53 UTC (permalink / raw)
To: buildroot
In-Reply-To: <201110012259.10267.arnout@mind.be>
> -----Urspr?ngliche Nachricht-----
> Von: Arnout Vandecappelle [mailto:arnout at mind.be]
> Gesendet: Samstag, 1. Oktober 2011 22:59
> An: buildroot at busybox.net
> Cc: Thomas Petazzoni; Rico Bachmann
> Betreff: Re: [Buildroot] [PATCH 2/2] add libfcgi to buildroot
>
> [I put Rico back in CC]
> On Thursday 29 September 2011 11:58:49, Thomas Petazzoni wrote:
> > Le Thu, 29 Sep 2011 10:45:17 +0200,
> >
> > Rico Bachmann <bachmann@tofwerk.com> a ?crit :
> > > added the patches, still not working
> > > could be a problem with the filepath in the patch?
> [snip]
> > Also:
> >
> > * Your two patches link_math_cgifcgi.patch and
> link_math_example.patch
> > should be just one patch.
>
> You typically generate the patch like this:
>
> 1. Make a copy of the unmodified source tree:
> cp -a output/build/fcgi-2.4.0 output/build/fcgi-2.4.0.orig
>
> 2. Hack away in output/build/fcgi-2.4.0 until it works.
>
> 3. Create the patch relative to the build directory:
> cd output/build
> diff -rup fcgi-2.4.0.orig fcgi-2.4.0 > \
> ../../package/fcgi/fcgi-2.4.0-add-lm.patch
>
> 4. Verify that it works by doing a clean build:
> make fcgi-dirclean; make fcgi
>
> 5.
> > * This patch should have a description and a Signed-off-by line.
>
> Edit package/fcgi/fcgi-2.4.0-add-lm.patch to add these things.
>
> The description goes in the beginning of the patch, before the first
> 'diff',
> and explains why the patch is necessary.
>
> The Signed-off-by line comes at the end of this patch comment. It
> shows that
> you are the author of the patch and that you agree to distribute it
> under GPL.
> See http://kerneltrap.org/files/Jeremy/DCO.txt
>
> Terminate the description with a line containing three dashes: ---
>
> And look at some of the existing patches for inspiration :-)
>
> > * Patching the Makefiles in an autotools-based package is really
> > wrong, it's the Makefile.am that should be patched. I think your
> > LDFLAGS = -lm is more a workaround than a correct fix.
>
> The Makefile will be overwritten next time you configure the package.
> You'll
> notice that in step 4 :-). Instead, you have to modify Makefile.am.
> Autotools uses Makefile.am to generate Makefile.in, and configure uses
> Makefile.in to generate Makefile. However, buildroot normally doesn't
> run
> autotools but uses the distributed Makefile.in instead. So, you should
> patch
> both Makefile.am and Makefile.in.
>
>
> 6. Send your patch to the upstream project. And be prepared to get
> completely different feedback from them than what you got here :-)
>
>
> Good luck!
>
> 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
Thanks for this nice explanation.
I will fix the package in the Makefile.in and in the Makefile.am
And add i nice patch with all necessary comments and signed-off-by line etc.
^ permalink raw reply
* [Buildroot] Bug in pkg-stats?
From: Thomas Petazzoni @ 2011-10-06 7:52 UTC (permalink / raw)
To: buildroot
In-Reply-To: <87obxvntns.fsf@macbook.be.48ers.dk>
Le Wed, 05 Oct 2011 21:08:07 +0200,
Peter Korsgaard <jacmet@uclibc.org> a ?crit :
> Well, I think the info in useful, but as buildroot already knows the
> version (except for unconfigured packages), wouldn't it make more
> sense to add a <pkg>-version target to GENTARGETS and use that
> instead?
I agree here. Grepping/seding/awking into the .mk file sounds horrible
and does not work for package that do:
pkg_VERSION = $(pkg_VERSION_MAJOR).5
So, yes, if you want to do this, please add a <pkg>-show-version in
GENTARGETS, just as we have <pkg>-show-depends. Of course, this will
slow down quite a bit the pkg-stats script, but we don't really need
this script to be fast.
Regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply
* [Buildroot] [PATCH] package: add the duma memory debugging library
From: Thomas Petazzoni @ 2011-10-06 8:16 UTC (permalink / raw)
To: buildroot
In-Reply-To: <201110051935.41682.yann.morin.1998@anciens.enib.fr>
Le Wed, 5 Oct 2011 19:35:41 +0200,
"Yann E. MORIN" <yann.morin.1998@anciens.enib.fr> a ?crit :
> > + $(MAKE1) CC=$(TARGET_CC) CXX=$(TARGET_CXX) AR=$(TARGET_AR) \
> > + RANLIB=$(TARGET_RANLIB) DUMA_OPTIONS=$(strip $(DUMA_OPTIONS)) \
> > + $(DUMA_CPP) -C $(@D)
>
> You forgot to pass HOSTCC, introduced in duma-2_5_15-cross-compile.patch
Probably something like:
$(MAKE1) $(TARGET_CONFIGURE_OPTS) DUMA_OPTIONS=... -C $(@D)
would be even better. But then, the duma-2_5_15-cross-compile.patch
would have to use CC_FOR_BUILD instead of HOSTCC to find the location
of the native host compiler.
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply
* [Buildroot] [PATCH 4/4] Add the libplayer package
From: Thomas Petazzoni @ 2011-10-06 8:19 UTC (permalink / raw)
To: buildroot
In-Reply-To: <871uurnqhp.fsf@macbook.be.48ers.dk>
Le Wed, 05 Oct 2011 22:16:34 +0200,
Peter Korsgaard <jacmet@uclibc.org> a ?crit :
> >>>>> "Maxime" == Maxime Ripard <maxime.ripard@free-electrons.com> writes:
>
> Hi,
>
> Maxime> From: Maxime Ripard <maxime@free-electrons.com>
> Maxime> +config BR2_PACKAGE_LIBPLAYER
> Maxime> + depends on BR2_LARGEFILE
> Maxime> + bool "libplayer"
> Maxime> + help
> Maxime> + libplayer provides a generic A/V API that relies on various multimedia
> Maxime> + player for Linux systems. It currently supports MPlayer, xine VLC and
> Maxime> + GStreamer only
> Maxime> +
> Maxime> + http://libplayer.geexbox.org/
> Maxime> +
> Maxime> +comment "Libplayer requires a toolchain with LARGEFILE support"
> Maxime> + depends on !BR2_LARGEFILE
> Maxime> +
>
> This should go under the 'if BR2_PACKAGE_LIBPLAYER' conditional.
Hu ? We want this comment to be shown when libplayer is *not* enabled,
to let the user know that if (s)he doesn't see libplayer, it's because
largefile is not enabled. So definitely, we don't want this comment to
depend on BR2_PACKAGE_LIBPLAYER. Or am I missing something here ?
> Maxime> +if BR2_PACKAGE_LIBPLAYER
> Maxime> +config BR2_PACKAGE_LIBPLAYER_MPLAYER
> Maxime> + depends on BR2_PACKAGE_MPLAYER
> Maxime> + bool "Libplayer mplayer backend"
> Maxime> +
> Maxime> +config BR2_PACKAGE_LIBPLAYER_GSTREAMER
> Maxime> + depends on BR2_PACKAGE_GSTREAMER
> Maxime> + bool "Libplayer GStreamer backend"
> Maxime> +
> Maxime> +config BR2_PACKAGE_LIBPLAYER_PYTHON
> Maxime> + depends on BR2_PACKAGE_PYTHON
> Maxime> + bool "Libplayer python bindings"
> Maxime> +endif
> Maxime> +
>
> Empty line.
>
> Does libplayer do anything sensible with both backends disabled or do we
> need to depend on BR2_PACKAGE_MPLAYER || BR2_PACKAGE_GSTREAMER?
I did ask the same question to Maxime, and libplayer builds fine with
neither backends enabled. It probably doesn't do anything useful, but I
would say that it's the user responsibility here.
> So it seems you need to depend on BR2_LARGEFILE.
Which it does:
> Maxime> +config BR2_PACKAGE_LIBPLAYER
> Maxime> + depends on BR2_LARGEFILE
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply
* [Buildroot] [PATCH 4/4] Add the libplayer package
From: Peter Korsgaard @ 2011-10-06 8:47 UTC (permalink / raw)
To: buildroot
In-Reply-To: <20111006101928.294b6580@skate>
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
Hi,
Maxime> +comment "Libplayer requires a toolchain with LARGEFILE support"
Maxime> + depends on !BR2_LARGEFILE
Maxime> +
>>
>> This should go under the 'if BR2_PACKAGE_LIBPLAYER' conditional.
Thomas> Hu ? We want this comment to be shown when libplayer is *not* enabled,
Thomas> to let the user know that if (s)he doesn't see libplayer, it's because
Thomas> largefile is not enabled. So definitely, we don't want this comment to
Thomas> depend on BR2_PACKAGE_LIBPLAYER. Or am I missing something here ?
No, just me not thinking. Nothing to see here, move along .. ;)
>> Does libplayer do anything sensible with both backends disabled or do we
>> need to depend on BR2_PACKAGE_MPLAYER || BR2_PACKAGE_GSTREAMER?
Thomas> I did ask the same question to Maxime, and libplayer builds fine with
Thomas> neither backends enabled. It probably doesn't do anything useful, but I
Thomas> would say that it's the user responsibility here.
Ok.
>> So it seems you need to depend on BR2_LARGEFILE.
Thomas> Which it does:
Maxime> +config BR2_PACKAGE_LIBPLAYER
Maxime> + depends on BR2_LARGEFILE
Argh, you're right. I need some more sleep it seems (or atleast stop
trying to do 5 things at once). When testing, I didn't run menuconfig
but just 'make libplayer' and my toolchain didn't have largefile
support.
Sorry about the noise, will commit now - Thanks both.
--
Bye, Peter Korsgaard
^ permalink raw reply
* [Buildroot] [PATCH] package: add the duma memory debugging library
From: Baruch Siach @ 2011-10-06 8:49 UTC (permalink / raw)
To: buildroot
In-Reply-To: <20111006101634.601b9e9f@skate>
Hi Thomas, Yann,
On Thu, Oct 06, 2011 at 10:16:34AM +0200, Thomas Petazzoni wrote:
> Le Wed, 5 Oct 2011 19:35:41 +0200,
> "Yann E. MORIN" <yann.morin.1998@anciens.enib.fr> a ?crit :
>
> > > + $(MAKE1) CC=$(TARGET_CC) CXX=$(TARGET_CXX) AR=$(TARGET_AR) \
> > > + RANLIB=$(TARGET_RANLIB) DUMA_OPTIONS=$(strip $(DUMA_OPTIONS)) \
> > > + $(DUMA_CPP) -C $(@D)
> >
> > You forgot to pass HOSTCC, introduced in duma-2_5_15-cross-compile.patch
Well, HOSTCC is set in the top Makefile.
> Probably something like:
>
> $(MAKE1) $(TARGET_CONFIGURE_OPTS) DUMA_OPTIONS=... -C $(@D)
>
> would be even better. But then, the duma-2_5_15-cross-compile.patch
> would have to use CC_FOR_BUILD instead of HOSTCC to find the location
> of the native host compiler.
Looks better. I'll update the patch. Thanks.
baruch
--
~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox