Opened 4 years ago

Closed 3 years ago

#21526 closed defect (fixed)

package autotools: '$SAGE_LOCAL/automake-1.11.6/share/aclocal': No such file or directory

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-8.1
Component: packages: optional Keywords:
Cc: Merged in:
Authors: Erik Bray Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 099da4f (Commits) Commit: 099da4f5ecbb16e91f1d153e3a72cc6fab8becae
Dependencies: Stopgaps:

Description

I get this when installing ecm.

A workaround is:

cd automake-1.11.6/share/ && ln -s aclocal-1.11/ aclocal

Unclear if this a bug of a specific automake version, or a general shortcoming of the installation scheme of our autotools. This needs investigating.

Change History (9)

comment:1 Changed 3 years ago by embray

I'm seeing this too when trying to build giac.

comment:2 Changed 3 years ago by embray

The directory it's complaining about is a path that aclocal searches for "system-wide" m4 macros (which since we install a bunch of automake versions ends up localized to local/automake-<version>). Some newer automake versions are creating this directory by default (empty, except for a README file) whereas older versions, it seems, do not create it by default.

For older versions we may have to create it.

aclocal has a function that scans this directory for m4 macros, but the function errors out if the directory does not exist (which itself seems a bit unnecessary...)

comment:3 Changed 3 years ago by embray

  • Authors set to Erik Bray
  • Branch set to u/embray/build/ticket-21526
  • Commit set to 2f2ea87699feb78ba3aecd974af147f60121cd99
  • Component changed from packages: experimental to packages: optional
  • Milestone changed from sage-7.4 to sage-8.1
  • Status changed from new to needs_review

This fixes the issue. The question remains why building giac is invoking autoconf in the first place--it shouldn't be. But that's a separate issue.


New commits:

2f2ea87Fixes #21526 by simply creating the empty directory if it doesn't exist.

comment:4 Changed 3 years ago by mkoeppe

Looks good to me.

It seems to be recommended though that the patch level is increased when there is a change to the package like this.

comment:5 Changed 3 years ago by mkoeppe

  • Reviewers set to Matthias Koeppe

comment:6 Changed 3 years ago by embray

You are right, it should probably bump the patch level.

comment:7 Changed 3 years ago by git

  • Commit changed from 2f2ea87699feb78ba3aecd974af147f60121cd99 to 099da4f5ecbb16e91f1d153e3a72cc6fab8becae

Branch pushed to git repo; I updated commit sha1. New commits:

099da4fBump autotools package version so it will reinstall with this fix

comment:8 Changed 3 years ago by mkoeppe

  • Status changed from needs_review to positive_review

comment:9 Changed 3 years ago by vbraun

  • Branch changed from u/embray/build/ticket-21526 to 099da4f5ecbb16e91f1d153e3a72cc6fab8becae
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.