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

Improve makefile #597

Merged

Conversation

jtbx
Copy link
Contributor

@jtbx jtbx commented Feb 11, 2024

Improve makefile by adding an install target and using filename prerequisities over phony targets. Full list of changes from the commit message:

  • add 'install' target
  • shorten the name of INCLUDE_PATHS to IMPORTS
  • use filename prerequisities and targets rather than phony keywords
  • add a dependency on .git/refs/tags for the githash target
  • add githash.txt and dfmt-test to the clean target
  • add more targets to .PHONY

@jtbx jtbx force-pushed the makefile-improvements branch 2 times, most recently from 62ec52f to 68c487d Compare February 12, 2024 06:04
 o  add 'install' target
 o  shorten the name of INCLUDE_PATHS to IMPORTS
 o  use filename prerequisities and targets rather than phony keywords
 o  add a dependency on .git/refs/tags for the githash target
 o  add githash.txt and dfmt-test to the clean target
 o  add more targets to .PHONY
@WebFreak001 WebFreak001 merged commit fa463b4 into dlang-community:master Feb 15, 2024
1 check passed
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