search

geodynamics / mineos

Mineos computes synthetic seismograms in a spherically symmetric non-rotating Earth by summing normal modes.
http://geodynamics.org/cig/software/mineos/
GNU General Public License v2.0
24 stars 19 forks source link

Commit configure #10

Open anowacki opened 4 years ago

anowacki commented 4 years ago

When distributing software using the GNU autotools system, it is usual to also distribute the configure script so that end-users don't need to install the whole autotools system themselves. Currently the repo does not include configure, so the installation instructions in INSTALL are not correct and can't be followed.

I managed to build by trying various combinations of autoreconf, autoconf, and automake --add-missing, but most users will not have much experience with this. My own attempts still throw errors along the way, and I don't think I know enough m4 to send a PR to fix those.

One solution to including configure more easily might be a git pre-commit hook.

(If interested, I can separately file a ticket explaining the errors an warnings I experience in running autoconf to build the configure script.)

ljhwang commented 6 months ago

Hi Andy

We are updating our repos with dockerfiles and noticed you have addressed this issue in a separate branch. Can you merge this branch back?

ty

anowacki commented 5 months ago

@ljhwang it looks like @JarettBakerDunn has done this in #13 but please do ping me again if any help is needed. I am also happy to point the Julia build infrastructure back to this repo once that is merged.

(If you aren't familiar with the Julia Yggdrasil repository, it cross-compiles binaries for all platforms from source to be used in Julia packages, but those binaries can still be used outside of Julia. Hence Mineos is currently being built for the platforms listed here and is used by Mineos.jl.)