Module: wine Branch: master Commit: 6705a710450b015de9f378a4a4480b7623c837ad URL: https://gitlab.winehq.org/wine/wine/-/commit/6705a710450b015de9f378a4a4480b7...
Author: Martin Storsjö martin@martin.st Date: Thu Oct 20 23:52:13 2022 +0300
winegcc: Prefer -Wl,--pdb=<file> over -Wl,-pdb,<file>.
The previously used syntax, -Wl,-pdb,<file>, was the originally supported one in lld (since 2018). Later (in 2019) the second syntax was added, allowing both -Wl,-pdb=<file> and -Wl,-pdb,<file>. (This other parameter syntax allows easier distinguishing an empty argument, for letting the linker implicitly pick the file name.)
Move over to using the more modern syntax - reducing the usage of the old form of the option. This potentially allows deprecating the original syntax and maybe allows using --pdb as a boolean flag for implying an automatically named PDB file (which currently requires using the awkward syntax "--pdb=").
Also prefer the long two dashes form, i.e. --pdb instead of -pdb; the single dash form is only allowed by getopt when there are no conflicts with single-letter options, while the form with two dashes is unambiguous.
---
tools/winegcc/winegcc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index e40aa9270ec..ddd513e55d0 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -464,7 +464,7 @@ static struct strarray get_link_args( struct options *opts, const char *output_n strarray_add( &flags, "-static-libgcc" );
if (opts->debug_file && strendswith(opts->debug_file, ".pdb")) - strarray_add(&link_args, strmake("-Wl,-pdb,%s", opts->debug_file)); + strarray_add(&link_args, strmake("-Wl,--pdb=%s", opts->debug_file));
if (opts->out_implib) strarray_add(&link_args, strmake("-Wl,--out-implib,%s", opts->out_implib));