Skip to content

Use absolute path when starting SNI#170

Merged
Berserker66 merged 1 commit intoArchipelagoMW:mainfrom
N00byKing:patch-1
Dec 27, 2021
Merged

Use absolute path when starting SNI#170
Berserker66 merged 1 commit intoArchipelagoMW:mainfrom
N00byKing:patch-1

Conversation

@N00byKing
Copy link
Collaborator

Causes reliability issues when relative path is used, see subprocess doc

In my case, it did not start SNI at all.

Causes reliability issues when relative path is used.
@Berserker66
Copy link
Member

It'd like to know "your case", such as system and method of starting the client (SNIClient.py vs. python3 SNIClient.py for example). The change is fine, so I'm accepting it, but would like to get that info regardless.
Thank you!

@Berserker66 Berserker66 merged commit 61310c5 into ArchipelagoMW:main Dec 27, 2021
@N00byKing
Copy link
Collaborator Author

That was fast, thanks for the merge! Arch Linux System, starting with python SNIClient.py

@N00byKing N00byKing deleted the patch-1 branch December 27, 2021 14:43
Salzkorn pushed a commit to Salzkorn/Archipelago that referenced this pull request Feb 19, 2024
Fix Spear of Adun & Kerrigan Primal Form exclusion options
Eijebong pushed a commit to Eijebong/Archipelago that referenced this pull request May 29, 2025
* Mount Mortar B1F item label shuffling

Changed item 1 -> 5
2 -> 1
3 -> 2
4 -> 3
5 -> 4

* Mt. Mortar has even more weird location numbers

* Update Sprout Tower 3F items

Item 2 is in the south,
Item 1 is in the north
You enter from the south
Changed it to Item South and Item North

* All the North/west/east/south Item location changed (+ some misc)

So many renamed locations....
There is a chance some weast incidents happened

* Extra e slipped in somewhere

* Weast incident #1

Found because Im also looking through rules.py

* Updated Rules.py with the location updates

* Fix duplicate location names

---------

Co-authored-by: James White <[email protected]>
mattman107 pushed a commit to mattman107/SoHArchipelago that referenced this pull request Oct 16, 2025
Fix a couple item groups and Epona item
carrotinator added a commit to DayKat/spirit-tracks that referenced this pull request Feb 13, 2026
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