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

Troubleshooting Windows Builds 2023-11-24 #815

Conversation

stephengtuggy
Copy link
Contributor

@stephengtuggy stephengtuggy commented Nov 24, 2023

Thank you for submitting a pull request and becoming a contributor to the Vega Strike Core Engine.

Please answer the following:

Code Changes:

Issues:
Closes #813
Fixes #781 for 0.9.x and following

Purpose:

  • What is this pull request trying to do? Fix the Windows CI builds. Also reduce the number of CI builds overall.
  • What release is this for? 0.9.x
  • Is there a project or milestone we should apply this to? 0.9.x

@stephengtuggy stephengtuggy added CI/CD Compiler Error Breaks build on at least one supported platform labels Nov 24, 2023
@stephengtuggy stephengtuggy self-assigned this Nov 24, 2023
@stephengtuggy
Copy link
Contributor Author

These 7 CI failures were more or less expected. We don't have a Mantic build image yet. And a 25% success rate for macOS is about as good as we've been able to achieve lately, for whatever reason. (Why 25%? Why not 0% or 100%? This needs further investigation.)

echo "Sorry, Ubuntu impish is no longer supported"
exit 2
;;
"hirsute")
Copy link
Member

Choose a reason for hiding this comment

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

FYI the case functionality can do multi-match via the pipe operator:

case $X in

"value1"|"value2"|"value3")
    ...
    ;;
"value4"|"value5")
   ...
   ;;
...
esac

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So you would like to combine some of these case clauses that essentially do the same thing?

@BenjamenMeyer
Copy link
Member

mantic image should be available; I merged the PR so it's just a matter of that getting through

@stephengtuggy
Copy link
Contributor Author

Okay, so the Rocky Linux and Fedora 38 failures were NOT expected. It looks like I'm running into #781 here, as well.

@stephengtuggy
Copy link
Contributor Author

Sweet! Looks like the latest fix for Issue 781 is working! (I mostly borrowed the solution from a couple of other people on the Web who were running into similar issues.)

@stephengtuggy stephengtuggy marked this pull request as ready for review November 25, 2023 22:23
@stephengtuggy
Copy link
Contributor Author

@royfalk I added you as a reviewer in case you're interested in taking a look. Among other things, I'm cutting way back on the number of CI jobs here. I think it was up to over 60 at one point. Now it's down to 31. A bit more manageable. 😄

@stephengtuggy
Copy link
Contributor Author

Specifically, I derived my fix from nlohmann/json#3895 and https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=64c986b49558a7

Thanks, guys.

Copy link
Member

@BenjamenMeyer BenjamenMeyer left a comment

Choose a reason for hiding this comment

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

I haven't play tested this, but LGTM

@stephengtuggy stephengtuggy merged commit 6b5262e into vegastrike:master Nov 27, 2023
31 checks passed
@stephengtuggy stephengtuggy deleted the troubleshooting_windows_builds_2023-11-24 branch November 27, 2023 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Compiler Error Breaks build on at least one supported platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vcpkg error on Windows: Unrecognized identifer [sic] name xbox Compilation with GCC 13
2 participants