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

Utilize BepInEx NuGet repository for game libraries. #18

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

Higgs1
Copy link

@Higgs1 Higgs1 commented Apr 15, 2022

We can utilize nuget and the restore target to automatically download RoR2 and BepInEx's libraries without having to download or install (or update) the actual game.

Please be sure to test this first! It works swell on my fork, but it differs considerably now and also I don't Windows. On *nix I use the msbuild -r command to restore and build the whole project.

BepInEx's repo does not distribute R2API or other plugins, but I might get to that later / in another mod.

@Vl4dimyr
Copy link
Owner

I will look into this.

@Higgs1
Copy link
Author

Higgs1 commented Apr 19, 2022

I updated the PR to fetch R2API from NuGet as well, which might help with the newest RoR2 patch.

I'm still weighing options for automagically referencing the two mod dependencies, MMHOOK and Roo, but using r2modman works for everyone on my side just fine.

`NoWarn="NU1701"` is to suppress the warning from `MMHOOK.RoR2`, which is only available for .NetFramework, and not .NetStandard - but that's okays here.

Since Roo is now the only unmanaged mod dependency now, the search path could be moved back into Roo's `%(HintPath)` instead of fiddling with AssemblySearchPaths if you wanted, but I prefer AssemblySearchPaths.
@Higgs1
Copy link
Author

Higgs1 commented Apr 20, 2022

Updated the PR to fetch MMHOOK from NuGet too, yay!

Git did some crazy stuff to the diff though, looks pretty hard to read IMO. Here's a quick linky to the whole CaptainShotgunModes.csproj

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