Support for Amazon Linux

Hal Finkel hfinkel at anl.gov
Tue May 6 14:08:56 PDT 2014


----- Original Message -----
> From: "Rafael EspĂ­ndola" <rafael.espindola at gmail.com>
> To: "Hal Finkel" <hfinkel at anl.gov>
> Cc: "llvm cfe" <cfe-commits at cs.uiuc.edu>, "Dave Johansen" <davejohansen at gmail.com>
> Sent: Tuesday, May 6, 2014 1:24:27 PM
> Subject: Re: Support for Amazon Linux
> 
> > Unfortunately, I don't think that these are currently testable
> > because the sysroot is not respected by DetectDistro:
> >
> >   if (!llvm::MemoryBuffer::getFile("/etc/redhat-release", File)) {
> >     StringRef Data = File.get()->getBuffer();
> >     if (Data.startswith("Fedora release"))
> >       return Fedora;
> >     else if (Data.startswith("Red Hat Enterprise Linux") &&
> >              Data.find("release 6") != StringRef::npos)
> >       return RHEL6;
> > ...
> >
> > We'd need to change that first, no?
> 
> The patch is just updating the list of triple aliases. That is used
> by
> the gcc installation detection logic, which can be tested.
> 
> Things that cannot be tested currently is behaviour like "pass this
> linker option if running on fedora".

Sorry, you're right. I was just confusing two recent patches.

 -Hal

> 
> Cheers,
> Rafael
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory




More information about the cfe-commits mailing list