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: add build to .PHONY #1041

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Makefile: add build to .PHONY #1041

merged 1 commit into from
Nov 7, 2023

Conversation

jxy
Copy link
Contributor

@jxy jxy commented Nov 7, 2023

When the build directory exists, the build target needs to be in .PHONY, such that make would consider remaking build.

When the `build` directory exists, the build target needs to be
in .PHONY, such that make would consider remaking `build`.
Copy link

netlify bot commented Nov 7, 2023

👷 Deploy request for elastic-ritchie-8f47f9 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 44e2b39

Copy link
Collaborator

@fare fare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fare fare merged commit 14efb31 into mighty-gerbils:master Nov 7, 2023
4 checks passed
fare pushed a commit that referenced this pull request Nov 8, 2023
When the `build` directory exists, the build target needs to be in
.PHONY, such that make would consider remaking `build`.
drewc pushed a commit to drewc/gerbil that referenced this pull request Nov 29, 2023
When the `build` directory exists, the build target needs to be in
.PHONY, such that make would consider remaking `build`.
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