Closed Argent77 closed 10 months ago
The problem here is that BASH_FOR is covered by backward-compatibility. The variables are expected to have uppercased content and I can't change that without potentially breaking something else. I'm honestly unsure what to do. You could fix this case by tying the uppercasing into the CL toggle, but I'm not eager to do that, because it's an ugly fix that will potentially cause complications further on. And then there are the other 11 million cases of String.upper/lowercase that WeiDU uses, possibly problematically, I don't know.
The CL switch --case-exact, added by commit 1bdf5bc677d83d178ec195af9d98c5b3eb96f250, doesn't affect variables set by the command
(ACTION|PATCH)_BASH_FOR
. Subsequent commands that are making use of them may fail to function because of it.As a quick & dirty fix I have made this hack. However, since I don't know whether it introduces unwanted side effects, I'm posting it here instead of making a regular pull request.