Skip to content

Only set env var if not set#14

Open
Anutim wants to merge 1 commit into
UltiNaruto:mainfrom
Anutim:patch-4
Open

Only set env var if not set#14
Anutim wants to merge 1 commit into
UltiNaruto:mainfrom
Anutim:patch-4

Conversation

@Anutim

@Anutim Anutim commented Jun 12, 2026

Copy link
Copy Markdown

To allow the terminal to set it if using nogui, otherwise whatevers in host.yaml gets set, even if it's 'None'. (Which is the default if filepicker got canceled)

Sorry for the PR spam lol.

To allow the terminal to set it if using nogui, otherwise whatevers in host.yaml gets set, even if it's 'None'.
(Which is the default if filepicker got canceled)
@UltiNaruto

UltiNaruto commented Jun 15, 2026

Copy link
Copy Markdown
Owner

it doesn't matter to check if it's set set or not. because the main part is if executable_path:

also it's okay for the PRs, at least I get help ^^

@Anutim

Anutim commented Jun 15, 2026

Copy link
Copy Markdown
Author

it doesn't matter to check if it's set set or not. because the main part is if executable_path:

It does matter, especially along with the other PR I did for Dolphin path/autostart overrides.

Say if you did a batch script to keep your setup portable or w/e, the way environment variable gets set is Batch/cmd -> Archipelago Launcher -> Prime client, so unless the additional check I made is there, you lose that control.
I have tested it to make sure too, and I don't think there are any possible downsides as it only lives during runtime.

Apart from convenience, it does also open up some fun gameplay possibilties, realized the other day that now that we can control the process to look for, we can do splitscreen Archipelago Prime.
All you'd really need is to duplicate your Dolphin executable so it gets a different name, then override the GC save path with a Dolphin launch argument so they don't overwrite eachothers saves.
Would be doable with just a batch script + a gecko code to make the ingame second save slot use controller port 2 etc.

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