From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-core@lists.openembedded.org,
Constantin Musca <constantinx.musca@intel.com>
Cc: openembedded-commits@lists.openembedded.org
Subject: Re: [oe-commits] Constantin Musca : augeas: upgrade to 1.0.0
Date: Sun, 24 Feb 2013 17:43:38 +0100 [thread overview]
Message-ID: <20130224164338.GC3300@jama> (raw)
In-Reply-To: <20130222140918.682E65029C@opal>
[-- Attachment #1: Type: text/plain, Size: 7033 bytes --]
On Fri, Feb 22, 2013 at 02:09:18PM +0000, git@git.openembedded.org wrote:
> Module: openembedded-core.git
> Branch: master
> Commit: a711a8617c39b06c9a494d0ceae9f96f65b024f9
> URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=a711a8617c39b06c9a494d0ceae9f96f65b024f9
>
> Author: Constantin Musca <constantinx.musca@intel.com>
> Date: Mon Feb 18 14:05:09 2013 +0200
>
> augeas: upgrade to 1.0.0
Fails to fetch here:
WARNING: Failed to fetch URL
http://augeas.net/download/augeas-1.0.0.tar.gz, attempting MIRRORS if
available
ERROR: Fetcher failure: Fetch command failed with exit code 8, output:
http://augeas.net/download/augeas-1.0.0.tar.gz:
2013-02-23 18:34:48 ERROR 404: Not Found.
http://download.augeas.net/augeas-1.0.0.tar.gz
Seems like correct URL now.
>
> 0001-Add-missing-argument-to-escape.patch: removed
> - included in the new version
>
> remove-gets.patch: not needed anymore
>
> add-missing-argz-conditional.patch: added
> - the Makefiles assume that the argz conditional
> is present in configure.ac
>
> Signed-off-by: Constantin Musca <constantinx.musca@intel.com>
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
>
> ---
>
> meta/recipes-extended/augeas/augeas.inc | 2 +-
> .../0001-Add-missing-argument-to-escape.patch | 29 --------------------
> .../augeas/add-missing-argz-conditional.patch | 20 +++++++++++++
> .../augeas/augeas/remove-gets.patch | 20 -------------
> meta/recipes-extended/augeas/augeas_0.10.0.bb | 8 -----
> meta/recipes-extended/augeas/augeas_1.0.0.bb | 6 ++++
> 6 files changed, 27 insertions(+), 58 deletions(-)
>
> diff --git a/meta/recipes-extended/augeas/augeas.inc b/meta/recipes-extended/augeas/augeas.inc
> index 479fb97..a0aa9dd 100644
> --- a/meta/recipes-extended/augeas/augeas.inc
> +++ b/meta/recipes-extended/augeas/augeas.inc
> @@ -6,7 +6,7 @@ LICENSE = "LGPLv2.1+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=bbb461211a33b134d42ed5ee802b37ff"
>
> SRC_URI = "http://augeas.net/download/${BP}.tar.gz \
> - file://0001-Add-missing-argument-to-escape.patch \
> + file://add-missing-argz-conditional.patch \
> "
>
> DEPENDS = "readline libxml2"
> diff --git a/meta/recipes-extended/augeas/augeas/0001-Add-missing-argument-to-escape.patch b/meta/recipes-extended/augeas/augeas/0001-Add-missing-argument-to-escape.patch
> deleted file mode 100644
> index 3d0d0e3..0000000
> --- a/meta/recipes-extended/augeas/augeas/0001-Add-missing-argument-to-escape.patch
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -Upstream-Status: Backport
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -
> -From 021ea39f8e400225e2d01b4c62eb9d56404f2ecd Mon Sep 17 00:00:00 2001
> -From: Michael Chapman <mike@very.puzzling.org>
> -Date: Fri, 16 Dec 2011 21:30:07 +1100
> -Subject: [PATCH] Add missing argument to escape()
> -
> -Fixes ticket #242.
> ----
> - src/regexp.c | 2 +-
> - 1 files changed, 1 insertions(+), 1 deletions(-)
> -
> -diff --git a/src/regexp.c b/src/regexp.c
> -index cf0ea5d..811087d 100644
> ---- a/src/regexp.c
> -+++ b/src/regexp.c
> -@@ -50,7 +50,7 @@ char *regexp_escape(const struct regexp *r) {
> - ret = fa_restrict_alphabet(r->pattern->str, strlen(r->pattern->str),
> - &nre, &nre_len, 2, 1);
> - if (ret == 0) {
> -- pat = escape(nre, nre_len);
> -+ pat = escape(nre, nre_len, RX_ESCAPES);
> - free(nre);
> - }
> - #endif
> ---
> -1.7.5.4
> -
> diff --git a/meta/recipes-extended/augeas/augeas/add-missing-argz-conditional.patch b/meta/recipes-extended/augeas/augeas/add-missing-argz-conditional.patch
> new file mode 100644
> index 0000000..abbdbaa
> --- /dev/null
> +++ b/meta/recipes-extended/augeas/augeas/add-missing-argz-conditional.patch
> @@ -0,0 +1,20 @@
> +Add missing GL_GENERATE_ARGZ_H conditional
> +
> +- GL_GENERATE_ARGZ_H is used in gnulib/lib/Makefile.am
> +
> +Upstream-Status: Pending
> +Signed-off-by: Constantin Musca <constantinx.musca@intel.com>
> +
> +Index: augeas-1.0.0/configure.ac
> +===================================================================
> +--- augeas-1.0.0.orig/configure.ac
> ++++ augeas-1.0.0/configure.ac
> +@@ -55,6 +55,8 @@ AC_ARG_WITH([failmalloc],
> +
> + AM_CONDITIONAL([WITH_FAILMALLOC], [test x$with_failmalloc != xno])
> +
> ++AM_CONDITIONAL([GL_GENERATE_ARGZ_H], [test -n "$ARGZ_H"])
> ++
> + dnl --enable-debug=(yes|no)
> + AC_ARG_ENABLE([debug],
> + [AC_HELP_STRING([--enable-debug=no/yes],
> diff --git a/meta/recipes-extended/augeas/augeas/remove-gets.patch b/meta/recipes-extended/augeas/augeas/remove-gets.patch
> deleted file mode 100644
> index bd6e92c..0000000
> --- a/meta/recipes-extended/augeas/augeas/remove-gets.patch
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -eglibc 2.16 has remove gets so check for it to be there before using it
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> -
> -Upstream-Status: Pending
> -Index: augeas-0.10.0/gnulib/lib/stdio.in.h
> -===================================================================
> ---- augeas-0.10.0.orig/gnulib/lib/stdio.in.h 2011-03-03 17:07:59.000000000 -0800
> -+++ augeas-0.10.0/gnulib/lib/stdio.in.h 2012-07-03 19:46:42.871894833 -0700
> -@@ -161,8 +161,10 @@
> - /* It is very rare that the developer ever has full control of stdin,
> - so any use of gets warrants an unconditional warning. Assume it is
> - always declared, since it is required by C89. */
> -+#if defined gets
> - #undef gets
> - _GL_WARN_ON_USE (gets, "gets is a security hole - use fgets instead");
> -+#endif
> -
> - #if @GNULIB_FOPEN@
> - # if @REPLACE_FOPEN@
> diff --git a/meta/recipes-extended/augeas/augeas_0.10.0.bb b/meta/recipes-extended/augeas/augeas_0.10.0.bb
> deleted file mode 100644
> index 0840428..0000000
> --- a/meta/recipes-extended/augeas/augeas_0.10.0.bb
> +++ /dev/null
> @@ -1,8 +0,0 @@
> -require augeas.inc
> -
> -SRC_URI += "file://remove-gets.patch \
> - "
> -PR = "r3"
> -
> -SRC_URI[md5sum] = "fe1834e90a066c3208ac0214622c7352"
> -SRC_URI[sha256sum] = "ec111af06186216930176ebe5ecccdf7bf528528aee9acde1d5d70088484afca"
> diff --git a/meta/recipes-extended/augeas/augeas_1.0.0.bb b/meta/recipes-extended/augeas/augeas_1.0.0.bb
> new file mode 100644
> index 0000000..2f7061a
> --- /dev/null
> +++ b/meta/recipes-extended/augeas/augeas_1.0.0.bb
> @@ -0,0 +1,6 @@
> +require augeas.inc
> +
> +PR = "r0"
> +
> +SRC_URI[md5sum] = "82131019432ecf8102e1491610ad2dd1"
> +SRC_URI[sha256sum] = "31bf757c5b8197765946b3805f3793c32b03cd92a7a77ec95d37e71a1f131912"
>
>
> _______________________________________________
> Openembedded-commits mailing list
> Openembedded-commits@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-commits
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
next parent reply other threads:[~2013-02-24 16:59 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20130222140918.682E65029C@opal>
2013-02-24 16:43 ` Martin Jansa [this message]
2013-02-24 16:51 ` [oe-commits] Constantin Musca : augeas: upgrade to 1.0.0 Gary Thomas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130224164338.GC3300@jama \
--to=martin.jansa@gmail.com \
--cc=constantinx.musca@intel.com \
--cc=openembedded-commits@lists.openembedded.org \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox