GNU bug report logs -
#72440
31.0.50; MacOS: TUINSRemoteViewController log message
Previous Next
Reported by: Gerd Möllmann <gerd.moellmann <at> gmail.com>
Date: Sat, 3 Aug 2024 10:34:02 UTC
Severity: normal
Found in version 31.0.50
Fixed in version 31.1
Done: Gerd Möllmann <gerd.moellmann <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
Message #25 received at 72440 <at> debbugs.gnu.org (full text, mbox):
On Mon, Aug 05, 2024 at 08:27:55PM +0200, Gerd Möllmann wrote:
> Mattias Engdegård <mattias.engdegard <at> gmail.com> writes:
>
> > 5 aug. 2024 kl. 17.27 skrev Gerd Möllmann <gerd.moellmann <at> gmail.com>:
> >
> >> Can you please check with what I just committed?
> >
> > Thank you but that doesn't make a difference -- it's a static, not dynamic, warning.
> > Instead we could put it inside
> >
> > #if MAC_OS_X_VERSION_MAX_ALLOWED >= 140000
> > ..
> > #endif
> >
> > which works here but I may be confusing the version macros.
> >
> > More to the point, is the [self setClipsToBounds: YES] line actually
> > required with older OS versions? If so, maybe it's just a matter of
> > declaring the property somehow since it's supposed to exist even if
> > the declaration was only made public in v14, but not sure if that is
> > even possible.
>
> I also don't know what's the right thing to do is here. One could use
> the #ifdef as you showed, but then an Emacs compiled on 13.x won't do
> what it should when run on 14.x. (I gather that it would otherwise run
> fine.)
IIRC apps built on an older version of macOS generally run as though
they are on that version of macOS even when on a newer version.
> A runtime check with NSAppKitVersionNumber would work, if one copies the
> float values for the version from NSApplication.h, and compares with
> that. One has to copy the version number if one compiles on older
> versions where the constant is not defined (can be either a define or a
> const double in 14.5 here).
>
> What a mess. Hopefully Alan has some advice.
The correct thing to do is only set this when compiling on v14 or
above, so the #ifdef is the right way to go.
FWIW, we have never guaranteed an executable compiled on one version
of macOS will work on any other version. At one time I put a fair bit
of effort into trying to get that working and it just turned into a
nightmare of #if's and runtime checks all over the place and made
maintenance far too hard.
Apple's approach would be that you *always* compile against the latest
version, and only support three versions back, or whatever, and that
way you can easily insert run-time checks as the API will always be
available at compile time, even if it's not there at runtime.
Emacs packagers generally build against multiple versions of macOS and
ship a bundle of multiple executables that are selected by a script at
runtime.
(Another approach, that I'd not recommend, would be to just put the
code in with no check whatsoever. A non-existent method will always
return nil at runtime, and just return a warning at compile time. This
means people compiling on older versions just have to live with
compiler warnings, but otherwise nothing should break.)
--
Alan Third
This bug report was last modified 285 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.