-
Notifications
You must be signed in to change notification settings - Fork 284
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
Dependency on git for building from source not fit for purpose. - Please fix the release archives/autogen script. #7254
Comments
Which tarball are you downloading? The github autogenerated archives are not official releases and should not be used. Unfortunately I have not found a way to disable them from being shown to users. Check the official release tarballs uploaded to release pages: https://github.com/pmodels/mpich/releases/tag/v4.3.0rc2 |
For the record: official GH discussion about the issue with these automatic tarballs and the confusion they cause users: https://github.com/orgs/community/discussions/6003. |
I clicked on releases and picked the source-code.tar.gz from the assets - it is the only file that indicates that it is source code. Edit: I just downloaded mpich-4.3.0rc2.tar.gz from your link - oh, it is also source code. It may be clearer to explicitly include And thank you for the linked issue. Indeed, it causes confusion and frustration. |
MPICH source releases are available on our website https://www.mpich.org/downloads/. Perhaps we can update the links to show the filename rather than |
I think its less the type but the content. Looking at it again it now makes sense the first is actually the source code... Basically, the labelling is unambiguous. - So adding some kind of reference to it being source code in the name or label of the file would help. -> Now I start reading the text, it even says "source code below" at the end. But its too small and hidden. When we build different source code, we tend to just grab the archive and check the readme for instructions (or not, because we built older versions n times) and run with it. |
I'm sure the developers know this autogen.sh-error well..
This is the default state when downloading a release from github...
No, installing from the repos is not a good solution, and no, using git clone is not suitable either.
Please fix the release instead of providing a broken download.
For the reasons why:
In the case of research, it can be relevant to refer to specific versions of software. Of course one can hunt down a specific git tag, but archiving the tar.gz of a release for reference is a lot easier.
Installing whatever random (maybe working, maybe not) version is supplied with the operating system is thus also out of the question. Plus, different software may need different versions.
This brings me to the use-case where this is really needed: HPC. In this case, software is provided from a central mount that all nodes read. Over time, one also accumulates different versions of these libraries (And no, the old ones cannot be replaced, can you guarantee old codes and software will work on the new version?) which are loaded as per user requirements using a modules environment such as lmod.
So we now come back to requiring version numbers, specific releases - and ideally, also would like to archive a version for future reference (if it needs to be rebuilt for example) - and again, few things are easier than archiving a release.
So please can you work towards the source code release provided on github in the releases being code that can be successfully configured & compiled?
...and rant over... - Its just that I've just now run into this (possibly again) and am feeling a bit irritated by this error.
Hopefully it still incites discussions and changes?
(And yes, using git clone works just fine for local testing, but it really isn't suitable for a central hpc provision as per the points mentioned above.)
Edit: And as an edit, there is no configure-script in the release archive, which means the instructions to build from the readme cannot be applied... The configure-script is typically generated by the autogen.sh script...
The text was updated successfully, but these errors were encountered: