Skip to content

Adding <cstdint> includes to allow building with gcc13#856

Merged
rob-p merged 7 commits intoCOMBINE-lab:developfrom
A-N-Other:patch-1
Jun 26, 2023
Merged

Adding <cstdint> includes to allow building with gcc13#856
rob-p merged 7 commits intoCOMBINE-lab:developfrom
A-N-Other:patch-1

Conversation

@A-N-Other
Copy link
Copy Markdown
Contributor

As per https://gcc.gnu.org/gcc-13/porting_to.html, gcc13 changes the way it handles includes. These three files needed patching to allow the build to work.

@rob-p rob-p changed the base branch from master to develop June 26, 2023 15:50
@rob-p rob-p merged commit ffb2a11 into COMBINE-lab:develop Jun 26, 2023
@rob-p
Copy link
Copy Markdown
Collaborator

rob-p commented Jun 26, 2023

Thanks @A-N-Other! I changed the base branch to develop, as we generally pull everything through that branch before it makes it to master. Otherwise, these changes look good. Hopefully we'll get around to pushing out a patch release with this change soon.

@A-N-Other
Copy link
Copy Markdown
Contributor Author

Happy to help! Have just been patching for the salmon package inspack, so figured I'd do a PR here as well.

@A-N-Other A-N-Other deleted the patch-1 branch June 26, 2023 15:54
rob-p pushed a commit that referenced this pull request Jun 26, 2023
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