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