Jump to page: 1 2
Thread overview
[Issue 14282] executeShell should use sh and ignore the SHELL env variable
[Issue 14282] executeShell should can sh and ignore the SHELL env variable
Mar 13, 2015
Martin Nowak
Mar 13, 2015
Martin Nowak
Mar 13, 2015
Martin Nowak
Sep 03, 2015
Vladimir Panteleev
Nov 02, 2015
Mark Isaacson
Jul 05, 2017
Vladimir Panteleev
March 13, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

--- Comment #1 from Martin Nowak <code@dawg.eu> ---
Quote from http://pubs.opengroup.org/onlinepubs/9699919799/functions/system.html

> One reviewer suggested that an implementation of system() might want to use an environment variable such as SHELL to determine which command interpreter to use. The supposed implementation would use the default command interpreter if the one specified by the environment variable was not available. This would allow a user, when using an application that prompts for command lines to be processed using system(), to specify a different command interpreter. Such an implementation is discouraged. If the alternate command interpreter did not follow the command line syntax specified in the Shell and Utilities volume of POSIX.1-2008, then changing SHELL would render system() non-conforming. This would affect applications that expected the specified behavior from system(), and since the Shell and Utilities volume of POSIX.1-2008 does not mention that SHELL affects system(), the application would not know that it needed to unset SHELL.

--
March 13, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

Martin Nowak <code@dawg.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|executeShell should can sh  |executeShell should use sh
                   |and ignore the SHELL env    |and ignore the SHELL env
                   |variable                    |variable

--
March 13, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

Martin Nowak <code@dawg.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #2 from Martin Nowak <code@dawg.eu> ---
https://github.com/D-Programming-Language/phobos/pull/3051

--
April 04, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

--- Comment #3 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/a524a3571b18e440c4dd751fcf4e2d00b834fb22 fix Issue 14282 - executeShell should use sh and ignore the SHELL env variable

https://github.com/D-Programming-Language/phobos/commit/865f36fccea8c0d95e31624aea22902def150fa1 Merge pull request #3051 from MartinNowak/fix14282

fix Issue 14282 - executeShell should use sh and ignore the SHELL env variable

--
April 04, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--
August 08, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

--- Comment #4 from github-bugzilla@puremagic.com ---
Commit pushed to stable at https://github.com/D-Programming-Language/dlang.org

https://github.com/D-Programming-Language/dlang.org/commit/e1c7712f52a22f49e86fff604f867d72a1e6964c mention executeShell change (Issue 14282)

--
August 10, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

--- Comment #5 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/D-Programming-Language/dlang.org

https://github.com/D-Programming-Language/dlang.org/commit/e1c7712f52a22f49e86fff604f867d72a1e6964c mention executeShell change (Issue 14282)

--
September 03, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://issues.dlang.org/sh
                   |                            |ow_bug.cgi?id=15000

--
September 03, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

--- Comment #6 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/5b2b1fb59450cfc32ecec3252f543a9f6f34c2a4
Revert "fix Issue 14282 - executeShell should use sh and ignore the SHELL env
variable"

This reverts commit a524a3571b18e440c4dd751fcf4e2d00b834fb22.

--
September 04, 2015
https://issues.dlang.org/show_bug.cgi?id=14282

hsteoh@quickfur.ath.cx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |hsteoh@quickfur.ath.cx
         Resolution|FIXED                       |---

--- Comment #7 from hsteoh@quickfur.ath.cx ---
Fix has been reverted, see https://issues.dlang.org/show_bug.cgi?id=15000

--
« First   ‹ Prev
1 2