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

Run make clean without options/environment variables #903

Open
marcindulak opened this issue Nov 4, 2023 · 1 comment
Open

Run make clean without options/environment variables #903

marcindulak opened this issue Nov 4, 2023 · 1 comment
Labels

Comments

@marcindulak
Copy link

Is your feature request related to a problem? Please describe.

From time to time the functionality of the "clean" target changes, and requires options/environment variables to be specified in order to execute "make clean". This requires adjusting automated build scripts.

For example, when trying the current master 45f057e, another option BLAS_SIZE is needed in additon to USE_INTERNALBLAS and USE_MPI.

/usr/bin/make USE_INTERNALBLAS=1 USE_MPI=y clean
***warning MPI_LIB ignored since FORCE_MPI_ENV not set***
***warning MPI_INCLUDE ignored since FORCE_MPI_ENV not set***
You must set
BLAS_SIZE
see https://nwchemgit.github.io/Compiling-NWChem.html#how-to-deal-with-integer-size-of-linear-algebra-libraries
config/makefile.h:4010: *** .  Stop.

Describe the solution you'd like

If it's feasible to generalize the makefile targets: ability to run "make clean" without options.
This may be difficult if the clean targets share the settings with other targets, but maybe those required options can be somehow hard coded for clean targets?

Describe alternatives you've considered

Discover what options "make clean" needs by running it, and add those options.

Additional context

@edoapra edoapra added the bug label Nov 4, 2023
@marcindulak

This comment was marked as off-topic.

edoapra added a commit to edoapra/nwchem that referenced this issue Nov 28, 2023
edoapra added a commit to edoapra/nwchem that referenced this issue Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants