Conversation
aeb12b1 to
f69809f
Compare
|
I also tested on Windows, I wonder if @notimaginative might be able to test on Linux or Mac? |
|
I'll give it a test as soon as I get the chance. Might be a few days though. |
im particulary worried about appimage, it might not be able to see the folder. |
|
Ok since the change was merged in FSO, i changed the minimal FSO version for the custom preferences path support to 24.3.0-20241211, so current nightly should work correctly with the custom FSO preferences path set inside the "kn_portable" folder. |
|
The appimage worked without issue. The Mac app requires code changes since the binary directory and the application directory aren't the same with a Mac bundle. I need to clean up my Mac test code and make a utility function for it (same path finding code is used for updates), then I'll post a diff of the needed modifications. I should also add that for these tests I'm simply creating a |
|
that should be fine. So as long as GetFSODataFolderPath() returns an absolute path, it should be fine. |
|
Should i merge this and you PR the changes later or i should wait? |
|
Here's the diff. Feel free to change that any way you like if there is better method to get the same result. I got stuck trying to debug a crash-on-exit with the Mac build, but that happens on |
Add portable mode, to enable this mode create a "kn_portable" folder in the same folder as were the Knet executable is in.
Then Fso prefereces (if supported) will be at:
\kn_portable\HardLightProductions\FreeSpaceOpen\
Knet data folder
\kn_portable\KnossosNET\
Knossos Library path:
\kn_portable\Library\
In portable mode you cant change the library path.
The FSO preferences path is passed via a env variable, im attaching a FSO build that support this
-removed- use a nightly build newer than 24.3.0-20241211
scp-fs2open/fs2open.github.com#6387
If the FSO version dosent support the changing the preferences path, "-portable_mode" will be passed instead, and settings and pilots will be saved to the working folder.
I only tested it in Windows, this needs more testing, specially on other OS and ill need to update the readme as well.