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

Revisit mods with NoMerge tag for WB v314+ #91

Open
sibir-ine opened this issue Nov 24, 2021 · 0 comments
Open

Revisit mods with NoMerge tag for WB v314+ #91

sibir-ine opened this issue Nov 24, 2021 · 0 comments
Labels
revisit Revisit at a later date.

Comments

@sibir-ine
Copy link
Member

sibir-ine commented Nov 24, 2021

The behavior of Wrye Bash's NoMerge tag has changed somewhat over the years (see: wrye-bash/wrye-bash#51), & the Wrye Bash team are planning to change it again at some point (see: wrye-bash/wrye-bash#161, currently slated for v314 but subject to change). This means that many mods that are tagged with NoMerge were done so expecting different results.

For instance, when the tag also deactivated the plugin when building a Bashed Patch, contributors may have omitted the Deactivate tag as it would've been redundant at the time. Now, since the plugin is reactivated after building but is still included in the disable dialog, contributors may also add the MustBeActiveIfImported tag if the plugin should remain active to omit it from the dialog & prevent confusion for users. However, in wrye-bash/wrye-bash#161, the WB team are planning to omit plugins tagged with NoMerge from the dialog by default unless they are tagged with Deactivate as well.

For us, this means that when this change is implemented in a release build of WB (whether that be v314 or later), we would benefit from reevaluating all uses of NoMerge currently in any of the masterlists. There are 231 instances across the masterlists at the time of writing with none in games that have light plugin support (i.e., Skyrim SE & Fallout 4) as merging is currently disabled in them. Re-enabling merging for these games is also planned at some point (see: wrye-bash/wrye-bash#596), so this would benefit from reevaluating uses of Deactivate in them instead. There are only 3 at the time of writing.

These instances can be found simply by searching for NoMerge in a given masterlist, but a handful of false positives can be omitted by using the regex [\[\-:] *NoMerge. This regex accounts for tag lists written in either block or flow style, tags in either map or scalar form, & any amount of whitespace preceding the tag. NoMerge can be replaced with Deactivate (or any other tag, for that matter) if needed.

@sibir-ine sibir-ine added the revisit Revisit at a later date. label Nov 24, 2021
@sibir-ine sibir-ine changed the title Revisit mods with NoMerge tag for WB v311+ Revisit mods with NoMerge tag for WB v312+ Jul 27, 2022
@sibir-ine sibir-ine changed the title Revisit mods with NoMerge tag for WB v312+ Revisit mods with __NoMerge__ tag for WB v312+ Jul 27, 2022
@sibir-ine sibir-ine changed the title Revisit mods with __NoMerge__ tag for WB v312+ Revisit mods with NoMerge tag for WB v312+ Jul 27, 2022
@sibir-ine sibir-ine changed the title Revisit mods with NoMerge tag for WB v312+ Revisit mods with NoMerge tag for WB v314+ Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
revisit Revisit at a later date.
Development

No branches or pull requests

1 participant