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

AllowMultiplePlatforms=1 various cores,frameworks, pre app store prep #2278

Closed
wants to merge 34 commits into from

Conversation

JoeMatt
Copy link
Member

@JoeMatt JoeMatt commented Jan 9, 2024

Set "AllowMultiplePlatforms" to some targets that weren't enabled.

TODO:

  • How does this feature work?
  • Is every target setup correctly for it
  • TVOS Fix these errors;
❌  /Users/runner/work/Provenance/Provenance/Provenance/NewUI/Consoles/ConsoleGamesView.swift:127:22: 'MagnificationGesture' is unavailable in tvOS

            .gesture(MagnificationGesture()
           ~~~~^~~~~~~     ~~~
❌  /Users/runner/work/Provenance/Provenance/Provenance/NewUI/Consoles/ConsoleGamesView.swift:296:33: 'orientation' is unavailable in tvOS

            if UIDevice.current.orientation.isLandscape
              ^
❌  /Users/runner/work/Provenance/Provenance/Provenance/NewUI/Consoles/ConsoleGamesView.swift:296:45: 'isLandscape' is unavailable in tvOS

            if UIDevice.current.orientation.isLandscape
             ^
❌  /Users/runner/work/Provenance/Provenance/Provenance/NewUI/Consoles/ConsoleGamesView.swift:307:33: 'orientation' is unavailable in tvOS

            if UIDevice.current.orientation.isLandscape
               ^
❌  /Users/runner/work/Provenance/Provenance/Provenance/NewUI/Consoles/ConsoleGamesView.swift:307:45: 'isLandscape' is unavailable in tvOS

            if UIDevice.current.orientation.isLandscape
             ^

@JoeMatt JoeMatt added dev implementation, code, development WIP labels Jan 9, 2024
@JoeMatt JoeMatt self-assigned this Jan 9, 2024
@JoeMatt JoeMatt requested a review from jasarien as a code owner January 9, 2024 20:54
@JoeMatt JoeMatt marked this pull request as draft January 9, 2024 21:53
Copy link

Builds for this Pull Request are available at
Provenance.ipa
Provenance-Lite.ipa
ProvenanceTV.ipa
Provenance-UnderDevelopment.ipa
Provenance-XL.ipa
Have a nice day.

@JoeMatt JoeMatt force-pushed the feature/MultiPlatformTargets branch from be8a76f to 2a04293 Compare January 14, 2024 02:36
@JoeMatt JoeMatt force-pushed the feature/MultiPlatformTargets branch from 2a04293 to b45faf6 Compare April 2, 2024 19:49
Copy link

github-actions bot commented Apr 3, 2024

Builds for this Pull Request are available at
Provenance.ipa
Provenance-Lite.ipa
ProvenanceTV.ipa
Provenance-UnderDevelopment.ipa
Provenance-XL.ipa
Have a nice day.

@JoeMatt JoeMatt force-pushed the feature/MultiPlatformTargets branch 2 times, most recently from 7afe60d to bec9bb6 Compare April 9, 2024 05:13
Copy link

github-actions bot commented Apr 9, 2024

Builds for this Pull Request are available at
Provenance.ipa
Provenance-Lite.ipa
ProvenanceTV.ipa
Provenance-UnderDevelopment.ipa
Provenance-XL.ipa
Have a nice day.

@JoeMatt JoeMatt force-pushed the feature/MultiPlatformTargets branch 7 times, most recently from c2581fb to ebaecaa Compare April 28, 2024 16:16
@JoeMatt JoeMatt changed the title [WIP] AllowMultiplePlatforms=1 various cores,frameworks AllowMultiplePlatforms=1 various cores,frameworks, pre app store prep Apr 28, 2024
@JoeMatt
Copy link
Member Author

JoeMatt commented Apr 28, 2024

Just for note, this PR basically also became the Pre App Store quick fixes branch for play and dolphin.

@JoeMatt JoeMatt marked this pull request as ready for review April 28, 2024 17:58
@JoeMatt JoeMatt modified the milestones: v2.3.x / Later…, 2.3.0 Apr 28, 2024
JoeMatt added 27 commits April 29, 2024 02:18
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
better flags for smaller build with less stuff we’re not using

Signed-off-by: Joseph Mattello <[email protected]>
reduced build complexity, less options turned on

Signed-off-by: Joseph Mattello <[email protected]>
Script should work with newer and current slightly older dolphiniOS cmake builds

Signed-off-by: Joseph Mattello <[email protected]>
Signed-off-by: Joseph Mattello <[email protected]>
intel simulator doens’t run because there are flags that are hardwired into the generated cmake xcode project for arm64.

i made a quick and dirty fix to get it to build for arm simulator at least but intel builds will fail (we probably don’t have intel vulkan anyway so whatever)

Signed-off-by: Joseph Mattello <[email protected]>
@JoeMatt JoeMatt force-pushed the feature/MultiPlatformTargets branch from 0fde198 to 8282163 Compare April 29, 2024 06:18
@JoeMatt
Copy link
Member Author

JoeMatt commented May 5, 2024

closed by #2333

@JoeMatt JoeMatt closed this May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev implementation, code, development WIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant