Re: [patch] don't check for execute bits of the liblto plugin

2012-05-08 Thread Richard Guenther
On Tue, May 8, 2012 at 1:07 AM, Matthias Klose d...@ubuntu.com wrote:
 The lto plugin is installed without x bits set, but gcc-ar.c still checks for
 the execute bits. There is no need to have the lto plugin to have the x bits
 set, so just check that it is readable.

 Ok for the trunk and the 4.7 branch?

Ok.

Thanks,
Richard.

  Matthias


[patch] don't check for execute bits of the liblto plugin

2012-05-07 Thread Matthias Klose
The lto plugin is installed without x bits set, but gcc-ar.c still checks for
the execute bits. There is no need to have the lto plugin to have the x bits
set, so just check that it is readable.

Ok for the trunk and the 4.7 branch?

  Matthias
* (main): Don't check for execute bits for the plugin.

--- gcc/gcc-ar.c
+++ gcc/gcc-ar.c
@@ -70,7 +70,7 @@
   dir_separator,
   LTOPLUGINSONAME,
   NULL);
-  if (access (plugin, X_OK))
+  if (access (plugin, R_OK))
 {
   fprintf (stderr, %s: Cannot find plugin %s\n, av[0], plugin);
   exit (1);