Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix and improve build system #26

Merged
merged 9 commits into from
Jan 5, 2016
Merged

Fix and improve build system #26

merged 9 commits into from
Jan 5, 2016

Conversation

hartwork
Copy link
Contributor

@hartwork hartwork commented Jan 4, 2016

This pull request fixes #25 but also resolves need to keep the many copied/generated files in Git. With these changes applied, I can run

# git clean -f -d
# git clean -f -d -X
./autogen.sh && make -j5 distcheck

to bootstrap from nothing to a final release tarball.
(I have commented the "git clean" commands out, because I want you to double check if those commands fit your environment before running them so you don't lose files by accident.)

If have tried my best to make these commits explain themselves and be small distinct units to ease review. If you have questions or want me to make further adjustments, just let me know.

Best, Sebastian

libtoolize: Consider adding 'AC_CONFIG_MACRO_DIRS([m4])' to configure.ac,
libtoolize: and rerunning libtoolize and aclocal.
configure.ac:5: warning: AM_INIT_AUTOMAKE: two- and three-arguments forms are deprecated.  For more info, see:
configure.ac:5: http://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation
$ ./autogen.sh && make -j5 distcheck
[..]
./makestub.pl *.xml > *.c
/bin/sh: line 0: test: too many arguments
Usage: ./makestub.pl <xml file> at ./makestub.pl line 8.
Makefile:1807: recipe for target '*.c' failed
make[1]: *** [*.c] Error 255
make[1]: Leaving directory '/some/path'
Makefile:1581: recipe for target 'dist' failed
make: *** [dist] Error 2
[..]
swh added a commit that referenced this pull request Jan 5, 2016
@swh swh merged commit 4482991 into swh:master Jan 5, 2016
@swh
Copy link
Owner

swh commented Jan 5, 2016

Excellent, thanks. It's been so many years since I used auto* that I've no idea what the best practices are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Script "mkinstalldirs" used by "make install" but missing?
3 participants