Skip to content

Fix installation for experimental appId 1890870#49

Merged
BrettMayson merged 4 commits intoacemod:mainfrom
hafometh88:experimental
Aug 18, 2025
Merged

Fix installation for experimental appId 1890870#49
BrettMayson merged 4 commits intoacemod:mainfrom
hafometh88:experimental

Conversation

@hafometh88
Copy link
Contributor

  • Added a known workround (thanks ceo_of_bacon) that ensures the experimental app is installed sucessfully

- Added a known workround (thanks ceo_of_bacon) that ensures the experimental app is installed sucessfully
- Introduced a sentinel file ensuring the platform switch only runs once to avoid unnecessary redownloads on restarts.
Copy link
Member

@BrettMayson BrettMayson left a comment

Choose a reason for hiding this comment

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

Looks fine to me, but I have not tested it

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a workaround for installation issues with experimental Steam app ID 1890870 by performing a two-phase installation process that first installs using Windows platform type, then Linux platform type.

  • Adds special handling logic for experimental app ID 1890870 with a two-phase installation approach
  • Implements a sentinel file mechanism to track when the Windows fix has been applied
  • Preserves standard installation behavior for all other app IDs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@hafometh88
Copy link
Contributor Author

thanks, I am currently using it and it works

@BrettMayson BrettMayson merged commit d034649 into acemod:main Aug 18, 2025
1 check passed
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