Skip to content

Update livesplit-core and rust edition#62

Merged
CryZe merged 5 commits intoLiveSplit:masterfrom
Eein:master
Sep 10, 2025
Merged

Update livesplit-core and rust edition#62
CryZe merged 5 commits intoLiveSplit:masterfrom
Eein:master

Conversation

@Eein
Copy link
Contributor

@Eein Eein commented May 29, 2025

Changes

  • changes required set_custom_variable and set_current_comparison params to Cow<'_, str>
  • updates livesplit-core
  • sets newer rust version and edition in Cargo.toml
  • runs cargo fix --edition

I did not run cargo fmt here because it changes files wildly - if that's okay i'll go ahead and run that too and commit

@AlexKnauth
Copy link
Contributor

Should you update livesplit-core again to incorporate the Layout parser changes?

@Eein
Copy link
Contributor Author

Eein commented Jul 1, 2025

Should you update livesplit-core again to incorporate the Layout parser changes?

Done

Note: I have not tested the layout parser changes and i wont be running for a bit.

Help would be appreciated in making sure things work.

@Eein
Copy link
Contributor Author

Eein commented Aug 12, 2025

probably bitrotting at this point

@CryZe
Copy link
Collaborator

CryZe commented Aug 25, 2025

It doesn't find the OBS shared library on Linux. I don't see what changed there or honestly how it ever even found it to begin with?!

Did the GitHub Runner images just have it installed previously?

@Eein
Copy link
Contributor Author

Eein commented Aug 25, 2025

It doesn't find the OBS shared library on Linux. I don't see what changed there or honestly how it ever even found it to begin with?!

Did the GitHub Runner images just have it installed previously?

Great question!

I'll set up the runners on my fork and see if i can diagnose this when i find some time.

It looks like macos had to do this previously.

https://github.com/LiveSplit/obs-livesplit-one/blob/master/.github/workflows/rust.yml#L134C2-L136C37

It may have included the libraries by default for some reason in an old ubuntu-latest image, and removed to prevent security issues? (maybe when the great XZ Utils purge happened) so just installing the obs libs later might fix this.

@CryZe
Copy link
Collaborator

CryZe commented Aug 25, 2025

We can also just build the fake shared library we have in this repository, which apparently is also just mostly dead code atm?!

@Eein
Copy link
Contributor Author

Eein commented Aug 25, 2025

We can also just build the fake shared library we have in this repository, which apparently is also just mostly dead code atm?!

Happy to take a look and give it a shot in the coming weeks - i'd be curious if this works for macos as well.

@Eein Eein force-pushed the master branch 2 times, most recently from eee0c2b to d0f61db Compare August 25, 2025 22:38
@Eein
Copy link
Contributor Author

Eein commented Aug 26, 2025

Figured I'd drop a ping here as well; seems good to me now!

Only did linux for now, but i can test the osx stuff down the road unless someone else gets to it

@CryZe CryZe merged commit aefc528 into LiveSplit:master Sep 10, 2025
15 checks 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.

3 participants