r223102 - Rely on fewer features of the 'env' command. Darwin only supports '-i'.
chandlerc at gmail.com
Tue Dec 2 20:16:28 PST 2014
On Tue, Dec 2, 2014 at 5:06 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> ----- Original Message -----
> > From: "Chandler Carruth" <chandlerc at gmail.com>
> > To: cfe-commits at cs.uiuc.edu
> > Sent: Monday, December 1, 2014 7:24:52 PM
> > Subject: r223102 - Rely on fewer features of the 'env' command. Darwin
> only supports '-i'.
> > Author: chandlerc
> > Date: Mon Dec 1 19:24:52 2014
> > New Revision: 223102
> > URL: http://llvm.org/viewvc/llvm-project?rev=223102&view=rev
> > Log:
> > Rely on fewer features of the 'env' command. Darwin only supports
> > '-i'.
> > I'm explicitly setting LC_ALL=C somewhat for documentation, but
> > hopefully this also removes some host variation from the test
> > results.
> FWIW, this test is failing on my system because the env command is
> removing the LD_LIBRARY_PATH value necessary for the just-built Clang to
> correctly function. Any ideas?
... not really, no. =[ Ugh. Any ideas? I can just nuke this test, but it
caught real bugs in our use of std::getenv. I have no idea how to
effectively manipulate the environment here.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits