Tidy bfd.pot
authorAlan Modra <amodra@gmail.com>
Thu, 30 Jan 2020 23:51:02 +0000 (10:21 +1030)
committerAlan Modra <amodra@gmail.com>
Fri, 31 Jan 2020 00:17:46 +0000 (10:47 +1030)
This patch removes the leak of Nick's source directory into bfd.pot,
and emits #line for some generated files so that those files aren't
referenced by comments in the .pot file.  You can see both of these
effects in the following diff.  I've also removed use of an
unnecessary temp file in the make rules.

@@ -92,10 +92,8 @@ msgstr ""
 #: elf64-nfp.c:238 elf64-ppc.c:1014 elf64-ppc.c:1349 elf64-ppc.c:1358
 #: elf64-s390.c:328 elf64-s390.c:378 elf64-x86-64.c:285 elfn32-mips.c:3786
 #: elfxx-ia64.c:324 elfxx-riscv.c:955 elfxx-sparc.c:589 elfxx-sparc.c:639
-#: elfxx-tilegx.c:912 elfxx-tilegx.c:952
-#: /work/sources/binutils/current/bfd/elfnn-aarch64.c:2215
-#: /work/sources/binutils/current/bfd/elfnn-aarch64.c:2313 elf32-ia64.c:214
-#: elf32-ia64.c:3862 elf64-ia64.c:214 elf64-ia64.c:3862
+#: elfxx-tilegx.c:912 elfxx-tilegx.c:952 elfnn-aarch64.c:2215
+#: elfnn-aarch64.c:2313 elfnn-ia64.c:214 elfnn-ia64.c:3862
 #, c-format
 msgid "%pB: unsupported relocation type %#x"
 msgstr ""

* Makefile.am (elf32-target.h, elf64-target.h): Don't use a temp
file.  Use $< and $@ in rules.
(elf32-aarch64.c, elf64-aarch64.c): Likewise.
(elf32-ia64.c, elf64-ia64.c): Likewise.
(elf32-riscv.c, elf64-riscv.c): Likewise.
(peigen.c, pepigen.c, pex64igen.c): Likewise.
(elf32-aarch64.c, elf64-aarch64.c): Don't emit $srcdir on #line.
(elf32-riscv.c, elf64-riscv.c): Likewise, and use $(SED).
(elf32-ia64.c, elf64-ia64.c): Do emit #line.
(peigen.c, pepigen.c, pex64igen.c): Likewise.
* Makefile.in: Regenerate.

bfd/ChangeLog
bfd/Makefile.am
bfd/Makefile.in

index 0061b6f355705aff11ffe31c67ed95d27e6ff2b4..59a787bf40ccce9df1734bdd0ce31c455adf514b 100644 (file)
@@ -1,3 +1,17 @@
+2020-01-31  Alan Modra  <amodra@gmail.com>
+
+       * Makefile.am (elf32-target.h, elf64-target.h): Don't use a temp
+       file.  Use $< and $@ in rules.
+       (elf32-aarch64.c, elf64-aarch64.c): Likewise.
+       (elf32-ia64.c, elf64-ia64.c): Likewise.
+       (elf32-riscv.c, elf64-riscv.c): Likewise.
+       (peigen.c, pepigen.c, pex64igen.c): Likewise.
+       (elf32-aarch64.c, elf64-aarch64.c): Don't emit $srcdir on #line.
+       (elf32-riscv.c, elf64-riscv.c): Likewise, and use $(SED).
+       (elf32-ia64.c, elf64-ia64.c): Do emit #line.
+       (peigen.c, pepigen.c, pex64igen.c): Likewise.
+       * Makefile.in: Regenerate.
+
 2020-01-31  Alan Modra  <amodra@gmail.com>
 
        PR 4110
index 7053a678a741a1e425f64b64b1bb368ecfdb28b3..b6088a3c4a86048b4d0ad37c49bbf7c515f1f36a 100644 (file)
@@ -827,63 +827,46 @@ endif
 endif
 
 elf32-target.h : elfxx-target.h
-       rm -f elf32-target.h
-       $(SED) -e s/NN/32/g < $(srcdir)/elfxx-target.h > elf32-target.new
-       mv -f elf32-target.new elf32-target.h
+       $(SED) -e s/NN/32/g < $< > $@
 
 elf64-target.h : elfxx-target.h
