Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
On Wed, 30 Sep 2020, Biswapriyo Nath wrote: Would you like to provide the configure command to build ffmpeg as per your need? ffmpeg has thousands of configure options and I am not familiar with all of them. Cross compiling, you'd build it like this: ./configure --arch=x86_64 --target-os=mingw32 --cross-prefix=x86_64-w64-mingw32- --enable-cross-compile make (Or replacing both instances of x86_64 with aarch64 if you want that.) If building in a separate directory, you'd do "../path/to/src/configure ..." instead. For a non-cross build, it's enough with just "./configure; make". // Martin ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] dwrite_3.h regression fix
On Thu, Sep 24, 2020 at 12:03 PM Jacek Caban wrote: > > Hi, > > > I sent this as a patch earlier, but it got stuck on moderation due to > the size. Please review commit: I increased the message size limit from 512KB to 999KB, which appears to be a sourceforge limit. If you think we need larger, I can inquire about doing so. ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] dwrite_3.h regression fix
Following up on Biswapriyo's point, at least for our current use case, the following two changes suffice: 1. Updating wine to include the DWRITE_MAKE_FONT_AXIS_TAG macro. See https://github.com/sgmenda/mingw-w64/commit/8c95e8da20b98cdadd17434887ed5ac7 a0d1b29c#diff-82ae7e785e092cc11713885893c11bea 2. Breaking compat with the IDL to rename GetGlyphImageFormats_ to GetGlyphImageFormats. See https://github.com/sgmenda/mingw-w64/commit/92df20a5f2744b250d80f4f9a08572b8 4037d0b0 and we can get back compat once widl supports function overloading. Best, Sanketh ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
Would you like to provide the configure command to build ffmpeg as per your need? ffmpeg has thousands of configure options and I am not familiar with all of them. ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] dwrite_3.h regression fix
(Reposted; my last email was stuck in moderation.) Following up on Biswapriyo's point, at least for our current use case, the following two changes suffice: 1. Updating wine to include the DWRITE_MAKE_FONT_AXIS_TAG macro. See https://github.com/sgmenda/mingw-w64/commit/8c95e8da20b98cdadd17434887ed5ac7 a0d1b29c#diff-82ae7e785e092cc11713885893c11bea 2. Breaking compat with the IDL to rename GetGlyphImageFormats_ to GetGlyphImageFormats. See https://github.com/sgmenda/mingw-w64/commit/92df20a5f2744b250d80f4f9a08572b8 4037d0b0 and we can get back compat once widl supports function overloading. Best, Sanketh ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
On Wed, 30 Sep 2020, Nikolay Sivov wrote: On 9/30/20 1:09 PM, Martin Storsjö wrote: On Wed, 30 Sep 2020, Biswapriyo Nath wrote: BTW, an unrelated point, but - if working on the mediafoundation headers, ffmpeg is a project happens to use them a bit, so if doing changes, I'd appreciate if you'd check that ffmpeg still builds (and ends up with mediafoundation enabled). The ffmpeg mediafoundation code also happens to use local redeclarations of many mediafoundation constants (to allow it to build with older versions of mingw-w64 and WinSDK) - so if working on header changes, it can also be useful to try building with a modified version that actually use the SDK headers properly: https://github.com/mstorsjo/ffmpeg/commits/mfenc-use-sdk (With current mingw-w64, all constants that are used are available, except for CODECAPI_AVScenarioInfo.) // Martin No, it is very much related. Thanks for that. This could probably be expanded to other major projects too, for validation before major mingw-w64 updates are released. Indeed - I run a nightly build consisting of a bunch of multimedia projects (VLC and all of their 100 dependency libraries, including Qt, and a few others, and a full test of ffmpeg) with latest mingw-w64, so generally that's at least build tested regularly (and I try to get that tested before major releases as well). But for wrangling header updates here, it's even nicer if the header changes are tested with a project known to exercise them - instead of finding out potential breakage after the fact. // Martin ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
On 9/30/20 1:09 PM, Martin Storsjö wrote: > On Wed, 30 Sep 2020, Biswapriyo Nath wrote: > >> > > BTW, an unrelated point, but - if working on the mediafoundation > headers, ffmpeg is a project happens to use them a bit, so if doing > changes, I'd appreciate if you'd check that ffmpeg still builds (and > ends up with mediafoundation enabled). > > The ffmpeg mediafoundation code also happens to use local > redeclarations of many mediafoundation constants (to allow it to build > with older versions of mingw-w64 and WinSDK) - so if working on header > changes, it can also be useful to try building with a modified version > that actually use the SDK headers properly: > https://github.com/mstorsjo/ffmpeg/commits/mfenc-use-sdk > > (With current mingw-w64, all constants that are used are available, > except for CODECAPI_AVScenarioInfo.) > > // Martin > > No, it is very much related. Thanks for that. This could probably be expanded to other major projects too, for validation before major mingw-w64 updates are released. ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
On Wed, 30 Sep 2020, Biswapriyo Nath wrote: BTW, an unrelated point, but - if working on the mediafoundation headers, ffmpeg is a project happens to use them a bit, so if doing changes, I'd appreciate if you'd check that ffmpeg still builds (and ends up with mediafoundation enabled). The ffmpeg mediafoundation code also happens to use local redeclarations of many mediafoundation constants (to allow it to build with older versions of mingw-w64 and WinSDK) - so if working on header changes, it can also be useful to try building with a modified version that actually use the SDK headers properly: https://github.com/mstorsjo/ffmpeg/commits/mfenc-use-sdk (With current mingw-w64, all constants that are used are available, except for CODECAPI_AVScenarioInfo.) // Martin ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
OK, as you wish. ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
Re: [Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
For Media Foundation headers I think we should work towards importing them from Wine, after backporting whatever it's missing comparing to mingw-w64 ones. ___ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
[Mingw-w64-public] [PATCH] headers/mfidl: Import mftransform.idl instead of header.
From 35f208a47a220a365b23f0a06a051990910e5e83 Mon Sep 17 00:00:00 2001
From: Biswapriyo Nath
Date: Wed, 30 Sep 2020 13:18:11 +0530
Subject: [PATCH] headers/mfidl: Import mftransform.idl instead of header.
Signed-off-by: Biswapriyo Nath
---
mingw-w64-headers/include/mfidl.h | 4 ++--
mingw-w64-headers/include/mfidl.idl | 3 +--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/mingw-w64-headers/include/mfidl.h
b/mingw-w64-headers/include/mfidl.h
index e5143be..13c1c95 100644
--- a/mingw-w64-headers/include/mfidl.h
+++ b/mingw-w64-headers/include/mfidl.h
@@ -1,4 +1,4 @@
-/*** Autogenerated by WIDL 5.0-rc1 from mfidl.idl - Do not edit ***/
+/*** Autogenerated by WIDL 5.16 from mfidl.idl - Do not edit ***/
#ifdef _WIN32
#ifndef __REQUIRED_RPCNDR_H_VERSION__
@@ -101,12 +101,12 @@ interface IMFClockStateSink;
/* Headers for imported files */
#include
+#include
#ifdef __cplusplus
extern "C" {
#endif
-#include
#include
typedef enum MFSESSION_SETTOPOLOGY_FLAGS {
MFSESSION_SETTOPOLOGY_IMMEDIATE = 0x1,
diff --git a/mingw-w64-headers/include/mfidl.idl
b/mingw-w64-headers/include/mfidl.idl
index 9dcf51f..baf0cff 100644
--- a/mingw-w64-headers/include/mfidl.idl
+++ b/mingw-w64-headers/include/mfidl.idl
@@ -5,8 +5,7 @@
*/
import "mfobjects.idl";
-
-cpp_quote("#include ") // FIXME: import "mftransform.idl";
+import "mftransform.idl";
cpp_quote("#include ")
--
2.27.0
___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
