Skip to content
This repository has been archived by the owner on Nov 6, 2022. It is now read-only.

Makefile: cleanup install, uninstall, clean, all #321

Closed
wants to merge 2 commits into from

Conversation

ploxiln
Copy link
Contributor

@ploxiln ploxiln commented Jun 23, 2016

Lots of little Makefile changes in here. (also, happens to include the changes in #243 and #264)
If any of these changes are not desired, I'm happy to revise with them omitted.


install now works on OS X in addition to Linux ("install -D" not supported on OS X)

also installs the static library

shared and static libraries are not rebuilt if no sources changed
(the files are now "targets" in the Makefile)
(so that "make && sudo make install" does not re-build the libs under sudo)

Make DESTDIR and PREFIX work together. They are usually both specified
to a typical configure / make system. Usually you would do something
like "./configure --prefix=/usr && make DESTDIR=tmp/pkg install".
Before this change, specifying PREFIX would cause DESTDIR to be ignored
(you would have to incorporate your intended DESTDIR into PREFIX,
which is surprising).

small fixups for uninstall and clean

new default "all" target that does not run tests
(run "make test" for previous default behavior)

ploxiln added 2 commits June 22, 2016 23:19
install now works on OS X in addition to Linux
("install -D" not supported on OS X)

also installs the static library

shared and static libraries are not rebuilt if no sources changed
(the files are now "targets" in the Makefile)

Make DESTDIR and PREFIX work together. They are usually both specified
to a typical configure / make system. Usually you would do something
like "./configure --prefix=/usr && make DESTDIR=tmp/pkg install".
Before this change, specifying PREFIX would cause DESTDIR to be ignored
(you would have to incorporate your intended DESTDIR into PREFIX,
which is surprising).

small fixups for uninstall and clean

new default "all" target that does not run tests
(run "make test" for previous default behavior)
since "test" is no longer the default target
@ploxiln
Copy link
Contributor Author

ploxiln commented Jun 15, 2017

this is now out-of-date

@ploxiln ploxiln closed this Jun 15, 2017
@ploxiln ploxiln deleted the make_install_libs branch September 26, 2017 23:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant