[PATCH] darwin: Fix non-Metal build
Keno Fischer
keno at juliacomputing.com
Sun Dec 12 01:30:24 CST 2021
On Sat, Dec 11, 2021 at 9:02 AM Dean Greer <gcenx83 at gmail.com> wrote:
>
> You also need to edit configure.ac to to account for this change to $(METAL_LIBS) else when metal is detected quartzcore will be added again.
The duplicate `-framework` shouldn't cause any issues, but I'll send a
v2 that removes this in configure.ac, since this is only used here, so
we might as well be clean.
> What SDK was used for this compile that failed?
This was using https://binarybuilder.org/ which currently does not
provide a Metal toolchain.
> On Sat, Dec 11, 2021 at 4:22 AM Keno Fischer <keno at juliacomputing.com> wrote:
>>
>> The build was relying on QuartzCore definitions even if
>> Metal was not found, so move the appropriate import out
>> of the ifdef and add the QuartzCore framework to the
>> make line.
>>
>> Signed-off-by: Keno Fischer <keno at juliacomputing.com>
>> ---
>> dlls/winemac.drv/Makefile.in | 2 +-
>> dlls/winemac.drv/cocoa_window.m | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/dlls/winemac.drv/Makefile.in b/dlls/winemac.drv/Makefile.in
>> index 6329e8e76c8..fc3dddbdae7 100644
>> --- a/dlls/winemac.drv/Makefile.in
>> +++ b/dlls/winemac.drv/Makefile.in
>> @@ -1,7 +1,7 @@
>> MODULE = winemac.drv
>> IMPORTS = uuid rpcrt4 user32 gdi32 advapi32 win32u
>> DELAYIMPORTS = ole32 shell32 imm32
>> -EXTRALIBS = -framework AppKit -framework Carbon -framework Security -framework OpenGL -framework IOKit -framework CoreVideo $(METAL_LIBS)
>> +EXTRALIBS = -framework AppKit -framework Carbon -framework Security -framework OpenGL -framework IOKit -framework CoreVideo -framework QuartzCore $(METAL_LIBS)
>>
>> EXTRADLLFLAGS = -mcygwin
>>
>> diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m
>> index d0672b7fb06..a18fc069604 100644
>> --- a/dlls/winemac.drv/cocoa_window.m
>> +++ b/dlls/winemac.drv/cocoa_window.m
>> @@ -25,8 +25,8 @@
>> #import <CoreVideo/CoreVideo.h>
>> #ifdef HAVE_METAL_METAL_H
>> #import <Metal/Metal.h>
>> -#import <QuartzCore/QuartzCore.h>
>> #endif
>> +#import <QuartzCore/QuartzCore.h>
>>
>> #import "cocoa_window.h"
>>
>> --
>> 2.25.1
>>
>>
More information about the wine-devel
mailing list