Skip to content

Pass -unattended to every commandlet invocation - #96

Open
marlonrichert wants to merge 1 commit into
X2CommunityCore:mainfrom
marlonrichert:fix-unattended-commandlets
Open

Pass -unattended to every commandlet invocation#96
marlonrichert wants to merge 1 commit into
X2CommunityCore:mainfrom
marlonrichert:fix-unattended-commandlets

Conversation

@marlonrichert

@marlonrichert marlonrichert commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

_RunMakeBase() already passed -unattended, but _RunMakeMod() and
_RunCookHL() didn't. Without it, the commandlet host might show an
interactive "Scripts are outdated. Would you like to rebuild now?"
dialog that blocks the build until a human clicks "No". -unattended
suppresses this dialog.

Note, though, that we shouldn't pass -unattended to precompileshaders,
because that causes it to crash due to a bug in Firaxis's code. There is
nothing we can do about that on our side.

@marlonrichert
marlonrichert marked this pull request as draft July 14, 2026 12:04
@marlonrichert

Copy link
Copy Markdown
Contributor Author

-unattended appears to make shader precompilation fail. Investigating.

@marlonrichert

Copy link
Copy Markdown
Contributor Author

Found the cause: Passing -unattended to precompileshaders triggers a bug in Firaxis's code that causes it to crash. I'll remove the flag there.

_RunMakeBase() already passed -unattended, but _RunMakeMod() and
_RunCookHL() didn't. Without it, the commandlet host might show an
interactive "Scripts are outdated. Would you like to rebuild now?"
dialog that blocks the build until a human clicks "No". -unattended
suppresses this dialog.

Note, though, that we shouldn't pass -unattended to precompileshaders,
because that causes it to crash due to a bug in Firaxis's code. There is
nothing we can do about that on our side.
@marlonrichert
marlonrichert force-pushed the fix-unattended-commandlets branch from 9b57300 to 7f7b788 Compare July 14, 2026 12:53
@marlonrichert
marlonrichert marked this pull request as ready for review July 14, 2026 12:57
@marlonrichert

Copy link
Copy Markdown
Contributor Author

Fixed and updated the description.

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.

1 participant