All of lore.kernel.org
 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: 20+ 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 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 ` [oe] [PATCH 0/7][RFC] Remove enforced --foreign in autoreconf Martin Jansa
2014-07-15 16:58   ` 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=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 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.