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

Two fixes for -o behavior #287

Closed
wants to merge 2 commits into from
Closed

Conversation

BryanAldrich
Copy link

setting the current directory when using -o causes issues loading templates (which read from current (skyrim data)).

create the missing output directory if it does not already exist.

setting the current directory when using -o causes issues loading templates (which read from current (skyrim data)).
@BryanAldrich
Copy link
Author

my intention for this is to have pandora installed outside of everything, -o point to an empty folder (that gets packaged into a mod of its own).

What I don't know is what to do if there are templates in both the launch folder + the skyrim\data folder that are pointing to the same template. e.g. one overwrites/updates part of a template that is in the launch folder. Who wins?

@Monitor221hz
Copy link
Owner

Why would you ever have template folders in both the launch and the skyrim\data folder? The template folders should be unseparable from the base engine as it requires those to launch.

@BryanAldrich
Copy link
Author

I mean from mods that alter the templates.

Like Archery Gameplay Overhaul. It has a Nemesis patch which puts files in the Nemesis_Engine folder.

https://www.nexusmods.com/skyrimspecialedition/mods/24296?tab=files

I'm also thinking of the "possible, but does it happen?" cases where someone posts mods to your templates to either acomodate their other mods or to change behavior. (khajit that walk on their toes vs feet).

@Monitor221hz
Copy link
Owner

There was a migration to Avalonia so this PR could not be carried over; however, the changes were recreated; see here: bb0238d

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