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

Update package gmsh #199

Merged
merged 1 commit into from Jun 29, 2021
Merged

Update package gmsh #199

merged 1 commit into from Jun 29, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jun 21, 2021

This updates the gmsh version as the gitlab download is used. An updated of the version and the checksum for the tar.gz. download will follow. Either symengine #198 or this has to be merged, afterwards I have to rebase this or #198.

@koecher
Copy link
Contributor

koecher commented Jun 21, 2021

#198 is merged now

@koecher koecher mentioned this pull request Jun 21, 2021
23 tasks
@koecher
Copy link
Contributor

koecher commented Jun 25, 2021

Does it make sense to keep all the old comments and configs from v3 (and v2) for the current v4.8?
Maybe we should split to gmsh-v3.package (with the old config) and gmsh.package for the current version.

@ghost
Copy link
Author

ghost commented Jun 25, 2021

As you prefer ... Creating an extra file is no problem. Do we want to create an extra directory with 'deprecated' package files, similar to the platform files?
The CI in my fork (gmsh+dealii and cmake+hdf5+dealii) were all successful. In particular, it was possible to use the cmake 3.20.4 version, which is the default on the github actions runners, see for example
https://github.com/gfcas/candi/actions/runs/969075570

@ghost
Copy link
Author

ghost commented Jun 25, 2021

Does it make sense to keep all the old comments and configs from v3 (and v2) for the current v4.8?
Maybe we should split to gmsh-v3.package (with the old config) and gmsh.package for the current version.

@tjhei what do you think?

@tjhei
Copy link
Member

tjhei commented Jun 25, 2021

I am not sure I understand what you want to keep. I don't see what is different or worth keeping. Can you elaborate?

Copy link
Member

@tjhei tjhei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Copy link
Contributor

@koecher koecher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A snap. Forgot to submit my review yesterday.
Since the option for slepc:off isn't used, this comment doesn't make sense.

deal.II-toolchain/packages/gmsh.package Show resolved Hide resolved
@koecher koecher merged commit d86dc4c into dealii:master Jun 29, 2021
@koecher
Copy link
Contributor

koecher commented Jun 29, 2021

I merge this here and push an update

@ghost
Copy link
Author

ghost commented Jun 29, 2021

I merge this here and push an update

Thanks @koecher!

@ghost ghost deleted the update_package_gmsh branch June 29, 2021 08:50
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

Successfully merging this pull request may close these issues.

2 participants