Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I noticed that the
make help
target had some ANSI escape sequencesthat didn't seem to do anything:
Those escape sequences at the end mean "change the color to blue, then
change it back to normal".
I've seen this pattern copy-pasted around the web for a while, so I was
curious to find what it was originally. I did some historical
spelunking. The earliest example I can find that closely matches this
branch of history is this:
https://www.thapaliya.com/en/writings/well-documented-makefiles/
In this version, there used to be a
<target>
inside the colorsequences. I can see why that might be helpful. Otherwise, why have the
first line that prints make at all?
Other options:
Delete the escape sequences but leave the printable output the same:
Delete the
make
line and possibly replace the heading withTargets:
:Restore
<target>
andTargets:
as in the original:Do nothing because this is a tiny detail that doesn't matter. 🙂