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.

Sign up for GitHub

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

Jump to bottom

Fix ogre2 include #179

Closed
wants to merge 2 commits into from
Closed

Fix ogre2 include #179

wants to merge 2 commits into from

Conversation

mjcarroll
Copy link
Contributor

Alternative to #178, tested on win-beefy CI machine against OGRE 2.1.

@mjcarroll
Fix OGRE2 include paths
d22b6b0 
Signed-off-by: Michael Carroll <[email protected]>
@mjcarroll
Fix missing braces
1246e18 
Signed-off-by: Michael Carroll <[email protected]>
@mjcarroll mjcarroll self-assigned this Aug 19, 2021
@github-actions github-actions bot added 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome labels Aug 19, 2021
@mjcarroll
Copy link
Contributor Author

I'm going to lump this with the re-revert

@mjcarroll mjcarroll closed this Aug 19, 2021
@mjcarroll mjcarroll deleted the fix_ogre2_include branch August 19, 2021 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adlarkin adlarkin Awaiting requested review from adlarkin

@j-rivero j-rivero Awaiting requested review from j-rivero j-rivero is a code owner

@mxgrey mxgrey Awaiting requested review from mxgrey

Assignees

@mjcarroll mjcarroll

Labels
🏰 citadel Ignition Citadel 🔮 dome Ignition Dome 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mjcarroll