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

add missed #include <cstdint> statement #1276

Merged

Conversation

waebbl
Copy link
Contributor

@waebbl waebbl commented Sep 15, 2022

While trying to apply the patch from PR #1264, and testing different builds for my Gentoo package, I noticed there's another include statement for cstdint missing.

Eventually this slipped through your checks / CI because currently you're not using gcc-13 for any of your test builds, as far as I can see.

Signed-off-by: Bernd Waibel waebbl-gentoo@posteo.net

Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
waebbl added a commit to waebbl/gentoo that referenced this pull request Sep 15, 2022
Additional patch has been provided to upstream at
AcademySoftwareFoundation/openexr#1276

Closes: https://bugs.gentoo.org/868693
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Sep 15, 2022
Additional patch has been provided to upstream at
AcademySoftwareFoundation/openexr#1276

Closes: https://bugs.gentoo.org/868693
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
Closes: #27169
Signed-off-by: Sam James <sam@gentoo.org>
Copy link
Contributor

@meshula meshula left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

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

LGTM

@cary-ilm cary-ilm merged commit cc508f7 into AcademySoftwareFoundation:main Sep 21, 2022
@waebbl waebbl deleted the additional-gcc-13-fix branch September 23, 2022 10:32
cary-ilm pushed a commit to cary-ilm/openexr that referenced this pull request Mar 3, 2023
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>

Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
cary-ilm pushed a commit that referenced this pull request Mar 5, 2023
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>

Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
@cary-ilm cary-ilm added the v3.1.6 label Jul 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants