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

diffutils: nvhpc not 11 compliant for __builtin_add_overflow #28451

Merged
merged 1 commit into from Mar 16, 2022

Conversation

cessenat
Copy link
Contributor

nvhpc even latest 22.1 fails to compile diffutils 3.8 because it erroneously pretends to be 11 compliant but this is not true.
Paul Eggert from UCLA gave me a patch for latest diffutils master, I adapted it for diffutils 3.8.

@spackbot-app
Copy link

spackbot-app bot commented Jan 17, 2022

Hi @cessenat! I noticed that the following package(s) don't yet have maintainers:

  • diffutils

Are you interested in adopting any of these package(s)? If so, simply add the following to the package class:

    maintainers = ['cessenat']

If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with spack blame:

$ spack blame diffutils

Thank you for your help! Please don't add maintainers without their consent.

You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer.

@cessenat
Copy link
Contributor Author

Hello @eugeneswalker would'nt you like to review that ?
Strange no one met the same problem, I got it with all nvhpc versions and systems I have.

@cessenat
Copy link
Contributor Author

Sorry @alalazo for asking, but would you review this ? I'm puzzled no one met this issue, maybe because it is a core-package.

@cessenat
Copy link
Contributor Author

Hello @sethrj would you agree to review that ? This PR is made so that nvhpc compiles latest difutils.

Copy link
Contributor

@sethrj sethrj left a comment

Choose a reason for hiding this comment

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

Seems reasonable, and it will only affect nvhpc. I wonder if other compilers will hit this too.

@sethrj sethrj merged commit 9b0c2cb into spack:develop Mar 16, 2022
@cessenat
Copy link
Contributor Author

Seems reasonable, and it will only affect nvhpc. I wonder if other compilers will hit this too.

I don't think so. Thanks for review.

@cessenat cessenat deleted the diffutils3.8_nvhpc_patch branch March 16, 2022 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants