[PATCH] ld/testsuite: handle Windows drive letter in persistent section

Clément Chigot chigot@adacore.com
Tue Feb 21 09:14:03 GMT 2023


Hi Alan,

Here is another test having an issue with paths starting with C:, that
I've missed in first place.

However, it had made me wonder if no other tests would have this issue
(even if I'm not launching because of whatever reasons).
For ld/testsuite, noinit sections and this one seems the only ones
having this pattern "[^:] :".

However, in gas/testsuite, there are plenty of them, more than 16k lines.
But, investigating deeper, I don't have issues with them on my setup
because gas errors are based on the targeted files and I'm passing
them with a Linux path-style.
  | $ gas /a/random/path.s
  | /a/random/path.s: error ...

With ld, the error is printing the ld executable path which in my case
is a Windows-style path.
  | $ ld /a/random/path.o
  | C:/path/to/ld.exe: error ...

All that to say, as is, I don't have to modify any testcase in
gas/testsuite. But to be coherent, I guess it would be better to do
it.
But as it represents huge modifications, I would like to have your
opinion on that first.

Thanks, Clément


More information about the Binutils mailing list