From d577b6fcdf9c8963329e01b12907d526b4481129 Mon Sep 17 00:00:00 2001 From: Sergey Kosukhin Date: Thu, 28 Jan 2021 11:21:04 +0100 Subject: [PATCH] autoconf: fix v2.70 and patching procedure of bin/autom4te.in (#21255) This PR adds: 1. A patch that fixes a bug in version 2.70 (will be fixed upstream in the next release: https://savannah.gnu.org/support/?110396). 2. A fix for the way we patch shebang in bin/autom4te.in. For 2, we need to keep the original modification timestamp of the file. Otherwise, we either get an empty man page for autom4te (versions 2.69 and before) or a failure at the build time (versions 2.70 and after). The difference has to do with the update of the missing script: https://git.savannah.gnu.org/cgit/automake.git/commit/lib/missing?id=a22717dffe37f30ef2ad2c355b68c9b3b5e4b8c7 It will take time until developers of Autotools-based packages adjust their scripts to the new version, therefore, 2.69 is marked as preferred. --- .../builtin/packages/autoconf/package.py | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/var/spack/repos/builtin/packages/autoconf/package.py b/var/spack/repos/builtin/packages/autoconf/package.py index 4e5c079bfa..1ba77a4240 100644 --- a/var/spack/repos/builtin/packages/autoconf/package.py +++ b/var/spack/repos/builtin/packages/autoconf/package.py @@ -13,11 +13,17 @@ class Autoconf(AutotoolsPackage, GNUMirrorPackage): gnu_mirror_path = 'autoconf/autoconf-2.69.tar.gz' version('2.70', sha256='f05f410fda74323ada4bdc4610db37f8dbd556602ba65bc843edb4d4d4a1b2b7') - version('2.69', sha256='954bd69b391edc12d6a4a51a2dd1476543da5c6bbf05a95b59dc0dd6fd4c2969') + version('2.69', sha256='954bd69b391edc12d6a4a51a2dd1476543da5c6bbf05a95b59dc0dd6fd4c2969', + preferred=True) version('2.62', sha256='83aa747e6443def0ebd1882509c53f5a2133f502ddefa21b3de141c433914bdd') version('2.59', sha256='9cd05c73c5fcb1f5ccae53dd6cac36bb8cb9c7b3e97ffae5a7c05c72594c88d8') version('2.13', sha256='f0611136bee505811e9ca11ca7ac188ef5323a8e2ef19cffd3edb3cf08fd791e') + # https://savannah.gnu.org/support/?110396 + patch('https://git.savannah.gnu.org/cgit/autoconf.git/patch/?id=05972f49ee632cd98057a3caf82ebfb9574846da', + sha256='eaa3f69d927a853313a0b06e2117c51adab6377a2278549b05abc5df93643e16', + when='@2.70') + # Note: m4 is not a pure build-time dependency of autoconf. m4 is # needed when autoconf runs, not only when autoconf is built. depends_on('m4@1.4.6:', type=('build', 'run')) @@ -39,9 +45,14 @@ def determine_version(cls, exe): def patch(self): # The full perl shebang might be too long; we have to fix this here # because autom4te is called during the build - filter_file('^#! @PERL@ -w', - '#! /usr/bin/env perl', - 'bin/autom4te.in') + patched_file = 'bin/autom4te.in' + + # We save and restore the modification timestamp of the file to prevent + # regeneration of the respective man page: + with keep_modification_time(patched_file): + filter_file('^#! @PERL@ -w', + '#! /usr/bin/env perl', + patched_file) @run_after('install') def filter_sbang(self):