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
[FeatureRequest] :LAST[mod] pass that runs right before :FINAL #96
Comments
I don't mind taking a shot at it myself if you like the idea but don't want to write it yourself |
What about |
Yeah that works fine for me if it's easier to implement 👍 |
As long as we don't start a race to the FINALer modifier I am fine with it. |
I only proposed it because people are already running at the most Zful mod :D if you feel like this would not add anything to the mod I'm fine with not getting this feature |
Now that I think about this a bit it would probably make sense to name it something else to avoid confusion. Maybe |
my initial idea was |
Yes, LAST may lower the risk of confusion |
in your idea, it would just be a simple alphabetical ordering right? |
All patches marked |
Oh, one more thing, should |
Addresses #96 Not to be confused with the popular password manager
no I think it should be analogue to :FOR so that people can use it instead of :FOR[zzzzzzwhatever] but it shouldn't add a mod definition. so if I do :LAST[SigmaBinary] the patch will apply regardless of SigmaBinary being installed, but if there is no :FOR[SigmaBinary] or no "SigmaBinary.dll" other patches using ":NEEDS[SigmaBinary]" will still fail |
Hi guys, I've been out of the loop for a while, is there any news on this? if you need help testing/coding feel free to let me know |
I did some work on it, then backed up to do some refactoring that would make it simpler. I can probably go back and re-work that now that the refactoring is done. |
This is done |
Thanks! |
I was looking for documentation on this feature and this exchange is sufficient, but took some digging to find; mostly because "last" is such a poor search term. The patch ordering wiki page acknowledges the LAST pass but doesn't describe that it is analogous to the FOR pass. I'd be happy to write in an update for the page, but don't want to speak imprecisely about code I didn't write if the convention is that contributors write the doc. |
Per the original feature design (#96) it should still run
Per the original feature design (sarbian#96) it should still run
I was going to ask for a new pass specifier that ran just before
:FINAL
but then I thought, would it be possible to allow both a normal pass specifier and
:FINAL
to be used?with the effect that:
:FINAL
are run following standard rules:FINAL
are run alphabetically:FINAL
are runthis will not change how current patches are applied, but open up a second pass of patches.
this would remove the need of using
:FOR[zzzzzzMyMod]
and should be pretty easy to implement into the current codeThe text was updated successfully, but these errors were encountered: