Closed chrisspen closed 5 years ago
Did you try running 'automake --add-missing' ???
On Wed, Sep 18, 2019 at 11:32 AM Chris Spencer notifications@github.com wrote:
When building the library from the git repo, the README instructs us to generate the ./configure script by running autoreconf. However, this fails on stock Ubuntu 18.04 with the errors:
configure.ac:85: error: required file 'config/compile' not foundconfigure.ac:85: 'automake --add-missing' can install 'compile'configure.ac:12: error: required file 'config/config.guess' not foundconfigure.ac:12: 'automake --add-missing' can install 'config.guess'configure.ac:12: error: required file 'config/config.sub' not foundconfigure.ac:12: 'automake --add-missing' can install 'config.sub'configure.ac:14: error: required file 'config/install-sh' not foundconfigure.ac:14: 'automake --add-missing' can install 'install-sh'configure.ac:85: error: required file 'config/ltmain.sh' not foundconfigure.ac:14: error: required file 'config/missing' not foundconfigure.ac:14: 'automake --add-missing' can install 'missing' benchmarks/Makefile.am: error: required file 'config/depcomp' not found benchmarks/Makefile.am: 'automake --add-missing' can install 'depcomp' doc/Makefile.am:57: error: required file 'config/mdate-sh' not found doc/Makefile.am:57: 'automake --add-missing' can install 'mdate-sh' doc/Makefile.am:57: error: required file 'config/texinfo.tex' not found doc/Makefile.am:57: 'automake --add-missing' can install 'texinfo.tex' parallel-tests: error: required file 'config/test-driver' not found parallel-tests: 'automake --add-missing' can install 'test-driver' lib/Makefile.am:10: warning: source file '$(top_srcdir)/src/globals.cpp' is in a subdirectory, lib/Makefile.am:10: but option 'subdir-objects' is disabled automake: warning: possible forward-incompatibility. automake: At least a source file is in a subdirectory, but the 'subdir-objects' automake: automake option hasn't been enabled. For now, the corresponding output automake: object file(s) will be placed in the top-level directory. However, automake: this behaviour will change in future Automake versions: they will automake: unconditionally cause object files to be placed in the same subdirectory automake: of the corresponding sources. automake: You are advised to start using 'subdir-objects' option throughout your automake: project, to avoid future incompatibilities. autoreconf: automake failed with exit status: 1
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/blitzpp/blitz/issues/146?email_source=notifications&email_token=AAOVY55FIATN6GTD62LC5XTQKJCWZA5CNFSM4IYAOMSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HMFWU2A, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOVY54HIKZDC5Z33RDVCSLQKJCWZANCNFSM4IYAOMSA .
I found an additional note in the separate INSTALL doc that said to run autoreconf -fiv
, which worked.
You may want to update README to note this or at least link to the INSTALL doc. If I see install instructions in the README and they don't work, I usually stop there, but I just happened to notice the other instructions and tried that. It's confusing that there's two sets of conflicting install instructions.
I agree. We should add instructions telling people to run autoremake if building from the git repo.
On Wed, Sep 18, 2019 at 12:31 PM Chris Spencer notifications@github.com wrote:
I found an additional note in the separate INSTALL doc that said to run autoreconf -fiv, which worked.
You may want to update README to note this or at least link to the INSTALL doc. If I see install instructions in the README and they don't work, I usually stop there, but I just happened to notice the other instructions and tried that. It's confusing that there's two sets of conflicting install instructions.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/blitzpp/blitz/issues/146?email_source=notifications&email_token=AAOVY5Y2OJEYDNU3OB6KRBDQKJJWRA5CNFSM4IYAOMSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7AVILA#issuecomment-532763692, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOVY5ZTFHPXPYGU5I6OWMTQKJJWRANCNFSM4IYAOMSA .
Chris,
Would you be able to submit a pull request adding this information? That would be really hepful!
-- Elizabeth
On Wed, Sep 18, 2019 at 12:33 PM Elizabeth A. Fischer < elizabeth.fischer@columbia.edu> wrote:
I agree. We should add instructions telling people to run autoremake if building from the git repo.
On Wed, Sep 18, 2019 at 12:31 PM Chris Spencer notifications@github.com wrote:
I found an additional note in the separate INSTALL doc that said to run autoreconf -fiv, which worked.
You may want to update README to note this or at least link to the INSTALL doc. If I see install instructions in the README and they don't work, I usually stop there, but I just happened to notice the other instructions and tried that. It's confusing that there's two sets of conflicting install instructions.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/blitzpp/blitz/issues/146?email_source=notifications&email_token=AAOVY5Y2OJEYDNU3OB6KRBDQKJJWRA5CNFSM4IYAOMSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7AVILA#issuecomment-532763692, or mute the thread https://github.com/notifications/unsubscribe-auth/AAOVY5ZTFHPXPYGU5I6OWMTQKJJWRANCNFSM4IYAOMSA .
When building the library from the git repo, the README instructs us to generate the
./configure
script by runningautoreconf
. However, this fails on stock Ubuntu 18.04 with the errors: