Skip to content
This repository was archived by the owner on Feb 5, 2026. It is now read-only.

Use BSMT OutputCopy for Copying Related Files to Destinations#5

Open
qe201020335 wants to merge 2 commits into
reactive-platform:masterfrom
qe201020335:master
Open

Use BSMT OutputCopy for Copying Related Files to Destinations#5
qe201020335 wants to merge 2 commits into
reactive-platform:masterfrom
qe201020335:master

Conversation

@qe201020335

@qe201020335 qe201020335 commented Oct 28, 2025

Copy link
Copy Markdown

BSMT supports customizing the output files being copied into destination folders. We can use this feature to include all the related files in the artifact without copying files ourselves.

Please ignore the change to the cs file. It is to make the project compilable so I can test CI. The submodule is referencing an outdated commit, which doesn't have the latest API changes. I will drop this commit when we want to merge.

Known issue: the native yoga.dll is not included at the moment because it is not involved in the build process at all. This needs to be fixed before we can use CI to create releases.

I also have some ideas for potential improvements I can add if you want:

  • Include all related files in the files array of the manifest, or create a manifest file for the lib dlls and declare it as a dependency
  • Create a manifest for the yoga.dll and declare it as a dependency after I figure out how to include yoga.dll in the archive
  • Incorporate the changes in the embedded branch

Not sure exactly how we are going to submit this library to BeatMods, so which files are going to be declared by which manifest(s) still needs to be decided.

@qe201020335

Copy link
Copy Markdown
Author

I made this PR a draft so the test commit won't be accidentally merged in

Comment thread BeatSaberSDK/BeatSaberSDK.csproj
@qe201020335 qe201020335 marked this pull request as ready for review October 28, 2025 23:29
@qe201020335

qe201020335 commented Oct 28, 2025

Copy link
Copy Markdown
Author

I have dropped the test commit and disabled the release steps in CI until we have a proper way to include yoga.dll

Comment thread BeatSaberSDK/BeatSaberSDK.csproj
@qe201020335

Copy link
Copy Markdown
Author

Rebased.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants