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

MMtry.pm: various improvements to VERBOSE behavior #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chrstphrchvz
Copy link
Contributor

Consume stderr from commands if $VERBOSE, otherwise suppress stderr
Always output command line when $VERBOSE regardless of command output, and for try_compile() as well
Make $VERBOSE behavior more consistent with other build tools: command line on one line with no trailing semicolon, command output on following lines (if any), no additional trailing newline.

Use warnings pragma
Avoid shadowing variables

Consume stderr from commands if $VERBOSE, otherwise suppress stderr
Always output command line when $VERBOSE regardless of command
output, and for try_compile() as well
Make $VERBOSE behavior more consistent with other build tools:
command line on one line with no trailing semicolon, command output
on following lines (if any), no additional trailing newline.

Use warnings pragma
Avoid shadowing variables
@chrstphrchvz
Copy link
Contributor Author

chrstphrchvz commented Mar 12, 2021

Pushed a commit to factor out duplicate code into a separate sub _do_try(), assuming that is a good idea.

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.

None yet

1 participant