-       rm -f elf64-target.h
-       $(SED) -e s/NN/64/g < $(srcdir)/elfxx-target.h > elf64-target.new
-       mv -f elf64-target.new elf64-target.h
+       $(SED) -e s/NN/64/g < $< > $@
 
 elf32-aarch64.c : elfnn-aarch64.c
-       rm -f elf32-aarch64.c
-       echo "#line 1 \"$(srcdir)/elfnn-aarch64.c\"" > elf32-aarch64.new
-       $(SED) -e s/NN/32/g < $(srcdir)/elfnn-aarch64.c >> elf32-aarch64.new
-       mv -f elf32-aarch64.new elf32-aarch64.c
+       echo "#line 1 \"elfnn-aarch64.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-aarch64.c : elfnn-aarch64.c
-       rm -f elf64-aarch64.c
-       echo "#line 1 \"$(srcdir)/elfnn-aarch64.c\"" > elf64-aarch64.new
-       $(SED) -e s/NN/64/g < $(srcdir)/elfnn-aarch64.c >> elf64-aarch64.new
-       mv -f elf64-aarch64.new elf64-aarch64.c
+       echo "#line 1 \"elfnn-aarch64.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 elf32-ia64.c : elfnn-ia64.c
-       rm -f elf32-ia64.c
-       $(SED) -e s/NN/32/g < $(srcdir)/elfnn-ia64.c > elf32-ia64.new
-       mv -f elf32-ia64.new elf32-ia64.c
+       echo "#line 1 \"elfnn-ia64.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-ia64.c : elfnn-ia64.c
-       rm -f elf64-ia64.c
-       $(SED) -e s/NN/64/g < $(srcdir)/elfnn-ia64.c > elf64-ia64.new
-       mv -f elf64-ia64.new elf64-ia64.c
+       echo "#line 1 \"elfnn-ia64.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 elf32-riscv.c : elfnn-riscv.c
-       rm -f elf32-riscv.c
-       echo "#line 1 \"$(srcdir)/elfnn-riscv.c\"" > elf32-riscv.new
-       sed -e s/NN/32/g < $(srcdir)/elfnn-riscv.c >> elf32-riscv.new
-       mv -f elf32-riscv.new elf32-riscv.c
+       echo "#line 1 \"elfnn-riscv.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-riscv.c : elfnn-riscv.c
-       rm -f elf64-riscv.c
-       echo "#line 1 \"$(srcdir)/elfnn-riscv.c\"" > elf64-riscv.new
-       sed -e s/NN/64/g < $(srcdir)/elfnn-riscv.c >> elf64-riscv.new
-       mv -f elf64-riscv.new elf64-riscv.c
+       echo "#line 1 \"elfnn-riscv.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 peigen.c : peXXigen.c
-       rm -f peigen.c
-       $(SED) -e s/XX/pe/g < $(srcdir)/peXXigen.c > peigen.new
-       mv -f peigen.new peigen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pe/g < $< >> $@
 
 pepigen.c : peXXigen.c
-       rm -f pepigen.c
-       $(SED) -e s/XX/pep/g < $(srcdir)/peXXigen.c > pepigen.new
-       mv -f pepigen.new pepigen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pep/g < $< >> $@
 
 pex64igen.c: peXXigen.c
-       rm -f pex64igen.c
-       $(SED) -e s/XX/pex64/g < $(srcdir)/peXXigen.c > pex64igen.new
-       mv -f pex64igen.new pex64igen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pex64/g < $< >> $@
 
 BFD_H_DEPS= $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h $(INCDIR)/diagnostics.h
 LOCAL_H_DEPS= libbfd.h sysdep.h config.h bfd_stdint.h
index c055a9e4a213246d84ccbeff6772ded382c4a579..dd3474d92b0fd0db84a5ffb0bee597b6b06f98f8 100644 (file)
@@ -1959,63 +1959,46 @@ dwarf2.lo: dwarf2.c Makefile
 @am__fastdepCC_FALSE@  $(LTCOMPILE) -c -o $@ -DDEBUGDIR=\"$(DEBUGDIR)\" $(srcdir)/dwarf2.c
 
 elf32-target.h : elfxx-target.h
-       rm -f elf32-target.h
-       $(SED) -e s/NN/32/g < $(srcdir)/elfxx-target.h > elf32-target.new
-       mv -f elf32-target.new elf32-target.h
+       $(SED) -e s/NN/32/g < $< > $@
 
 elf64-target.h : elfxx-target.h
-       rm -f elf64-target.h
-       $(SED) -e s/NN/64/g < $(srcdir)/elfxx-target.h > elf64-target.new
-       mv -f elf64-target.new elf64-target.h
+       $(SED) -e s/NN/64/g < $< > $@
 
 elf32-aarch64.c : elfnn-aarch64.c
-       rm -f elf32-aarch64.c
-       echo "#line 1 \"$(srcdir)/elfnn-aarch64.c\"" > elf32-aarch64.new
-       $(SED) -e s/NN/32/g < $(srcdir)/elfnn-aarch64.c >> elf32-aarch64.new
-       mv -f elf32-aarch64.new elf32-aarch64.c
+       echo "#line 1 \"elfnn-aarch64.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-aarch64.c : elfnn-aarch64.c
-       rm -f elf64-aarch64.c
-       echo "#line 1 \"$(srcdir)/elfnn-aarch64.c\"" > elf64-aarch64.new
-       $(SED) -e s/NN/64/g < $(srcdir)/elfnn-aarch64.c >> elf64-aarch64.new
-       mv -f elf64-aarch64.new elf64-aarch64.c
+       echo "#line 1 \"elfnn-aarch64.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 elf32-ia64.c : elfnn-ia64.c
-       rm -f elf32-ia64.c
-       $(SED) -e s/NN/32/g < $(srcdir)/elfnn-ia64.c > elf32-ia64.new
-       mv -f elf32-ia64.new elf32-ia64.c
+       echo "#line 1 \"elfnn-ia64.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-ia64.c : elfnn-ia64.c
-       rm -f elf64-ia64.c
-       $(SED) -e s/NN/64/g < $(srcdir)/elfnn-ia64.c > elf64-ia64.new
-       mv -f elf64-ia64.new elf64-ia64.c
+       echo "#line 1 \"elfnn-ia64.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 elf32-riscv.c : elfnn-riscv.c
-       rm -f elf32-riscv.c
-       echo "#line 1 \"$(srcdir)/elfnn-riscv.c\"" > elf32-riscv.new
-       sed -e s/NN/32/g < $(srcdir)/elfnn-riscv.c >> elf32-riscv.new
-       mv -f elf32-riscv.new elf32-riscv.c
+       echo "#line 1 \"elfnn-riscv.c\"" > $@
+       $(SED) -e s/NN/32/g < $< >> $@
 
 elf64-riscv.c : elfnn-riscv.c
-       rm -f elf64-riscv.c
-       echo "#line 1 \"$(srcdir)/elfnn-riscv.c\"" > elf64-riscv.new
-       sed -e s/NN/64/g < $(srcdir)/elfnn-riscv.c >> elf64-riscv.new
-       mv -f elf64-riscv.new elf64-riscv.c
+       echo "#line 1 \"elfnn-riscv.c\"" > $@
+       $(SED) -e s/NN/64/g < $< >> $@
 
 peigen.c : peXXigen.c
-       rm -f peigen.c
-       $(SED) -e s/XX/pe/g < $(srcdir)/peXXigen.c > peigen.new
-       mv -f peigen.new peigen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pe/g < $< >> $@
 
 pepigen.c : peXXigen.c
-       rm -f pepigen.c
-       $(SED) -e s/XX/pep/g < $(srcdir)/peXXigen.c > pepigen.new
-       mv -f pepigen.new pepigen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pep/g < $< >> $@
 
 pex64igen.c: peXXigen.c
-       rm -f pex64igen.c
-       $(SED) -e s/XX/pex64/g < $(srcdir)/peXXigen.c > pex64igen.new
-       mv -f pex64igen.new pex64igen.c
+       echo "#line 1 \"peXXigen.c\"" > $@
+       $(SED) -e s/XX/pex64/g < $< >> $@
 $(BFD32_LIBS) \
  $(BFD64_LIBS) \
  $(ALL_MACHINES) \