Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-12 Thread Martin Storsjö
On Mon, 13 Feb 2023, Ziemowit Laski wrote: Not at all - it's entirely a technical thing. MSYS, which is a fork of cygwin, is an environment to provide a full POSIX (i.e. unix) environment on top of Windows, which involves a lot of trickery to make system calls like fork() work on top of a OS

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-12 Thread Ziemowit Laski
> Not at all - it's entirely a technical thing. MSYS, which is a fork of > cygwin, is an environment to provide a full POSIX (i.e. unix) environment > on top of Windows, which involves a lot of trickery to make system calls > like fork() work on top of a OS that doesn't provide that. But that's

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-12 Thread Martin Storsjö
On Sat, 11 Feb 2023, Ziemowit Laski wrote: Thanks for the detailed explanation. Now my head hurts. I guess by "native" you mean those apps that do not depend on MSYS/Cygwin DLL functionality? All of the apps are native. Yeah, more or less. They use the regular win32 api just like MSVC

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-10 Thread Ziemowit Laski
Thanks for the detailed explanation. Now my head hurts. I guess by "native" you mean those apps that do not depend on MSYS/Cygwin DLL functionality? All of the apps are native. So I can definitely understand the appeal of MinGW. But what about MSYS? Is it some sort of copyright/GPL avoidance

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-10 Thread Martin Storsjö
On Fri, 10 Feb 2023, Ziemowit Laski wrote: The build uses /bin/bash, which is configured as x86_64-pc-msys, and that's where the rewriting happens. I always assumed that MinGW was basically built on top of MSYS. No, that's a rather incorrect understanding. MinGW is an environment which

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-09 Thread Ziemowit Laski
> paths; it's msys2 which does the whole unix-style paths and which > automatically tries to rewrite command line arguments as if they were > paths, with some level of heuristics. Yes, this makes sense. The build uses /bin/bash, which is configured as x86_64-pc-msys, and that's where the

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-09 Thread Martin Storsjö
On Thu, 9 Feb 2023, Ziemowit Laski wrote: These changes seem fine, but you're apparently not touching the case at the top, used for --version, where it is calling 'rc.exe /?'. For me, this That's an interesting point. I guess MinGW is "smart enough" not to rewrite "/?" because it doesn't

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-09 Thread Hendrik Leppkes
On Thu, Feb 9, 2023 at 10:02 PM Ziemowit Laski wrote: > > > FWIW, this setup is definitely being used by lots of others already - so > > whenever there's such an issue, the main question to ask is why others > > haven't run into the issue before. But improvements are definitely > > welcome! > >

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-09 Thread Ziemowit Laski
> FWIW, this setup is definitely being used by lots of others already - so > whenever there's such an issue, the main question to ask is why others > haven't run into the issue before. But improvements are definitely > welcome! You have to have PATH set up so that rc.exe is found inside the

Re: [FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-09 Thread Martin Storsjö
Hi, On Sat, 4 Feb 2023, Ziemowit Laski wrote: I've been bringing up FFMPEG using Visual Studio 2022 and the MINGW64 environment, and came across sundry things that absolutely needed fixes, so I thought I'd submit them as a series of small patches for you to consider. Here is the first

[FFmpeg-devel] [PATCH] mswindres: Use '-' instead of '/' for rc.exe options

2023-02-04 Thread Ziemowit Laski
Hello Gentlefolk, I've been bringing up FFMPEG using Visual Studio 2022 and the MINGW64 environment, and came across sundry things that absolutely needed fixes, so I thought I'd submit them as a series of small patches for you to consider. Here is the first patch. --Zem