Re: [ptxdist] pull-request; package additions, package updates

2010-06-25 Thread Marc Kleine-Budde
Carsten Schlote wrote:
 Hi,
 
 these patches provide new or updated packages. 
 
 As usual the package patches provide fixes for cross-compilation and
 compiler issues.

I've revied about the first half of the patches...

cheers, Marc

 
 Carsten
 ---
 The following changes since commit
 dee596b721d29d45d721866b38cba693a72eab3d:
   Marc Kleine-Budde (1):
 [lmbench] cleanups: remove unneeded prepare stage
 
 are available in the git repository at:
 
   git://git.vahanus.net/csc/ptxdist-kp.git/ for-upstream
 
 Carsten Schlote (22):
   [gdb] Updated default value to 7.1

what about the patch from 7.0.1?

   [lshw] Fixed GCC 4.5.0 warning which causes build break (-Werror)

Please add a Subject: and From: line to the patch

   [6tunnel] Added package for 6tunnel, version 0.11rc2

Please add a Subject: and From: line to the patches.
The strip can stay in, we've a dummy-strip now.

Regading the get stage: what about downloading the tarball from here:
http://toxygen.net/6tunnel/6tunnel-0.11rc2.tar.gz, or have a look at
rules/miidiag.make how to handle debian's/ubuntu's orig filenames.

prepare: please don't call autogen.sh in the prepare stage, create a
link to ../autogen.sh in the patches dir or supply a private autogen.sh
there.

   [statserial] Added statserial 1.1 with Ubuntu patches

Please add a Subject: and From: line to the patch

please modify the CFLAGS line to:

CFLAGS+= -Wall -O3 -fomit-frame-pointer $(CPPFLAGS)

it won't overwrite external provided CFLAGS

Please remove the LDFALGS = -s this is stripping and we don't want
this. I think our fake strip tool won't help here.

Please unzip that file: .../statserial-1.1.orig/statserial_1.1-22.diff.gz

Comment for the debian-orig-filename applies here, too.

Please use - as source in @$(call install_copy)

   [zsync] Added version 0.6.1 as package to ptxdist

looks good, cherry-picked :)

   [midnight commander] Updated version to 4.7.0, adding more install
 options

can you please use the proposed debian filename handling

   [udev] Updated to udev version 151
   [ptxdist][v2] Updated ptxdist usage text for bash, make, removed
 autobuild
   [vpnc][v4] Added vpnc 0.5.3 package, a VPN client for Cisco
 concentrators
   [miredo][v5] Added miredo 1.2.2, a teredo IPV6 client
   [ppp] Added plog and pppoe-discovery options and install
   [ppp] Updated generic/{pon|poff} to maintainer version
   [mtd-utils] Fixes LZO Compression and crashes
   [nfsutils] Added dependancies when compiling NFSv4 support
   [mtd-utils] Added config options and install
   [mtd-utils] Workaround the -Werror problems
   [dbench] Added dependency for libattr.so
   [sysvinit] Added comments for busybox builtins used.
   [acpid] Removed -Werror to allow compilation with GCC 4.4.1
   [dhcp] Fixed GCC 4.4.1 compile error
   [dhcp] Fixed problem with -Werror and GCC 4.5.0-snapshot

please squash these patches

   [perl] Added version 5.10.1
 
  bin/ptxdist|   19 +-
  generic/etc/init.d/miredo-server   |   89 +
  generic/etc/miredo/client-hook |   70 +
  generic/etc/miredo/miredo-server.conf  |   22 +
  generic/etc/miredo/miredo.conf |   35 +
  generic/etc/vpnc/default.conf  |6 +
  generic/etc/vpnc/vpnc-script   |  529 ++
  generic/usr/bin/poff   |  102 +-
  generic/usr/bin/pon|   38 +-
  patches/6tunnel-0.11rc2/crosscompile-fixes.patch   |   55 +
  patches/6tunnel-0.11rc2/fix-compile-warnings.patch |   22 +
  patches/6tunnel-0.11rc2/series |2 +
  .../fix-gcc-4.4.1-error-warnings.patch |   25 +
  patches/acpid-1.0.10/series|1 +
  patches/{udev-150 = dhcp-4.1.1}/autogen.sh|0
  .../dhcp-4.1.1/gcc4.4-fixup-error-warning.patch|   39 +
  patches/dhcp-4.1.1/series  |1 +
  patches/lshw-B.02.14/0003-fix-gcc450-issues.patch  |   22 +
  patches/lshw-B.02.14/series|1 +
  patches/{udev-150 = mc-4.7.0}/autogen.sh  |0
  .../debian/02_use_correct_smb_conf_path.patch  |   19 +
  patches/mc-4.7.0/debian/03_use_awk.patch   |   19 +
  .../11_unzip_needs_regex_update_lp397505.patch |   30 +
  patches/mc-4.7.0/mc-install-fixes.patch|   57 +
  patches/mc-4.7.0/series|4 +
  patches/{udev-150 = miredo-1.2.2}/autogen.sh  |0
  patches/miredo-1.2.2/series|3 +
  patches/mtd-utils-1.3.1/lzo-compr-fixes.patch  |   59 +
  patches/mtd-utils-1.3.1/series |2 +
  patches/perl-5.10.0/100-static_uudmap.patch|   50 +
  patches/perl-5.10.0/perl-cross-fixes.patch | 
 +++
  

[ptxdist] [PATCH] The '-q' option breaks 'ptxdist -q allyesconfig collection'

2010-06-25 Thread Remy Bohmer
The result is an almost empty collection config file with no options selected,
like this:
===
PTXCONF__collectionconfig_MAGIC__=y
PTXCONF_COLLECTION_ALL=y
===

Signed-off-by: Remy Bohmer li...@bohmer.net
---
 bin/ptxdist |4 
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/bin/ptxdist b/bin/ptxdist
index 466c712..a1e4ff9 100755
--- a/bin/ptxdist
+++ b/bin/ptxdist
@@ -1399,6 +1399,10 @@ do_config()
collection)
local file_dotconfig=${3}
 
+   # ptxdist -q allyesconfig collection gives an empty collection
+   # ignore the '-q' option to get a valid collection.
+   unset PTXDIST_QUIET
+
if [ -n ${file_dotconfig} ]; then
file_dotconfig=$(ptxd_abspath ${file_dotconfig}) || 
exit 1
else
-- 
1.7.0.4


-- 
ptxdist mailing list
ptxdist@pengutronix.de