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

Use GNU make's wildcard expansion instead of the shell's, because the… #532

Closed
wants to merge 1 commit into from

Conversation

monniaux
Copy link
Contributor

… shell will leave a export/.glob item in there if there are no export/.glob files.

The Makefile only works with gmake anyway.
It would be possible to use option nullglob in bash, but many systems now use dash.

… shell will leave a export/*.glob item in there if there are no export/*.glob files.

The Makefile only works with gmake anyway.
It would be possible to use option nullglob in bash, but many systems now use dash.
@monniaux
Copy link
Contributor Author

And this script also works with MacOS Make.

@xavierleroy
Copy link
Contributor

In general I like the use of make functions instead of shell scripts, e.g. $(foreach ...) instead of for. The problem with $(wildcard ...) in commands is that it can generate very long command lines that exceed Windows' limits. We ran into this problem with the rm commands in make clean; that's why they still use shell globbing.

I'm going to push a different fix for issue #533.

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