Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/4][RFT] Stop disabling autoheader warnings
@ 2014-05-28 11:56 Ross Burton
  2014-05-28 11:56 ` [PATCH 1/4] readline: add missing STRUCT_DIRENT_D_* symbols to config.h.in Ross Burton
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Ross Burton @ 2014-05-28 11:56 UTC (permalink / raw)
  To: openembedded-core, openembedded-devel

Hi,

When autoheader emits a warning along the lines of "missing template for
variable HAVE_FOO", it's not moaning that you need to be more verbose for nice
comments in the config.h files.  In fact it's saying that it's not writing that
symbol at all to config.h.in, so even if that test succeeds this isn't reflected
in config.h.

I removed the patch and did a build of core-image-sato: readline was the only
recipe that failed.  I'll spare you the anger but readline is special...  the
short version is that if an upstream package hand-maintains config.h.in instead
of using autoheader then set EXTRA_AUTORECONF += "--exclude=autoheader" and
autoheader won't be executed.

This is also a call out for a request for testing: I've done a core-image-sato
build with this but haven't yet done world or any other layers.  If there's
breakage it's often due to the upstream being unconventional so feel free to
prod me for help.

Ross

The following changes since commit c42df8cb48ceb6081680ffb802e85dad557cf29c:

  i2c-tools: Add i2c-tools to the core (2014-05-28 09:42:14 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ross/autoconf

for you to fetch changes up to a12ceb971fefe2b2f412f46b82c0d5ea6043ad62:

  autoconf: don't disable the autoheader warnings (2014-05-28 12:47:58 +0100)

----------------------------------------------------------------
Ross Burton (4):
      readline: add missing STRUCT_DIRENT_D_* symbols to config.h.in
      readline: exclude autoheader from autoreconf
      readline: use upstream's aclocal.m4 as acinclude.m4
      autoconf: don't disable the autoheader warnings

 meta/recipes-core/readline/files/acinclude.m4      | 1871 --------------------
 .../readline-6.3/config-dirent-symbols.patch       |   34 +
 meta/recipes-core/readline/readline.inc            |   10 +-
 .../autoconf/autoheader-nonfatal-warnings.patch    |   15 -
 meta/recipes-devtools/autoconf/autoconf_2.69.bb    |    1 -
 5 files changed, 41 insertions(+), 1890 deletions(-)
 delete mode 100644 meta/recipes-core/readline/files/acinclude.m4
 create mode 100644 meta/recipes-core/readline/readline-6.3/config-dirent-symbols.patch
 delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoheader-nonfatal-warnings.patch

Ross Burton (4):
  readline: add missing STRUCT_DIRENT_D_* symbols to config.h.in
  readline: exclude autoheader from autoreconf
  readline: use upstream's aclocal.m4 as acinclude.m4
  autoconf: don't disable the autoheader warnings

 meta/recipes-core/readline/files/acinclude.m4      | 1871 --------------------
 .../readline-6.3/config-dirent-symbols.patch       |   34 +
 meta/recipes-core/readline/readline.inc            |   10 +-
 .../autoconf/autoheader-nonfatal-warnings.patch    |   15 -
 meta/recipes-devtools/autoconf/autoconf_2.69.bb    |    1 -
 5 files changed, 41 insertions(+), 1890 deletions(-)
 delete mode 100644 meta/recipes-core/readline/files/acinclude.m4
 create mode 100644 meta/recipes-core/readline/readline-6.3/config-dirent-symbols.patch
 delete mode 100644 meta/recipes-devtools/autoconf/autoconf/autoheader-nonfatal-warnings.patch

-- 
1.7.10.4



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-05-28 11:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-28 11:56 [PATCH 0/4][RFT] Stop disabling autoheader warnings Ross Burton
2014-05-28 11:56 ` [PATCH 1/4] readline: add missing STRUCT_DIRENT_D_* symbols to config.h.in Ross Burton
2014-05-28 11:56 ` [PATCH 2/4] readline: exclude autoheader from autoreconf Ross Burton
2014-05-28 11:56 ` [PATCH 3/4] readline: use upstream's aclocal.m4 as acinclude.m4 Ross Burton
2014-05-28 11:56 ` [PATCH 4/4] autoconf: don't disable the autoheader warnings Ross Burton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox