public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] ld: fix ld-elf/linux-x86.exp for r/o source tree
@ 2020-06-02  7:12 Jan Beulich
  2020-06-02 11:14 ` H.J. Lu
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2020-06-02  7:12 UTC (permalink / raw)
  To: binutils

Since the copying ofthe individual files gets done more than once, make
sure the destinations can be copied to on subsequent copying attempts.

ld/
2020-06-XX  Jan Beulich  <jbeulich@suse.com>

	* testsuite/ld-elf/linux-x86.exp: Make copied source files
	writeable.

---
An alternative would be to "rm -f" the destinations up front; there are
surely further alternatives. Not sure which one's the most portable one.
---
I'd like to point out that, with the issue here fixed and on an oldish
system, several of the subcases of this test fail for me with
"compilation failed", and four subcases actually fail their run tests.
I didn't have the time to look into those, yet.

--- a/ld/testsuite/ld-elf/linux-x86.exp
+++ b/ld/testsuite/ld-elf/linux-x86.exp
@@ -134,6 +134,7 @@ proc check_pr25749a {testname srcfilea s
     }
 
     exec cp $srcdir/$subdir/$srcfilea $srcfilea
+    exec chmod +w $srcfilea
     set pr25749_bin "$objcopy -B $output_arch -I binary -O $output_target $srcfilea tmpdir/pr25749-bin.o"
     send_log "$pr25749_bin\n"
     set got [remote_exec host "$pr25749_bin"]
@@ -237,6 +238,7 @@ proc check_pr25749b {testname srcfilea s
     }
 
     exec cp $srcdir/$subdir/$srcfilea $srcfilea
+    exec chmod +w $srcfilea
     set pr25749_bin "$objcopy -B $output_arch -I binary -O $output_target $srcfilea tmpdir/pr25749-bin.o"
     send_log "$pr25749_bin\n"
     set got [remote_exec host "$pr25749_bin"]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] ld: fix ld-elf/linux-x86.exp for r/o source tree
  2020-06-02  7:12 [PATCH] ld: fix ld-elf/linux-x86.exp for r/o source tree Jan Beulich
@ 2020-06-02 11:14 ` H.J. Lu
  0 siblings, 0 replies; 2+ messages in thread
From: H.J. Lu @ 2020-06-02 11:14 UTC (permalink / raw)
  To: Jan Beulich; +Cc: Binutils

On Tue, Jun 2, 2020 at 12:12 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> Since the copying ofthe individual files gets done more than once, make
> sure the destinations can be copied to on subsequent copying attempts.
>
> ld/
> 2020-06-XX  Jan Beulich  <jbeulich@suse.com>
>
>         * testsuite/ld-elf/linux-x86.exp: Make copied source files
>         writeable.
>

OK.

Thanks.

-- 
H.J.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-06-02 11:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-02  7:12 [PATCH] ld: fix ld-elf/linux-x86.exp for r/o source tree Jan Beulich
2020-06-02 11:14 ` H.J. Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).