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 duplicate definition with GCC 10 #2085

Closed
wants to merge 1 commit into from

Conversation

remicollet
Copy link
Contributor

@sergeyklay
Copy link
Contributor

sergeyklay commented Apr 27, 2020

@remicollet There are failed Linux tests due to changed template. Do not consider macOS ones (they are almost all broken and I turn them off today).

P.S. We use black box tests (in addition to Unit ones) to prove Zephir generates expected C-code and handles console commands. For more see https://github.com/phalcon/zephir/blob/development/unit-tests/sharness/README.md

@remicollet
Copy link
Contributor Author

Sorry, but I have no idea what you are talking about.

Please see the linked issue.
And provide a fix.

@sergeyklay
Copy link
Contributor

Fixed in #2087. Thank you for the patch.

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