r223102 - Rely on fewer features of the 'env' command. Darwin only supports '-i'.

[Chandler Carruth]

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...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141202/0871a240/attachment.html>