Hi Andreas,

On 16/10/17 20:00, Andreas Tobler wrote:
Hi all,

I struggled over a bootstrap issue while building gcc-7 for armv7-*-freebsd*

I got a 'permission denied' while creating the arm-tables.opt file.

The source tree is located on a nfs server.

The below patch fixed it for me.

Ok to apply?

TIA,
Andreas

2017-10-16  Andreas Tobler  <andre...@gcc.gnu.org>

        * config/arm/t-arm (MD_INCLUDES): Create arm-tables.opt via
        intermediate arm-tables.new like the other awk generated files.

Index: config/arm/t-arm
===================================================================
--- config/arm/t-arm    (revision 253792)
+++ config/arm/t-arm    (working copy)
@@ -75,8 +75,8 @@
  $(srcdir)/config/arm/arm-tables.opt: $(srcdir)/config/arm/parsecpu.awk \
    $(srcdir)/config/arm/arm-cpus.in
         $(AWK) -f $(srcdir)/config/arm/parsecpu.awk -v cmd=opt \
-               $(srcdir)/config/arm/arm-cpus.in > \
-               $(srcdir)/config/arm/arm-tables.opt
+               $(srcdir)/config/arm/arm-cpus.in > arm-tables.new
+       mv arm-tables.new $(srcdir)/config/arm/arm-tables.opt


This looks ok to me as it makes the rule consistent with the other awk-generated files.

Out of interest, this looks like a small subset of Richard's patch [1] at r249971.
Have you tried that patch on the branch?

[1] https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00223.html

Thanks,
Kyrill
  $(srcdir)/config/arm/arm-cpu.h: $(srcdir)/config/arm/parsecpu.awk \
    $(srcdir)/config/arm/arm-cpus.in

Reply via email to