On Sun, Apr 12, 2020 at 11:54:53AM +0530, Ashish SHUKLA wrote:
> On Apr 11, 2020, at 18:59, Alan Third <alan@idiocy.org> wrote:
> > 
> > You may have to set this CFLAG to enable the changes:
> > 
> >    -DMAC_OS_X_VERSION_MAX_ALLOWED=101500
> 
> Hi
> 
> Without setting CFLAGS following error occurred:
> 
> ==================================================================
>   CC       image.o
>   CC       json.o
>   CC       nsterm.o
>   CC       nsfns.o
>   CC       nsmenu.o
>   CC       nsselect.o
>   CC       nsimage.o
>   CC       macfont.o
>   CC       terminfo.o
>   CC       lastfile.o
> nsterm.m:6300:21: error: use of undeclared identifier 'drawingBuffer'
>   CGContextRelease (drawingBuffer);
>                     ^
> 1 error generated.

I’m surprised this didn’t show up before.

> In file included from <built-in>:360:
> <command line>:4:9: warning: 'MAC_OS_X_VERSION_MAX_ALLOWED' macro redefined [-Wmacro-redefined]
> #define MAC_OS_X_VERSION_MAX_ALLOWED 101500
>         ^
> <command line>:3:9: note: previous definition is here
> #define MAC_OS_X_VERSION_MAX_ALLOWED 101200
>         ^

This looks suspiciously like you’re already setting that define in
your configure command. If you’re building with nix check the build
script. You, or whoever created it, probably had to add the define to
101200 (which is macOS 10.12) so it would correctly handle some
features added since macOS 10.10. I know that John Weigley uses (or
used) that value.

There’s a possibility it will appear in a different form, but it will
be something that either has 101200 or 10.12.

> nsterm.m:1148:8: error: use of undeclared identifier 'self'
>   if ([self wantsUpdateLayer])
>        ^

Silly mistake on my part.

New patch attached. You still have to use the other patch from my last
email as well.
-- 
Alan Third