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

trilinos md5sum changed #125

Closed
tjhei opened this issue Dec 17, 2019 · 5 comments
Closed

trilinos md5sum changed #125

tjhei opened this issue Dec 17, 2019 · 5 comments

Comments

@tjhei
Copy link
Member

tjhei commented Dec 17, 2019

I checked the log in https://github.com/koecher/candi/issues/83 and it looks like something is messed up:
download from https://tjhei.info/candi-mirror/trilinos-release-12-10-1.tar.gz 40f28628b63310f9bd17c26d9ebe32b1
download from https://github.com/trilinos/Trilinos/archive/trilinos-release-12-10-1.tar.gz 667333dbd7c0f031d47d7c5511fd0810
in our package: 40f28628b63310f9bd17c26d9ebe32b1

It looks like they changed their tarballs in some way.

@koecher do you have a suggestion on what to do? (we should bump to a more recent release, but that doesn't fix the older candi release for 9.1)

@koecher
Copy link
Contributor

koecher commented Dec 18, 2019

curious.

I believe it is better if we change the tarball on your server to the current version and update the checksum for the branches dealii-8.5, dealii-9.0, dealii-9.1 and master.

Then, in a second step, we should update the trilinos package (I was asked for this last week in Hannover and hope that there will be a pull request soonly.

@tjhei
Copy link
Member Author

tjhei commented Dec 18, 2019

hope that there will be a pull request soonly.

you mean #92 ? I should update to the latest version...

I believe it is better if we change the tarball on your server to the current version and update the checksum for the branches dealii-8.5, dealii-9.0, dealii-9.1 and master.

The problem is that any current checkout of candi will fail the moment I change the file on the server. We could try changing the filename and keep both files around?

@koecher
Copy link
Contributor

koecher commented Dec 18, 2019

hope that there will be a pull request soonly.

you mean #92 ? I should update to the latest version...

Oh, this was so long ago - I forgot the issues we had there.

I believe it is better if we change the tarball on your server to the current version and update the checksum for the branches dealii-8.5, dealii-9.0, dealii-9.1 and master.

The problem is that any current checkout of candi will fail the moment I change the file on the server. We could try changing the filename and keep both files around?

The problem is, that the natural source has now a new checksum. Maybe we should add the opportunity to give more than one checksum. Then we could keep the old tarball on the server and update the trilinos package file. What do you think on that?

@tjhei
Copy link
Member Author

tjhei commented Dec 18, 2019

something like #127 ?

@tjhei
Copy link
Member Author

tjhei commented Dec 20, 2019

this is now fixed on master, 9.1, 9.0, and 8.5.

@tjhei tjhei closed this as completed Dec 23, 2019
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