Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Github Action MacOSX revamp #343

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

dlmiles
Copy link
Contributor

@dlmiles dlmiles commented Oct 10, 2024

The FREAD macro name is renamed due to clash with sys/fcntl.h in Xcode SDK on MacOSX.

The GHA will build for both legacy x86_64 and newer ARM64 platforms.

The configure_mac resolves an issue with the wrong X/Xlib.h header being included due to a (bad) copy being bundled with tk-tcl when the copy from quartz is the one needed.

@dlmiles dlmiles force-pushed the master-upstream-20231010-macosx branch from d0e97e7 to b09e8d3 Compare October 10, 2024 19:26
@dlmiles
Copy link
Contributor Author

dlmiles commented Oct 10, 2024

Updated: Somehow lost TMPFILE=value assignment in YML during final prepare

Example build issue using MacOS 12 (Xcode 14.2 from MacOSX.platform).

In file included from grTk1.c:23:
In file included from ../utils/main.h:26:
In file included from ../windows/windows.h:26:
../utils/magic.h:143:13: warning: 'FREAD' macro redefined [-Wmacro-redefined]
    #define FREAD(a,b,c,d)    gzread(d,a,b*c)
            ^
/Applications/Xcode_14.2.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/sys/fcntl.h:110:9: note: previous definition is here
 #define FREAD           0x00000001
The -L option needs to be placed before the -l it needs to affect.
This maybe important on MacOSX where a nonstandard/optional package
provides X11 support so the locations are not in system locations.
@dlmiles dlmiles force-pushed the master-upstream-20231010-macosx branch from b09e8d3 to 360a158 Compare October 19, 2024 13:39
@dlmiles
Copy link
Contributor Author

dlmiles commented Oct 19, 2024

Next iteration on MacOS CI in Github Acton:

  • diagnostics improved
  • autoconf X11 compiler/linker switches fixed (maybe this wasn't causing an issue, but manual inspection of the -L/usr/X11/lib placement indicated it would have no effect at the end of the linker options)
  • Github Action Summary output (everything you might want to know is listed)
  • Performs install
  • Performs kick the tyres testing, to confirm it runs without crashing
  • It prepares download files with what is built (maybe the binaries work, maybe they don't, I try to solicit feedback)
    As before:
  • Builds for x86_64 and ARM64 MacOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant