On Sun, Jun 28, 2015 at 1:22 PM, Robert Goldman rpgoldman@sift.net wrote:
If my claim is correct, then an alternative patch is to:
- Supply :FORCE-SHELL T in G-F-R-P
- Change %normalize-system-command to explicitly supply "CMD /C" on
Windows + ACL.
I think that might be more correct than making ACL always explicitly use "CMD /C", since always supply CMD.EXE would make it impossible for ACL + Windows to ever provide the simpler behavior, right?
<sigh>I should have put this in a launchpad ticket</sigh>
Yes, I believe the reason I didn't add the cmd /c is because I wanted to expose the ability to not use cmd /c, at which point we probably should indeed somehow arrange for it to be explicitly used in this case. — then test case, not the code, is wrong (though the code is still wrong for the :force-shell t case if that case fails to add cmd /c without our asking — I didn't check).
PS: note that nothing is guaranteed to exist in /bin except /bin/sh itself. Indeed, nothing else exists on NixOS.
—♯ƒ • François-René ÐVB Rideau •Reflection&Cybernethics• http://fare.tunes.org — Question authority! — Yeah, says who?