[FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Carl Eugen Hoyos]

[-- Attachment #1: Type: text/plain, Size: 88 bytes --]

Hi!

Attached patch slightly simplifies building with MSVC.

Please comment, Carl Eugen

[-- Attachment #2: 0001-configure-Assume-host-os-win32-for-toolchain-msvc.patch --]
[-- Type: application/octet-stream, Size: 709 bytes --]

[-- Attachment #3: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Hendrik Leppkes]

On Sun, Nov 27, 2022 at 6:13 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
>
> Hi!
>
> Attached patch slightly simplifies building with MSVC.
>
> Please comment, Carl Eugen

I don't think any toolchain value should make assumptions about the
host it is running on, because between WSL and Wine, you can certainly
do a bunch of specific things where this is not true.

- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Carl Eugen Hoyos]

Am So., 27. Nov. 2022 um 18:16 Uhr schrieb Hendrik Leppkes
<h.leppkes@gmail.com>:
>
> On Sun, Nov 27, 2022 at 6:13 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> >
> > Hi!
> >
> > Attached patch slightly simplifies building with MSVC.
> >
> > Please comment, Carl Eugen
>
> I don't think any toolchain value should make assumptions about the
> host it is running on, because between WSL and Wine, you can certainly
> do a bunch of specific things where this is not true.

If this is true, then my patch is of course incorrect.
But could you give an example?
I was under the impression that cl.exe can only run
in win32.

Thank you, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Carl Eugen Hoyos]

Am So., 27. Nov. 2022 um 18:42 Uhr schrieb Carl Eugen Hoyos
<ceffmpeg@gmail.com>:
>
> Am So., 27. Nov. 2022 um 18:16 Uhr schrieb Hendrik Leppkes
> <h.leppkes@gmail.com>:
> >
> > On Sun, Nov 27, 2022 at 6:13 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> > >
> > > Hi!
> > >
> > > Attached patch slightly simplifies building with MSVC.
> > >
> > > Please comment, Carl Eugen
> >
> > I don't think any toolchain value should make assumptions about the
> > host it is running on, because between WSL and Wine, you can certainly
> > do a bunch of specific things where this is not true.
>
> If this is true, then my patch is of course incorrect.
> But could you give an example?
> I was under the impression that cl.exe can only run
> in win32.

Ping.
Could you give an example where host-os is not win32 when
compiling with msvc?

Thank you, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Hendrik Leppkes]

On Tue, Dec 13, 2022 at 8:02 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
>
> Am So., 27. Nov. 2022 um 18:42 Uhr schrieb Carl Eugen Hoyos
> <ceffmpeg@gmail.com>:
> >
> > Am So., 27. Nov. 2022 um 18:16 Uhr schrieb Hendrik Leppkes
> > <h.leppkes@gmail.com>:
> > >
> > > On Sun, Nov 27, 2022 at 6:13 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> > > >
> > > > Hi!
> > > >
> > > > Attached patch slightly simplifies building with MSVC.
> > > >
> > > > Please comment, Carl Eugen
> > >
> > > I don't think any toolchain value should make assumptions about the
> > > host it is running on, because between WSL and Wine, you can certainly
> > > do a bunch of specific things where this is not true.
> >
> > If this is true, then my patch is of course incorrect.
> > But could you give an example?
> > I was under the impression that cl.exe can only run
> > in win32.
>
> Ping.
> Could you give an example where host-os is not win32 when
> compiling with msvc?
>

When using wine, your host is linux.

- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Carl Eugen Hoyos]

Am Di., 13. Dez. 2022 um 20:21 Uhr schrieb Hendrik Leppkes
<h.leppkes@gmail.com>:
>
> On Tue, Dec 13, 2022 at 8:02 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> >
> > Am So., 27. Nov. 2022 um 18:42 Uhr schrieb Carl Eugen Hoyos
> > <ceffmpeg@gmail.com>:
> > >
> > > Am So., 27. Nov. 2022 um 18:16 Uhr schrieb Hendrik Leppkes
> > > <h.leppkes@gmail.com>:
> > > >
> > > > On Sun, Nov 27, 2022 at 6:13 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> > > > >
> > > > > Hi!
> > > > >
> > > > > Attached patch slightly simplifies building with MSVC.
> > > > >
> > > > > Please comment, Carl Eugen
> > > >
> > > > I don't think any toolchain value should make assumptions about the
> > > > host it is running on, because between WSL and Wine, you can certainly
> > > > do a bunch of specific things where this is not true.
> > >
> > > If this is true, then my patch is of course incorrect.
> > > But could you give an example?
> > > I was under the impression that cl.exe can only run
> > > in win32.
> >
> > Ping.
> > Could you give an example where host-os is not win32 when
> > compiling with msvc?
> >
>
> When using wine, your host is linux.

Understood, thank you!

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Timo Rothenpieler]

On 13.12.2022 20:00, Carl Eugen Hoyos wrote:
> Ping.
> Could you give an example where host-os is not win32 when
> compiling with msvc?

WSL, which I quite regularly use to test MSVC builds.
Just need to launch it from inside an msvc shell.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH]configure: Assume target-os=win32 for toolchain msvc

[Carl Eugen Hoyos]

Am Mi., 14. Dez. 2022 um 00:43 Uhr schrieb Timo Rothenpieler
<timo@rothenpieler.org>:
>
> On 13.12.2022 20:00, Carl Eugen Hoyos wrote:
> > Ping.
> > Could you give an example where host-os is not win32 when
> > compiling with msvc?
>
> WSL, which I quite regularly use to test MSVC builds.
> Just need to launch it from inside an msvc shell.

This is the bug that my patch is trying to fix:
When you build with msvc in wsl, you are doing a native build on win32.
I want to avoid the "--host-os=win32" for this case.

I had not imagined an msvc build using wine on Linux.

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".