build: Drop deprecated ACLOCAL_AMFLAGS variable #18333

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:20200311-deprecated-amflags changing 1 files +0 −1
  1. hebasto commented at 3:39 PM on March 12, 2020: member

    This PR is a followup of #18290.

    The special make variable ACLOCAL_AMFLAGS is deprecated since Automake 1.13 release. AC_CONFIG_MACRO_DIR macro is traced by aclocal, and is used to declare the local m4 include directories.

    For the reference:

  2. build: Drop deprecated ACLOCAL_AMFLAGS variable
    The special make variable ACLOCAL_AMFLAGS is deprecated since Automake
    1.13 release. AC_CONFIG_MACRO_DIR macro is traced by aclocal, and is
    used to declare the local m4 include directories.
    ef8cdb0e6c
  3. dongcarl commented at 3:49 PM on March 12, 2020: member

    Concept ACK, thanks for keeping track of these deprecations!


    Here's the relevant excerpt from the automake 1.14 release notes:

      - The ACLOCAL_AMFLAGS special make variable will be fully deprecated in
        Automake 2.0: it will raise warnings in the "obsolete" category (but
        still no hard error of course, for compatibilities with the many, many
        packages that still rely on that variable).  You are advised to
        start relying on the new Automake support for AC_CONFIG_MACRO_DIRS
        instead (which was introduced in Automake 1.13).
    

    It seems that we're already using AC_CONFIG_MACRO_DIR{,S} here https://github.com/bitcoin/bitcoin/blob/ef8cdb0e6cb2c019a2e4a05aa15cc463a10477d7/configure.ac#L15

  4. DrahtBot added the label Build system on Mar 12, 2020
  5. fanquake commented at 11:46 PM on March 12, 2020: member

    ~0. I realize I pointed this out in the previous PR. However keeping this as is is harmless, and dropping it just introduces a warning:

    glibtoolize: Consider adding '-I build-aux/m4' to ACLOCAL_AMFLAGS in Makefile.am.
    

    that someone will just as likely try and fix.

  6. hebasto commented at 10:09 AM on March 13, 2020: member

    From libtool docs:

    If libtoolize detects an explicit call to AC_CONFIG_MACRO_DIRS ... in your configure.ac, it will put the Libtool macros in the specified directory.

    Unless --no-warn is passed, libtoolize displays hints for adding libtool support to your package, as well.

    I think that is a hint, not a warning. @fanquake

    ... someone will just as likely try and fix.

    Yeah, the maintainers' burden could be increased. So, it is up to you to close this PR.

  7. fanquake commented at 7:42 AM on April 9, 2020: member

    Let's come back to this in the future.

  8. fanquake closed this on Apr 9, 2020

  9. hebasto deleted the branch on Apr 9, 2020
  10. DrahtBot locked this on Feb 15, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 21:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me