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

Fix build with GNAT 2020 #72

Closed
wants to merge 1 commit into from

Conversation

mosteo
Copy link

@mosteo mosteo commented Mar 24, 2021

This is a minimal untested fix. I do not know how to easily test this part of the library.

@mosteo mosteo marked this pull request as ready for review March 24, 2021 12:06
@mosteo
Copy link
Author

mosteo commented Mar 24, 2021

Nevermind, this is a duplicate of a queued PR with a better fix.

@mosteo mosteo closed this Mar 24, 2021
@Lucretia
Copy link
Collaborator

Lucretia commented Mar 24, 2021

So, there was nothing wrong with this code? It was a compiler bug?

@mosteo
Copy link
Author

mosteo commented Mar 24, 2021

Sorry, I was unclear. The queued PR #62 addresses the same problem. I don't know which of the gnats is right here, but the problem still exists with current CE2020.

@Lucretia
Copy link
Collaborator

Ok, thanks for the clarification, may as well keep this open just in case.

@Lucretia Lucretia reopened this Mar 24, 2021
@anisimkov
Copy link
Contributor

anisimkov commented Aug 20, 2021

the patch #62 (and the same #75) are better fixing the same issue on GNAT Community 2020 and 2021

@Lucretia
Copy link
Collaborator

Fixed with Commit 45731de

@Lucretia Lucretia closed this Dec 31, 2021
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.

3 participants