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

SH scanning #67

Merged
merged 3 commits into from
Jun 25, 2021
Merged

SH scanning #67

merged 3 commits into from
Jun 25, 2021

Conversation

ParkerTewell
Copy link
Contributor

Also changed all the " to ' so that everything is uniform.

@teuben teuben merged commit 7102d85 into teuben:master Jun 25, 2021
@teuben
Copy link
Owner

teuben commented Jun 25, 2021

i merged, but each check now say something like:

  ccdflip
  man: ['\\fBin', '\\fBout', '\\fBflip']
  bin: ['in', 'out', 'flip']

an easy check is to run "make checkpars" from the $NEMO directory.

@ParkerTewell
Copy link
Contributor Author

ParkerTewell commented Jun 28, 2021 via email

@ParkerTewell
Copy link
Contributor Author

ParkerTewell commented Jun 28, 2021 via email

@teuben
Copy link
Owner

teuben commented Jun 28, 2021 via email

@ParkerTewell
Copy link
Contributor Author

ParkerTewell commented Jun 28, 2021 via email

@ParkerTewell
Copy link
Contributor Author

ParkerTewell commented Jun 28, 2021 via email

@teuben
Copy link
Owner

teuben commented Jun 28, 2021 via email

@teuben
Copy link
Owner

teuben commented Jun 28, 2021

I realize with your example of where you added the -v that make will see it as a flag for make, not for checkpars. The make program can only pass parameters to the targets via a macro assignment, and the Makefile must then actually implement it. Which in this case it didn't. A way to find out what targets there are could be unix grep :-)
grep : Makefile
and the -n flag would say what it would do , but not do it, e.g.
grep -n checkpars
now you will see what it does, there -v was pre-defined by me in the Makefile

A makefile can be surprisingly powerful, but has simple rules.

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