bug#78410: Patch for tiny improve cl wrapper script

2025-05-15 Thread Kirill Makurin
Hi Karl, I see you're somewhat confused about changes proposed by Yang Kun. Let me add some context here. "cl" (cl.exe) is Microsoft's C/C++ compiler, sometimes referred as MSVC. One thing to keep in mind about cl.exe is that it does not understand common flags like `-L`, ` -l`, `-Wl` or combi

bug#78410: Patch for tiny improve cl wrapper script

2025-05-15 Thread Nick Bowler
On Thu, May 15, 2025 at 02:07:39PM -0600, Karl Berry wrote: > > I think we'd better also search for lib$lib.dll.a even it's a odd > > behavior to search lib$lib.a for cl. > > What is "cl"? "Command line"? Doesn't make sense to me. "cl" is Microsoft's C compiler. > > However I don't know if tha

bug#78410: Patch for tiny improve cl wrapper script

2025-05-15 Thread Karl Berry
Hi Nick, Yang, IFS=$' \t\n' is a bashism which is not supported in most other shells. The original code is portable. This seems to have nothing to do with the rest of your change, so why change it at all? Indeed, I was going to make the same comment. I won't be installing that part o

bug#78410: Patch for tiny improve cl wrapper script

2025-05-15 Thread Nick Bowler
On Tue, May 13, 2025 at 11:05:54PM +0800, Yang Kun wrote: [...] > diff --git a/lib/compile b/lib/compile > index c404e89..a2afc93 100755 > --- a/lib/compile > +++ b/lib/compile > @@ -28,12 +28,8 @@ scriptversion=2025-02-03.05; # UTC > # bugs to or send patches to > # . > > -nl=' > -' > - > -#