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

configure fails: set_target_properties called with incorrect number of arguments. #34

Closed
yurivict opened this issue Nov 4, 2018 · 6 comments

Comments

@yurivict
Copy link

yurivict commented Nov 4, 2018

CMake Error at src/CMakeLists.txt:40 (set_target_properties):
  set_target_properties called with incorrect number of arguments.

This is the offending instruction:

set_target_properties(libdavix PROPERTIES
                            VERSION ${VERSION_MAJOR}.${VERSION_MINOR}.${VERSION_PATCH}
                            SOVERSION ${VERSION_MAJOR}
                            CLEAN_DIRECT_OUTPUT 1
                            OUTPUT_NAME ${OUTPUT_NAME_DAVIX})

Regression in version 0.7.1
FreeBSD 11.2 amd64
cmake-3.12.3

@gbitzes
Copy link
Contributor

gbitzes commented Nov 4, 2018

Should be the same issue as #30 - the github tarballs are not recommended, run packaging/make-dist.sh to make a good tarball. You'll find the result in build.

By default, Github simply bundles the source code in the automatically generated tarballs.. I'll see if I can fix that in future releases, but for now simply run packaging/make-dist.sh.

@gbitzes
Copy link
Contributor

gbitzes commented Nov 4, 2018

Ok, I figured it out: Try downloading davix-0.7.1.tar.gz from https://github.com/cern-fts/davix/releases, this should work.

@yurivict
Copy link
Author

yurivict commented Nov 4, 2018

I use the tarball http://grid-deployment.web.cern.ch/grid-deployment/dms/lcgutil/tar/davix/0.7.1/davix-0.7.1.tar.gz. It's not from github.

@gbitzes
Copy link
Contributor

gbitzes commented Nov 4, 2018

Something was broken in the release scripts, that one is not good either, sorry. Only the one I uploaded on github just now should work.

@gbitzes
Copy link
Contributor

gbitzes commented Nov 4, 2018

The tarball in the link has been updated, feel free to download again from there.

@yurivict
Copy link
Author

yurivict commented Nov 4, 2018

This works, thanks for fixing it!

@yurivict yurivict closed this as completed Nov 4, 2018
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

2 participants