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

Allow rpaks to use starpaks that are in the mod's folder #279

Merged
merged 17 commits into from
Oct 22, 2022

Conversation

ASpoonPlaysGames
Copy link
Contributor

Before the starpaks had to be inside the r2 directory somewhere, this PR adds a field to the rpak.json that registers starpak paths as being relative to the mod/paks directory instead.

I'm making this PR now, however I am going to try and read the starpak paths from the rpak header instead of using the json here, so yeah

@ASpoonPlaysGames
Copy link
Contributor Author

Did the reading from the rpak file thing, as far as i'm concerned this is complete and ready

@IJARika
Copy link

IJARika commented Sep 29, 2022

works as intended, tested myself. see clip for example

https://streamable.com/ug1kdw

NorthstarDLL/rpakfilesystem.cpp Outdated Show resolved Hide resolved
NorthstarDLL/modmanager.cpp Outdated Show resolved Hide resolved
@pg9182 pg9182 self-requested a review October 6, 2022 14:07
@pg9182
Copy link
Member

pg9182 commented Oct 6, 2022

docker: ghcr.io/pg9182/northstar-dedicated-ci:1-tf2.0.11.0-ns1.9.7-ci.59de24a5.12.3197971268-nslauncher.6e71dcdc.3197262890

Copy link
Member

@pg9182 pg9182 left a comment

Choose a reason for hiding this comment

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

LGTM. Code is okay, and it doesn't break in wine.

NorthstarDLL/rpakfilesystem.cpp Outdated Show resolved Hide resolved
NorthstarDLL/rpakfilesystem.cpp Outdated Show resolved Hide resolved
NorthstarDLL/rpakfilesystem.cpp Show resolved Hide resolved
NorthstarDLL/rpakfilesystem.cpp Show resolved Hide resolved
NorthstarDLL/rpakfilesystem.cpp Outdated Show resolved Hide resolved
@BobTheBob9 BobTheBob9 merged commit 5a0f03a into R2Northstar:main Oct 22, 2022
@ASpoonPlaysGames ASpoonPlaysGames deleted the starpak-paths branch January 4, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants