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

Makefile: remove dependency-tracking and auto-cleaning #45

Merged
merged 1 commit into from
Nov 29, 2020

Conversation

jprjr
Copy link
Contributor

@jprjr jprjr commented Sep 17, 2020

The dependency-tracking winds up creating issues with
wanting to run a "make install" after a "make release" -
since the Makefile was calling "mkdir -p" to auto-create object
and dependency folders, timestamps would (maybe) be updated and cause
a rebuild using default CFLAGS/CXXFLAGS. "maybe" because it differed
based on what operating system was in use.

This simplifies things a bit - a user can just run their own "make
clean" if they need a rebuild with different flags, now object file
timestamps will only be compared against their source file timestamps
and not trigger unnecessary rebuilds. The dependency-tracking for Make
doesn't seem to make a big difference with regard to build times.

This also auto-sets default CFLAGS/CXXFLAGS if not given (a user
running "make demo" with no other flags would trigger this).

The dependency-tracking winds up creating issues with
wanting to run a "make install" after a "make release" -
since the Makefile was calling "mkdir -p" to auto-create object
and dependency folders, timestamps would (maybe) be updated and cause
a rebuild using default CFLAGS/CXXFLAGS. "maybe" because it differed
based on what operating system was in use.

This simplifies things a bit - a user can just run their own "make
clean" if they need a rebuild with different flags, now object file
timestamps will only be compared against their source file timestamps
and not trigger unnecessary rebuilds. The dependency-tracking for Make
doesn't seem to make a big difference with regard to build times.
@bbbradsmith bbbradsmith merged commit fbe6a58 into bbbradsmith:master Nov 29, 2020
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.

2 participants