[oe] [PATCH 1/4] coreutils: add version 7.2
Otavio Salvador
otavio at ossystems.com.br
Thu Apr 30 11:28:04 UTC 2009
Acked-by: Otavio Salvador <otavio at ossystems.com.br>
On Thu, Apr 30, 2009 at 8:22 AM, Roman I Khimov <khimov at altell.ru> wrote:
> ---
> conf/checksums.ini | 4 +
> .../coreutils/coreutils-7.2/automake-version.patch | 13 +++
> .../coreutils-7.2-fix-sort-segfault.patch | 103 ++++++++++++++++++++
> recipes/coreutils/coreutils-7.2/man.patch | 47 +++++++++
> recipes/coreutils/coreutils-7.2/onceonly.m4 | 63 ++++++++++++
> recipes/coreutils/coreutils-native_7.2.bb | 5 +
> recipes/coreutils/coreutils_7.2.bb | 89 +++++++++++++++++
> 7 files changed, 324 insertions(+), 0 deletions(-)
> create mode 100644 recipes/coreutils/coreutils-7.2/automake-version.patch
> create mode 100644 recipes/coreutils/coreutils-7.2/coreutils-7.2-fix-sort-segfault.patch
> create mode 100644 recipes/coreutils/coreutils-7.2/man.patch
> create mode 100644 recipes/coreutils/coreutils-7.2/onceonly.m4
> create mode 100644 recipes/coreutils/coreutils-native_7.2.bb
> create mode 100644 recipes/coreutils/coreutils_7.2.bb
>
> diff --git a/conf/checksums.ini b/conf/checksums.ini
> index ffc84c6..9a2c473 100644
> --- a/conf/checksums.ini
> +++ b/conf/checksums.ini
> @@ -3990,6 +3990,10 @@ sha256=efa27532ec6dc12a21f703ad4a0f612e613e9cc2575147685db81cc701952ac9
> md5=cbb2b3d1718ee1237b808e00b5c11b1e
> sha256=813cb19fa19a885f342664109c7c5810f0081b624ff317bba8d1b6ccd83c2a05
>
> +[ftp://ftp.gnu.org/gnu/coreutils/coreutils-7.2.tar.gz]
> +md5=427c2914d3eab956f317c9ec6a45e62a
> +sha256=dd77bfec92e5a3ad48abd8a5bda3f8d40149c4e24744e4173abc3cc6a731fdb2
> +
> [http://www.rpsys.net/openzaurus/patches/archive/corgi_rearrange_lcd-r0.patch]
> md5=c1bec44c92f5da5abad97137c73b1365
> sha256=d4dc8f0583e616e74ea41b103d5f6a29b6d079ce37eaf9f362825514c7082490
> diff --git a/recipes/coreutils/coreutils-7.2/automake-version.patch b/recipes/coreutils/coreutils-7.2/automake-version.patch
> new file mode 100644
> index 0000000..497ef15
> --- /dev/null
> +++ b/recipes/coreutils/coreutils-7.2/automake-version.patch
> @@ -0,0 +1,13 @@
> +Index: coreutils-7.1/configure.ac
> +===================================================================
> +--- coreutils-7.1.orig/configure.ac
> ++++ coreutils-7.1/configure.ac
> +@@ -32,7 +32,7 @@
> + AC_CONFIG_AUX_DIR([build-aux])
> + AC_CONFIG_HEADERS([lib/config.h:lib/config.hin])
> +
> +-AM_INIT_AUTOMAKE([1.10a dist-xz])
> ++AM_INIT_AUTOMAKE([1.10.2])
> +
> + AC_PROG_CC_STDC
> + AM_PROG_CC_C_O
> diff --git a/recipes/coreutils/coreutils-7.2/coreutils-7.2-fix-sort-segfault.patch b/recipes/coreutils/coreutils-7.2/coreutils-7.2-fix-sort-segfault.patch
> new file mode 100644
> index 0000000..e9d7b4e
> --- /dev/null
> +++ b/recipes/coreutils/coreutils-7.2/coreutils-7.2-fix-sort-segfault.patch
> @@ -0,0 +1,103 @@
> +OE modifications to the original patch:
> + * Removed NEWS and THANKS hunks to apply on vanilla 7.2 coreutils
> +
> +>From 570beb56f58bb087a614af885bec7e9cf6b19423 Mon Sep 17 00:00:00 2001
> +From: Jim Meyering <meyering at redhat.com>
> +Date: Wed, 22 Apr 2009 08:45:27 +0200
> +Subject: [PATCH] sort -m: don't segfault when output file is also an input =
> +file
> +
> +* src/sort.c (avoid_trashing_input): Fix an off-by-one error and
> +guard the use of memmove.
> +* NEWS (Bug fixes): Mention it.
> +* tests/misc/sort: Add tests to exercise the offending code.
> +* THANKS: Update.
> +Reported by Otavio Salvador in http://bugs.debian.org/525048.
> +---
> + NEWS | 6 +++++-
> + THANKS | 1 +
> + src/sort.c | 10 ++++++----
> + tests/misc/sort | 17 ++++++++++++++++-
> + 4 files changed, 28 insertions(+), 6 deletions(-)
> +
> +diff --git a/THANKS b/THANKS
> +index fe523fe..876a6b6 100644
> +--- a/THANKS
> ++++ b/THANKS
> +@@ -435,6 +435,7 @@ Ole Laursen olau at hardworking.dk
> + Oliver Kiddle okiddle at yahoo.co.uk
> + Ørn E. Hansen oehansen at daimi.aau.dk
> + Oskar Liljeblad osk at hem.passagen.se
> ++Otavio Salvador otavio at ossystems.com.br
> + Pádraig Brady P at draigBrady.com
> + Patrick Mauritz oxygene at studentenbude.ath.cx
> + Paul D. Smith psmith at gnu.org
> +diff --git a/src/sort.c b/src/sort.c
> +index 2e6ce87..f48d727 100644
> +--- a/src/sort.c
> ++++ b/src/sort.c
> +@@ -1,5 +1,5 @@
> + /* sort - sort lines of text (with all kinds of options).
> +- Copyright (C) 1988, 1991-2008 Free Software Foundation, Inc.
> ++ Copyright (C) 1988, 1991-2009 Free Software Foundation, Inc.
> +
> + This program is free software: you can redistribute it and/or modify
> + it under the terms of the GNU General Public License as published by
> +@@ -2602,18 +2602,20 @@ avoid_trashing_input (struct sortfile *files, size_t ntemps,
> + pid_t pid;
> + char *temp = create_temp (&tftp, &pid);
> + size_t num_merged = 0;
> +- while (i + num_merged < nfiles)
> ++ do
> + {
> + num_merged += mergefiles (&files[i], 0, nfiles - i, tftp, temp);
> + files[i].name = temp;
> + files[i].pid = pid;
> +
> +- memmove(&files[i], &files[i + num_merged],
> +- num_merged * sizeof *files);
> ++ if (i + num_merged < nfiles)
> ++ memmove(&files[i + 1], &files[i + num_merged],
> ++ num_merged * sizeof *files);
> + ntemps += 1;
> + nfiles -= num_merged - 1;;
> + i += num_merged;
> + }
> ++ while (i < nfiles);
> + }
> + }
> +
> +diff --git a/tests/misc/sort b/tests/misc/sort
> +index 4f377df..a27f47a 100755
> +--- a/tests/misc/sort
> ++++ b/tests/misc/sort
> +@@ -1,6 +1,6 @@
> + #!/usr/bin/perl
> +
> +-# Copyright (C) 2008 Free Software Foundation, Inc.
> ++# Copyright (C) 2008, 2009 Free Software Foundation, Inc.
> +
> + # This program is free software: you can redistribute it and/or modify
> + # it under the terms of the GNU General Public License as published by
> +@@ -339,6 +339,21 @@ my @Tests =
> + ["sort-numeric", '--sort=numeric', {IN=>".01\n0\n"}, {OUT=>"0\n.01\n"}],
> + ["sort-gennum", '--sort=general-numeric',
> + {IN=>"1e2\n2e1\n"}, {OUT=>"2e1\n1e2\n"}],
> ++
> ++# -m with output file also used as an input file
> ++# In coreutils-7.2, this caused a segfault.
> ++# This test looks a little strange. Here's why:
> ++# since we're using "-o f", standard output will be empty, hence OUT=>''
> ++# We still want to ensure that the output file, "f" has expected contents,
> ++# hence the added CMP=> directive.
> ++["output-is-input", '-m -o f', {IN=> {f=> "a\n"}}, {OUT=>''},
> ++ {CMP=> ["a\n", {'f'=> undef}]} ],
> ++["output-is-input-2", '-m -o f', {OUT=>''},
> ++ {IN=> {f=> "a\n"}}, {IN=> {g=> "b\n"}}, {IN=> {h=> "c\n"}},
> ++ {CMP=> ["a\nb\nc\n", {'f'=> undef}]} ],
> ++["output-is-input-3", '-m -o f', {OUT=>''},
> ++ {IN=> {g=> "a\n"}}, {IN=> {h=> "b\n"}}, {IN=> {f=> "c\n"}},
> ++ {CMP=> ["a\nb\nc\n", {'f'=> undef}]} ],
> + );
> +
> + # Add _POSIX2_VERSION=199209 to the environment of each test
> diff --git a/recipes/coreutils/coreutils-7.2/man.patch b/recipes/coreutils/coreutils-7.2/man.patch
> new file mode 100644
> index 0000000..3de7c01
> --- /dev/null
> +++ b/recipes/coreutils/coreutils-7.2/man.patch
> @@ -0,0 +1,47 @@
> +
> +#
> +# Patch managed by http://www.holgerschurig.de/patcher.html
> +#
> +
> +Index: coreutils-7.1/configure.ac
> +===================================================================
> +--- coreutils-7.1.orig/configure.ac
> ++++ coreutils-7.1/configure.ac
> +@@ -373,6 +373,20 @@
> + # For a test of uniq: it uses the $LOCALE_FR envvar.
> + gt_LOCALE_FR
> +
> ++AC_MSG_CHECKING([whether to build man pages])
> ++AC_ARG_WITH(manpages,
> ++ AS_HELP_STRING([--with-manpages],
> ++ [Enable building of manpages (default=yes)]),
> ++ [cu_cv_build_manpages=$enableval],
> ++ [cu_cv_build_manpages=yes])
> ++# help2man doesn't work when crosscompiling, as it needs to run the
> ++# binary that was built.
> ++if test x"$cross_compiling" = x"yes"; then
> ++ cu_cv_build_manpages=no
> ++fi
> ++AC_MSG_RESULT($cu_cv_build_manpages)
> ++AM_CONDITIONAL(ENABLE_MANPAGES, test x"$cu_cv_build_manpages" = x"yes")
> ++
> + AC_CONFIG_FILES(
> + Makefile
> + doc/Makefile
> +Index: coreutils-7.1/Makefile.am
> +===================================================================
> +--- coreutils-7.1.orig/Makefile.am
> ++++ coreutils-7.1/Makefile.am
> +@@ -15,7 +15,12 @@
> + # You should have received a copy of the GNU General Public License
> + # along with this program. If not, see <http://www.gnu.org/licenses/>.
> +
> ++if ENABLE_MANPAGES
> + SUBDIRS = lib src doc man po tests gnulib-tests
> ++else
> ++SUBDIRS = lib src doc po tests gnulib-tests
> ++endif
> ++
> + EXTRA_DIST = cfg.mk maint.mk \
> + .prev-version THANKS-to-translators THANKStt.in \
> + .version \
> diff --git a/recipes/coreutils/coreutils-7.2/onceonly.m4 b/recipes/coreutils/coreutils-7.2/onceonly.m4
> new file mode 100644
> index 0000000..f6fec37
> --- /dev/null
> +++ b/recipes/coreutils/coreutils-7.2/onceonly.m4
> @@ -0,0 +1,63 @@
> +# onceonly.m4 serial 3
> +dnl Copyright (C) 2002, 2003 Free Software Foundation, Inc.
> +dnl This file is free software, distributed under the terms of the GNU
> +dnl General Public License. As a special exception to the GNU General
> +dnl Public License, this file may be distributed as part of a program
> +dnl that contains a configuration script generated by Autoconf, under
> +dnl the same distribution terms as the rest of that program.
> +
> +dnl This file defines some "once only" variants of standard autoconf macros.
> +dnl AC_CHECK_HEADERS_ONCE like AC_CHECK_HEADERS
> +dnl AC_CHECK_FUNCS_ONCE like AC_CHECK_FUNCS
> +dnl AC_CHECK_DECLS_ONCE like AC_CHECK_DECLS
> +dnl AC_REQUIRE([AC_HEADER_STDC]) like AC_HEADER_STDC
> +dnl The advantage is that the check for each of the headers/functions/decls
> +dnl will be put only once into the 'configure' file. It keeps the size of
> +dnl the 'configure' file down, and avoids redundant output when 'configure'
> +dnl is run.
> +dnl The drawback is that the checks cannot be conditionalized. If you write
> +dnl if some_condition; then gl_CHECK_HEADERS(stdlib.h); fi
> +dnl inside an AC_DEFUNed function, the gl_CHECK_HEADERS macro call expands to
> +dnl empty, and the check will be inserted before the body of the AC_DEFUNed
> +dnl function.
> +
> +dnl Autoconf version 2.57 or newer is recommended.
> +AC_PREREQ(2.54)
> +
> +# AC_CHECK_HEADERS_ONCE(HEADER1 HEADER2 ...) is a once-only variant of
> +# AC_CHECK_HEADERS(HEADER1 HEADER2 ...).
> +AC_DEFUN([AC_CHECK_HEADERS_ONCE], [
> + :
> + AC_FOREACH([gl_HEADER_NAME], [$1], [
> + AC_DEFUN([gl_CHECK_HEADER_]m4_quote(translit(defn([gl_HEADER_NAME]),
> + [-./], [___])), [
> + AC_CHECK_HEADERS(gl_HEADER_NAME)
> + ])
> + AC_REQUIRE([gl_CHECK_HEADER_]m4_quote(translit(gl_HEADER_NAME,
> + [-./], [___])))
> + ])
> +])
> +
> +# AC_CHECK_FUNCS_ONCE(FUNC1 FUNC2 ...) is a once-only variant of
> +# AC_CHECK_FUNCS(FUNC1 FUNC2 ...).
> +AC_DEFUN([AC_CHECK_FUNCS_ONCE], [
> + :
> + AC_FOREACH([gl_FUNC_NAME], [$1], [
> + AC_DEFUN([gl_CHECK_FUNC_]defn([gl_FUNC_NAME]), [
> + AC_CHECK_FUNCS(defn([gl_FUNC_NAME]))
> + ])
> + AC_REQUIRE([gl_CHECK_FUNC_]defn([gl_FUNC_NAME]))
> + ])
> +])
> +
> +# AC_CHECK_DECLS_ONCE(DECL1 DECL2 ...) is a once-only variant of
> +# AC_CHECK_DECLS(DECL1, DECL2, ...).
> +AC_DEFUN([AC_CHECK_DECLS_ONCE], [
> + :
> + AC_FOREACH([gl_DECL_NAME], [$1], [
> + AC_DEFUN([gl_CHECK_DECL_]defn([gl_DECL_NAME]), [
> + AC_CHECK_DECLS(defn([gl_DECL_NAME]))
> + ])
> + AC_REQUIRE([gl_CHECK_DECL_]defn([gl_DECL_NAME]))
> + ])
> +])
> diff --git a/recipes/coreutils/coreutils-native_7.2.bb b/recipes/coreutils/coreutils-native_7.2.bb
> new file mode 100644
> index 0000000..057dfa6
> --- /dev/null
> +++ b/recipes/coreutils/coreutils-native_7.2.bb
> @@ -0,0 +1,5 @@
> +FILESDIR = "${@os.path.dirname(bb.data.getVar('FILE',d,1))}/coreutils-${PV}"
> +S = "${WORKDIR}/coreutils-${PV}"
> +
> +require coreutils_${PV}.bb
> +inherit native
> diff --git a/recipes/coreutils/coreutils_7.2.bb b/recipes/coreutils/coreutils_7.2.bb
> new file mode 100644
> index 0000000..f80921a
> --- /dev/null
> +++ b/recipes/coreutils/coreutils_7.2.bb
> @@ -0,0 +1,89 @@
> +require coreutils.inc
> +
> +SRC_URI = "\
> + ftp://ftp.gnu.org/gnu/coreutils/coreutils-${PV}.tar.gz \
> + file://automake-version.patch;patch=1 \
> + file://man.patch;patch=1 \
> + file://coreutils-7.2-fix-sort-segfault.patch;patch=1 \
> + file://onceonly.m4 \
> +"
> +
> +# [ gets a special treatment and is not included in this
> +bindir_progs = "basename cksum comm csplit cut df dir dircolors dirname du \
> + env expand expr factor fmt fold groups head hostid id install \
> + join link logname md5sum mkfifo nice nl nohup od paste pathchk \
> + pinky pr printenv printf ptx readlink seq sha1sum shred sort \
> + split stat sum tac tail tee test tr tsort tty unexpand uniq \
> + unlink users vdir wc who whoami yes \
> + "
> +
> +# hostname gets a special treatment and is not included in this
> +base_bindir_progs = "cat chgrp chmod chown cp date dd echo false kill \
> + ln ls mkdir mknod mv pwd rm rmdir sleep stty sync touch \
> + true uname \
> + "
> +
> +sbindir_progs= "chroot"
> +
> +do_configure_prepend() {
> + install -m 0644 ${WORKDIR}/onceonly.m4 ${S}/m4
> +}
> +
> +do_install () {
> + autotools_do_install
> +
> + # Renaming the utilities that should go in /usr/bin
> + for i in ${bindir_progs}; do mv ${D}${bindir}/$i ${D}${bindir}/$i.${PN}; done
> +
> + # Renaming and moving the utilities that should go in /bin (FHS)
> + install -d ${D}${base_bindir}
> + for i in ${base_bindir_progs}; do mv ${D}${bindir}/$i ${D}${base_bindir}/$i.${PN}; done
> +
> + # Renaming and moving the utilities that should go in /usr/sbin (FHS)
> + install -d ${D}${sbindir}
> + for i in ${sbindir_progs}; do mv ${D}${bindir}/$i ${D}${sbindir}/$i.${PN}; done
> +
> + # [ requires special handling because [.coreutils will cause the sed stuff
> + # in update-alternatives to fail, therefore use lbracket - the name used
> + # for the actual source file.
> + mv ${D}${bindir}/[ ${D}${bindir}/lbracket.${PN}
> + # hostname and uptime separated. busybox's versions are preferred
> + if [ -e ${D}${bindir}/hostname ]; then
> + mv ${D}${bindir}/hostname ${D}${base_bindir}/hostname.${PN}
> + fi
> + if [ -e ${D}${bindir}/uptime ]; then
> + mv ${D}${bindir}/uptime ${D}${bindir}/uptime.${PN}
> + fi
> +}
> +
> +pkg_postinst_${PN} () {
> + # The utilities in /usr/bin
> + for i in ${bindir_progs}; do update-alternatives --install ${bindir}/$i $i $i.${PN} 100; done
> +
> + # The utilities in /bin
> + for i in ${base_bindir_progs}; do update-alternatives --install ${base_bindir}/$i $i $i.${PN} 100; done
> +
> + # The utilities in /usr/sbin
> + for i in ${sbindir_progs}; do update-alternatives --install ${sbindir}/$i $i $i.${PN} 100; done
> +
> + # Special cases. uptime and hostname is broken, prefer busybox's version. [ needs to be treated separately.
> + update-alternatives --install ${bindir}/uptime uptime uptime.${PN} 10
> + update-alternatives --install ${base_bindir}/hostname hostname hostname.${PN} 10
> + update-alternatives --install '${bindir}/[' '[' 'lbracket.${PN}' 100
> +}
> +
> +pkg_prerm_${PN} () {
> + # The utilities in /usr/bin
> + for i in ${bindir_progs}; do update-alternatives --remove $i $i.${PN}; done
> +
> + # The utilities in /bin
> + for i in ${base_bindir_progs}; do update-alternatives --remove $i $i.${PN}; done
> +
> + # The utilities in /usr/sbin
> + for i in ${sbindir_progs}; do update-alternatives --remove $i $i.${PN}; done
> +
> + # The special cases
> + update-alternatives --remove hostname hostname.${PN}
> + update-alternatives --remove uptime uptime.${PN}
> + update-alternatives --remove '[' 'lbracket.${PN}'
> +}
> --
> 1.6.2.4
>
>
> --
> This message has been scanned for viruses and
> dangerous content by MailScanner, and is
> believed to be clean.
>
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>
--
Otavio Salvador O.S. Systems
E-mail: otavio at ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
More information about the Openembedded-devel
mailing list