There is nothing wrong with the patch. =] It looks fine.

However, a test case using sysroot and a fake opensuse tree in the
Driver tests would be really nice to get before we keep adding new
versions....

Also, I wonder if you might rather make this code a bit more smart and
able to parse out any particular version number in the format open
suse uses? Then we wouldn't have to update this for each point
release.

On Wed, Nov 7, 2012 at 4:51 AM, İsmail Dönmez <[email protected]> wrote:
> Hi;
>
> A simple patch attached.
>
> Regards,
> ismail
>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to