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

feat: changed directory for where data is stored to be fixed (+ some code fixes) #164

Merged
merged 13 commits into from
Jan 12, 2025

Conversation

SleepySpaceBear
Copy link
Contributor

This initially was supposed to be just be changing where the directory for data is stored to be fixed, but "Libary" and a header with a space in it inflicted psychic damage on me.

I'm now working on fixing a crash when trying to deploy my mods in Baldur's Gate 3. It has something to do with 7zip now. Earlier, it was a throw from the missing overlay directory for the app data stuff (see b5c862e for the fix).

@SleepySpaceBear SleepySpaceBear changed the title Code clean up + changed directory for where data is stored to be fixed feat: changed directory for where data is stored to be fixed Dec 31, 2024
@SleepySpaceBear SleepySpaceBear changed the title feat: changed directory for where data is stored to be fixed feat: changed directory for where data is stored to be fixed (+ some code fixes) Dec 31, 2024
@SleepySpaceBear
Copy link
Contributor Author

I think I have it where I want to be with respect to some code fixes + the change to the working directory. I'll have another PR branching off from this with some changes to how the bit7zip dependency is set up.

@SleepySpaceBear SleepySpaceBear marked this pull request as ready for review December 31, 2024 14:16

//((std::string)std::getenv("HOME")) + "/.lamp/
const lampString baseDataPath = "Lamp_Data/";
const lampString baseDataPath = lampString(std::getenv("HOME")) + "/.lampray/";
Copy link
Contributor

Choose a reason for hiding this comment

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

For reference, the placement of files was @CHollingworth's preference (see ##92). Assuming he considers this PR, you may have some difficulties getting it merged with this change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At this point, with how inactive this repo is, I might just focus on maintaining my fork

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The Baldur's Gate 3 implementation was broken, and I'm working on fixing it. There's a lot of other weird stuff with its current state as well that I've been working on fixing

Copy link
Contributor

Choose a reason for hiding this comment

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

That is fair. If you have any interest in my changes on #147 feel free to either steal them, or let me know what you might want and I can try to get it rebased or altered for you to pull in.

I don't think I have made much of any other changes that you would really want or need.

Copy link
Contributor Author

@SleepySpaceBear SleepySpaceBear Jan 1, 2025

Choose a reason for hiding this comment

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

Disclaimer that I'm using arch and haven't tested on other systems.

I was able to fix the issues with using bit7z by just setting up my fork to track the latest tagged version of bit7z since there seems to have been breaking changes with 7zip at some point between bit7z 4.0.0-rc and the latest version. Of course .unrar support would be nice.

Right now I got the overlay filesystem implementation working with Baldur's Gate. I need to figure out why the mods aren't being installed properly though. I think I'm missing the overlay set up for the player profile stuff.

I want to also implement a checker for whether the current mod setting is deployed rather than just always having the deployed button. I think also having it configurable as to whether overlayfs is used or just overwriting the game files would be nice. I do have to fix some stuff with respect to overlayfs since it's very easy for it to get into a state where I have to manually it.

@SleepySpaceBear
Copy link
Contributor Author

Going to change it to use the XDG spec for the default directory and make it using the PWD as an option to be set in cmake

@SleepySpaceBear
Copy link
Contributor Author

@CHollingworth This should be ready for merging now

@CappedMonke
Copy link

On a fresh build the applicaion crashes on startup with the following error:

[LAMP:LOG:0] Loading LAMP CONFIG:LanguagePath
[LAMP:WARNING:9] Unable to load LAMP CONFIG:LanguagePath
[LAMP:LOG:0] File not found:

@CHollingworth
Copy link
Owner

Ill add it to the test 2.0.0

@CHollingworth CHollingworth merged commit 29d9296 into CHollingworth:master Jan 12, 2025
2 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.

4 participants