[argyllcms] Re: Patch for Apple Silicon M1 to correctly detect display names and build native

  • From: Lars Tore Gustavsen <lars.tore@xxxxxxxxxxxxxx>
  • To: argyllcms@xxxxxxxxxxxxx
  • Date: Fri, 12 Feb 2021 18:20:56 +0100

Just some quick testing on Macbook M1.
Compared to the patch from Lehmann, no detected display name.

./Argyll_V2.1.3_beta/bin/dispcal
Calibrate a Display, Version 2.1.3_beta
Author: Graeme W. Gill, licensed under the AGPL Version 3
Diagnostic: Too few arguments
usage: dispcal [options] outfile
 -v [n]               Verbose mode
 -d n                 Choose the display from the following list (default 1)
dispcal: Warning - IODisplayCreateInfoDictionary returned NULL!
    1 = '(unknown), at 0, 0, width 1440, height 900 (Primary Display)'

./Argyll-macos-arm-main/bin/dispcal
Calibrate a Display, Version 2.1.2
Author: Graeme W. Gill, licensed under the AGPL Version 3
Diagnostic: Too few arguments
usage: dispcal [options] outfile
 -v [n]               Verbose mode
 -d n                 Choose the display from the following list (default 1)
    1 = 'Built-in Retina Display, at 0, 0, width 1440, height 900 (Primary
Display)'


And it's much slower.
$ time Argyll_V2.1.3_beta//bin/colprof   -D"time test" -qh -al test5
real 1m31.294s

$ time Argyll-macos-arm-main/bin/colprof   -D"time test" -qh -al test5
real 0m32.865s


On Fri, Feb 12, 2021 at 5:37 AM Graeme Gill <graeme@xxxxxxxxxxxxx> wrote:

Simone Karin Lehmann wrote:

Hi,

I’ve written some small patches against ArgyllCMS 2.1.2. These will use
a new API in macOS 10.15
and later, which will allow disown.c to correctly detect the display
names.

thanks for your work.

And there are some small modifications to smoothly build as native Apple
Silicon binaries.

You could have saved a little work by starting with the current
source code snapshot in <https://www.argyllcms.com/Argyll_dev_src.zip> :-)

So maybe you can take a look t it and I’d be glad, if you’ll apply the
patches to ArgyllCMS.

So I've updated <https://www.argyllcms.com/Argyll_dev_src.zip> with the
patches
applied, but I've had to make a number of changes and additions to get it
to work on older compile environments and OS X operating systems, so it
would
be good if you could test it.

Thanks,
        Graeme Gill.


Other related posts: