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

Unable to build 5.0.0 RC1 from GitHub tarball #1088

Closed
chrisburr opened this issue Nov 22, 2019 · 3 comments
Closed

Unable to build 5.0.0 RC1 from GitHub tarball #1088

chrisburr opened this issue Nov 22, 2019 · 3 comments

Comments

@chrisburr
Copy link
Contributor

The GitHub release tarball doesn't contain externals/crc32c which causes the build to fail.

See
conda-forge/xrootd-feedstock#16
and
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=97016

@simonmichal
Copy link
Contributor

@chrisburr : thanks for reporting it, fixed in b1f669c

That said, the GitHub release tarball does not have the submodules checked out and it is stripped from git metadata so I cannot check it out from CMake either. Hence my advice to build from source RPM, which will allow you to build also submodules:
http://xrootd.web.cern.ch/xrootd/sw/releases/5.0.0-rc1/rpms/source/

@chrisburr
Copy link
Contributor Author

I'm not sure if that is the best fix. It's probably better for it to fail loudly and require people to pass -DBUILD_CRC32C=OFF if that's what they want.

I don't know what your release process it like but you can also upload your own source tarballs to GitHub releases which might be a better solution? It's nice having them on GitHub as it makes it easier for generic updater bots to find new versions (I think conda-forge's bot is smart enough to also find it on http://xrootd.web.cern.ch/ but the point remains).

@simonmichal
Copy link
Contributor

@chrisburr : again thanks for your feedback, I've reverted b1f669c and added on option for enabling CRC32C submodule to the cmake infrastructure: -DENABLE_CRC32C=ON/OFF (339452b).

Regarding the release tarballs I can have a look on that, could you please create a separate issue for that (so I don't forget)?

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