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

No rule to make target install #241

Closed
JohanMabille opened this issue Nov 25, 2022 · 4 comments
Closed

No rule to make target install #241

JohanMabille opened this issue Nov 25, 2022 · 4 comments

Comments

@JohanMabille
Copy link

When trying ot install the package as explained in the README, I have the following error:

make: *** No rule to make target 'install'.  Stop.

I try to build and install in a conda environment with the following command:

cmake -DCMAKE_PREFIX_PATH=$CONDA_PREFIX -DCMAKE_INSTALL_PREFIX=$CONDA_PREFIX $SRC_DIR -DCMAKE_INSTALL_LIBDIR=lib

The following fails the same way:

cmake $SRC_DIR

I cannot find a reason that would prevent CPack to generate the install command in the Makefile.

@12AT7
Copy link

12AT7 commented Jan 14, 2023

I have this experience too. I do not know the reason either, but also determined that v2.9 still works, and that there was a CMakeLists.txt update to change the install behavior after that tag. The commit actually looks fine to me, so I really don't know what is the specific problem.

So a temporary workaround is to revert to v2.9.

skrobinson added a commit to skrobinson/argparse that referenced this issue Jan 18, 2023
The option() command expects a help string between the variable and the
initial value.

Closes p-ranav#241

Signed-off-by: Sean Robinson <[email protected]>
@skrobinson
Copy link
Contributor

@JohnGalbraith Thank you for mentioning the break happened after v2.9. That was the hint I needed.

@terroo
Copy link

terroo commented Jan 22, 2023

He did a check on CMakeLists, so he needs to enable this argument for cmake. The command to generate the Makefile should actually be:
cmake -DARGPARSE_BUILD_SAMPLES=on -DARGPARSE_BUILD_TESTS=on -DARGPARSE_INSTALL=on ..

@JohanMabille
Copy link
Author

@terroo thanks!

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

No branches or pull requests

4 participants