Fix mac os nightly builds

[jakenvac]

I've managed to get mac os nightly builds working, but with a couple of caveats. (screenshots below)

I'm not really familiar with CMake or how it works so I'm sure this is very hacky. A lot of what I came up with in ci-script-mac.sh was trial and error. Any guidance is much appreciated.

Caveats:

• Build must be done with ICU disabled due to a version mismatch (I think) between the versions available in brew and what Ansel expects. I'm not sure what the impacts of this being disabled in the build are. Generally the app seems to run and perform as normal.
• Package signing needs configuring. For now you will have to allow the app to run from within the settings app (see screenshot)
• Currently only built for arm
• DMG name needs fixing, currently looks like Ansel--arm64.dmg need to use tag to insert version

Any advice of what else needs to be done to get this merged would be greatly appreciated.

You can see the github action on my fork producing a build here:
https://github.com/jakenvac/ansel/actions/runs/11742245723/job/32712579211

And you can download the release from my fork here:
https://github.com/jakenvac/ansel/releases/download/v0.0.0/Ansel--arm64.dmg

screenshots:

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[mmoqui]

Hi @jakenvac,

In your PR , some shell scripts are modified for only a CI context. But those shell scripts are also used to build by hand a MacOS X package of Ansel. And by modifying only a couple of scripts used in a chain of build, this chain is then broken.

In order to fix this chain, could you please modify also the following scripts in the packaging/macosx/ directory:

• 1_install_hb_dependencies.sh: replace the installation of ImageMagick by MagickGraphics;
• 2_build_hb_ansel_custom.sh:
  • update the value of the installDir variable with the new installation directory as used in the 3_make_hb_ansel_package.sh and 4_make_hb_ansel_dmg.sh scripts in the PR: ${scriptDir}/../../install;
  • add the build config parameter -DUSE_ICU=OFF in the cmake statement

[sidney]

I have submitted PR #391 that fixes the issue with ICU in the Mac build

[jakenvac]

@mmoqui Apologies for the delay in my response, I took some time away over xmas.

Thanks for the feedback, if I can find the time, I will address your comments.

[mmoqui]

Hello @jakenvac,

The PR of @sidney has been merged. Now, as the icu4c library issue has been fixed, once your have some time, could you please accordingly rebase your branch with master and restore the use of the icu4c lib in your fix (id est no need to add -DUSE_ICU=OFF in the cmake statement) ?