[cfe-dev] CentOS 6 distro detection

[Rafael Espíndola]

This should be refactored so that it looks like

if (RHEL OR CentOS) {
  figure out the version.
}

instead of repeating the RHEL or CentOS check for each version.


On 16 January 2014 01:42, Dmitri Shubin <sbn at tbricks.com> wrote:
> Hi,
>
> Currently CentOS 6 isn't detected as RHEL6 linux distro.
>
> Fix:
>
> Index: lib/Driver/ToolChains.cpp
> ===================================================================
> --- lib/Driver/ToolChains.cpp   (revision 199352)
> +++ lib/Driver/ToolChains.cpp   (working copy)
> @@ -2294,7 +2294,8 @@
>      StringRef Data = File.get()->getBuffer();
>      if (Data.startswith("Fedora release"))
>        return Fedora;
> -    else if (Data.startswith("Red Hat Enterprise Linux") &&
> +    else if ((Data.startswith("Red Hat Enterprise Linux") ||
> +              Data.startswith("CentOS")) &&
>               Data.find("release 6") != StringRef::npos)
>        return RHEL6;
>      else if ((Data.startswith("Red Hat Enterprise Linux") ||
>
> Thanks!
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev