-
Notifications
You must be signed in to change notification settings - Fork 472
Makefile: Use order-only deps for directories #2728
Conversation
|
@glebm if i knew how to use git/github properly, and could figure out how to pull this for a local test, i could probably tell you that :/. (i'm a Fossil SCM guy, not a git guy :/) i have no personal experience with order-only prerequisites, so i can't say with certainty whether it resolves the problem. |
|
@sgbeal Like this: git clone https://github.com/glebm/libsass.git && cd libsass && git checkout patch-2 |
That's after doing a fresh clone. |
|
OK, try |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally someone found the real solution 👍 if I only knew about order-only deps, was looking for this up and down the documentation for make 😃 also tested it quickly on my windows machine and seems to work.
|
P.s. |
|
@glebm Unfortunately: @mgreter that does the trick. This is exactly why i'm not a git guy - even the most banal operations are cryptic :/. So, after building... (noting that Success! |
|
Thank you all for seeing this through ❤️ |
See https://www.gnu.org/software/make/manual/html_node/Prerequisite-Types.html
With this PR the targets only depend on directories' existence, but not on their timestamps.
Fixes #2727, fixes #1992, fixes #1433.