bug-binutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug binutils/27594] build processes broken by changed space handling


From: eliz at gnu dot org
Subject: [Bug binutils/27594] build processes broken by changed space handling
Date: Mon, 10 May 2021 17:13:58 +0000

https://sourceware.org/bugzilla/show_bug.cgi?id=27594

--- Comment #12 from Eli Zaretskii <eliz at gnu dot org> ---
(In reply to Martin Storsjö from comment #11)
> > Here is where I disagree: the "\"path to/gcc.exe\" or '"path to/gcc.exe"' is
> > NOT quoting, it's double quoting.  Quoting is just "path to/gcc.exe" or (for
> > Posix shells) 'path to/gcc.exe'.
> 
> Yes, that's true: We need to make sure windres receives a pre-quoted string,
> thus the caller has to double quote it.

That means `windres` will be the only program which requires such strange
invocation rules.  Every other program which does something similar doesn't
require such "over-quoting".

And I don't really understand why you are willing to require such strange
rules: is that only because of some build scripts that use such broken multiple
quoting?  IOW, instead of fixing things (and the fixes are really simple), we
are forever doomed to be bug-compatible with them?  And only on Windows on top
of that?

> Indeed, I think that was a mistake in the revert. I'll try to follow up with
> a patch for reverting the right thing in a couple hours, and for clarifying
> the documentation regarding it - to codify the existing status quo regarding
> it.

Thanks.

> Yes I agree that the most sane thing to do would be to have the user specify
> -DSTRING="quotedstring", but there are projects that have ended up working
> around this, so fixing this case does break them.

Those projects should get their act together, if you ask me.  It isn't hard,
really.  Not my call, obviously, but still...

-- 
You are receiving this mail because:
You are on the CC list for the bug.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]