From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [oe] [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf
Date: Tue, 15 Jul 2014 18:58:42 +0200 [thread overview]
Message-ID: <20140715165842.GN2433@jama> (raw)
In-Reply-To: <cover.1405440532.git.ross.burton@intel.com>
[-- Attachment #1: Type: text/plain, Size: 5474 bytes --]
On Tue, Jul 15, 2014 at 05:18:30PM +0100, Ross Burton wrote:
> Hi,
>
> Since 2004 we've been forcibly passing --foreign to automake in autoreconf.
> This is a change from upstream behaviour and can actually break packages that
> expect GNU behaviour (admittedly the failure case reported was "interesting"),
> and I'm on a bit of a mission to remove patches that don't have a solid reason
> behind them.
>
> Forcing --foreign doesn't seem that important these days as the majority of
> packages are telling automake they're foreign, There are a few packages that
> need patches for this change, notably they're mostly packages that haven't seen
> a release for ten years.
>
> This series is build tested against a world build in oe-core for NUC, so I'd
> call it a demonstration that the patch is viable and would appreciate others
> running build tests against other machines or layers.
world builds from
http://www.openembedded.org/wiki/Bitbake_World_Status
already have around ~50 failed recipes per architecture, I don't think
we need more now :/
Do we have some volunteers to fix-up meta-oe/* layers?
I'm going to use PNBLACKLIST more aggressively as agreed on OEDAM, but
with this speed next meta-oe release will have half of recipes which
were available in daisy.
> The following changes since commit 9e99bcffec594cb5b4c3533200e8eb56d32e6a77:
>
> insane: Ensure do_package_qa happens after do_packagedata (2014-07-11 23:20:04 +0100)
>
> are available in the git repository at:
>
> git://git.yoctoproject.org/poky-contrib ross/automake
>
> for you to fetch changes up to 406153008dc78f4e3198c012f2db98fcac09b5c1:
>
> autoconf: remove automake patch enforcing --foreign (2014-07-15 17:05:29 +0100)
>
> ----------------------------------------------------------------
> Ross Burton (7):
> bzip2: use "foreign" automake strictness
> libid3tag: use "foreign" automake strictness
> which: use "foreign" automake strictness
> libmad: use "foreign" automake strictness
> oprofile: use "foreign" automake strictness
> ltp: use "foreign" automake strictness
> autoconf: remove automake patch enforcing --foreign
>
> .../autoconf/autoconf/autoreconf-foreign.patch | 13 -------------
> meta/recipes-devtools/autoconf/autoconf_2.69.bb | 1 -
> .../bzip2/bzip2-1.0.6/configure.ac | 2 +-
> .../ltp/ltp/automake-foreign.patch | 20 ++++++++++++++++++++
> meta/recipes-extended/ltp/ltp_20140422.bb | 1 +
> .../which/which-2.20/automake.patch | 10 ++++++++++
> meta/recipes-extended/which/which_2.20.bb | 1 +
> meta/recipes-kernel/oprofile/oprofile.inc | 1 +
> .../oprofile/oprofile/automake-foreign.patch | 7 +++++++
> .../libid3tag/obsolete_automake_macros.patch | 3 ++-
> .../libmad/libmad/automake-foreign.patch | 12 ++++++++++++
> meta/recipes-multimedia/libmad/libmad_0.15.1b.bb | 1 +
> 12 files changed, 56 insertions(+), 16 deletions(-)
> delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoreconf-foreign.patch
> create mode 100644 meta/recipes-extended/ltp/ltp/automake-foreign.patch
> create mode 100644 meta/recipes-extended/which/which-2.20/automake.patch
> create mode 100644 meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
> create mode 100644 meta/recipes-multimedia/libmad/libmad/automake-foreign.patch
>
> Ross Burton (7):
> bzip2: use "foreign" automake strictness
> libid3tag: use "foreign" automake strictness
> which: use "foreign" automake strictness
> libmad: use "foreign" automake strictness
> oprofile: use "foreign" automake strictness
> ltp: use "foreign" automake strictness
> autoconf: remove automake patch enforcing --foreign
>
> .../autoconf/autoconf/autoreconf-foreign.patch | 13 -------------
> meta/recipes-devtools/autoconf/autoconf_2.69.bb | 1 -
> .../bzip2/bzip2-1.0.6/configure.ac | 2 +-
> .../ltp/ltp/automake-foreign.patch | 20 ++++++++++++++++++++
> meta/recipes-extended/ltp/ltp_20140422.bb | 1 +
> .../which/which-2.20/automake.patch | 10 ++++++++++
> meta/recipes-extended/which/which_2.20.bb | 1 +
> meta/recipes-kernel/oprofile/oprofile.inc | 1 +
> .../oprofile/oprofile/automake-foreign.patch | 7 +++++++
> .../libid3tag/obsolete_automake_macros.patch | 3 ++-
> .../libmad/libmad/automake-foreign.patch | 12 ++++++++++++
> meta/recipes-multimedia/libmad/libmad_0.15.1b.bb | 1 +
> 12 files changed, 56 insertions(+), 16 deletions(-)
> delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoreconf-foreign.patch
> create mode 100644 meta/recipes-extended/ltp/ltp/automake-foreign.patch
> create mode 100644 meta/recipes-extended/which/which-2.20/automake.patch
> create mode 100644 meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
> create mode 100644 meta/recipes-multimedia/libmad/libmad/automake-foreign.patch
>
> --
> 1.7.10.4
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf
Date: Tue, 15 Jul 2014 18:58:42 +0200 [thread overview]
Message-ID: <20140715165842.GN2433@jama> (raw)
In-Reply-To: <cover.1405440532.git.ross.burton@intel.com>
[-- Attachment #1: Type: text/plain, Size: 5474 bytes --]
On Tue, Jul 15, 2014 at 05:18:30PM +0100, Ross Burton wrote:
> Hi,
>
> Since 2004 we've been forcibly passing --foreign to automake in autoreconf.
> This is a change from upstream behaviour and can actually break packages that
> expect GNU behaviour (admittedly the failure case reported was "interesting"),
> and I'm on a bit of a mission to remove patches that don't have a solid reason
> behind them.
>
> Forcing --foreign doesn't seem that important these days as the majority of
> packages are telling automake they're foreign, There are a few packages that
> need patches for this change, notably they're mostly packages that haven't seen
> a release for ten years.
>
> This series is build tested against a world build in oe-core for NUC, so I'd
> call it a demonstration that the patch is viable and would appreciate others
> running build tests against other machines or layers.
world builds from
http://www.openembedded.org/wiki/Bitbake_World_Status
already have around ~50 failed recipes per architecture, I don't think
we need more now :/
Do we have some volunteers to fix-up meta-oe/* layers?
I'm going to use PNBLACKLIST more aggressively as agreed on OEDAM, but
with this speed next meta-oe release will have half of recipes which
were available in daisy.
> The following changes since commit 9e99bcffec594cb5b4c3533200e8eb56d32e6a77:
>
> insane: Ensure do_package_qa happens after do_packagedata (2014-07-11 23:20:04 +0100)
>
> are available in the git repository at:
>
> git://git.yoctoproject.org/poky-contrib ross/automake
>
> for you to fetch changes up to 406153008dc78f4e3198c012f2db98fcac09b5c1:
>
> autoconf: remove automake patch enforcing --foreign (2014-07-15 17:05:29 +0100)
>
> ----------------------------------------------------------------
> Ross Burton (7):
> bzip2: use "foreign" automake strictness
> libid3tag: use "foreign" automake strictness
> which: use "foreign" automake strictness
> libmad: use "foreign" automake strictness
> oprofile: use "foreign" automake strictness
> ltp: use "foreign" automake strictness
> autoconf: remove automake patch enforcing --foreign
>
> .../autoconf/autoconf/autoreconf-foreign.patch | 13 -------------
> meta/recipes-devtools/autoconf/autoconf_2.69.bb | 1 -
> .../bzip2/bzip2-1.0.6/configure.ac | 2 +-
> .../ltp/ltp/automake-foreign.patch | 20 ++++++++++++++++++++
> meta/recipes-extended/ltp/ltp_20140422.bb | 1 +
> .../which/which-2.20/automake.patch | 10 ++++++++++
> meta/recipes-extended/which/which_2.20.bb | 1 +
> meta/recipes-kernel/oprofile/oprofile.inc | 1 +
> .../oprofile/oprofile/automake-foreign.patch | 7 +++++++
> .../libid3tag/obsolete_automake_macros.patch | 3 ++-
> .../libmad/libmad/automake-foreign.patch | 12 ++++++++++++
> meta/recipes-multimedia/libmad/libmad_0.15.1b.bb | 1 +
> 12 files changed, 56 insertions(+), 16 deletions(-)
> delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoreconf-foreign.patch
> create mode 100644 meta/recipes-extended/ltp/ltp/automake-foreign.patch
> create mode 100644 meta/recipes-extended/which/which-2.20/automake.patch
> create mode 100644 meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
> create mode 100644 meta/recipes-multimedia/libmad/libmad/automake-foreign.patch
>
> Ross Burton (7):
> bzip2: use "foreign" automake strictness
> libid3tag: use "foreign" automake strictness
> which: use "foreign" automake strictness
> libmad: use "foreign" automake strictness
> oprofile: use "foreign" automake strictness
> ltp: use "foreign" automake strictness
> autoconf: remove automake patch enforcing --foreign
>
> .../autoconf/autoconf/autoreconf-foreign.patch | 13 -------------
> meta/recipes-devtools/autoconf/autoconf_2.69.bb | 1 -
> .../bzip2/bzip2-1.0.6/configure.ac | 2 +-
> .../ltp/ltp/automake-foreign.patch | 20 ++++++++++++++++++++
> meta/recipes-extended/ltp/ltp_20140422.bb | 1 +
> .../which/which-2.20/automake.patch | 10 ++++++++++
> meta/recipes-extended/which/which_2.20.bb | 1 +
> meta/recipes-kernel/oprofile/oprofile.inc | 1 +
> .../oprofile/oprofile/automake-foreign.patch | 7 +++++++
> .../libid3tag/obsolete_automake_macros.patch | 3 ++-
> .../libmad/libmad/automake-foreign.patch | 12 ++++++++++++
> meta/recipes-multimedia/libmad/libmad_0.15.1b.bb | 1 +
> 12 files changed, 56 insertions(+), 16 deletions(-)
> delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoreconf-foreign.patch
> create mode 100644 meta/recipes-extended/ltp/ltp/automake-foreign.patch
> create mode 100644 meta/recipes-extended/which/which-2.20/automake.patch
> create mode 100644 meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
> create mode 100644 meta/recipes-multimedia/libmad/libmad/automake-foreign.patch
>
> --
> 1.7.10.4
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
next prev parent reply other threads:[~2014-07-15 16:58 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-15 16:18 [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf Ross Burton
2014-07-15 16:18 ` [PATCH 1/7] bzip2: use "foreign" automake strictness Ross Burton
2014-07-15 16:18 ` [PATCH 2/7] libid3tag: " Ross Burton
2014-07-15 16:18 ` [PATCH 3/7] which: " Ross Burton
2014-07-15 16:18 ` [PATCH 4/7] libmad: " Ross Burton
2014-07-15 16:18 ` [PATCH 5/7] oprofile: " Ross Burton
2014-07-15 16:18 ` [PATCH 6/7] ltp: " Ross Burton
2014-07-15 16:18 ` [PATCH 7/7] autoconf: remove automake patch enforcing --foreign Ross Burton
2014-07-15 20:14 ` [oe] " Khem Raj
2014-07-15 20:14 ` Khem Raj
2014-07-15 21:51 ` [oe] " Burton, Ross
2014-07-15 21:51 ` Burton, Ross
2014-07-16 16:39 ` [oe] " Khem Raj
2014-07-16 16:39 ` [OE-core] " Khem Raj
2014-07-15 16:58 ` Martin Jansa [this message]
2014-07-15 16:58 ` [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf Martin Jansa
2014-07-16 22:47 ` [oe] " Richard Purdie
2014-07-16 22:47 ` [OE-core] " Richard Purdie
2014-07-17 22:11 ` [oe] " Richard Purdie
2014-07-17 22:11 ` [OE-core] " Richard Purdie
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=20140715165842.GN2433@jama \
--to=martin.jansa@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=openembedded-devel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.