Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Ross Burton <ross.burton@intel.com>
To: openembedded-core@lists.openembedded.org
Cc: openembedded-devel@lists.openembedded.org
Subject: [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf
Date: Tue, 15 Jul 2014 17:18:30 +0100	[thread overview]
Message-ID: <cover.1405440532.git.ross.burton@intel.com> (raw)

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.

Ross

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



             reply	other threads:[~2014-07-15 16:18 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-15 16:18 Ross Burton [this message]
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 21:51     ` Burton, Ross
2014-07-16 16:39       ` Khem Raj
2014-07-15 16:58 ` [oe] [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf Martin Jansa
2014-07-16 22:47   ` Richard Purdie
2014-07-17 22:11     ` 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=cover.1405440532.git.ross.burton@intel.com \
    --to=ross.burton@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox