Skip to content

Avoid using mktemp -p#60091

Merged
fingolfin merged 1 commit intomasterfrom
mh/Makefile-mktemp-darwin
Nov 10, 2025
Merged

Avoid using mktemp -p#60091
fingolfin merged 1 commit intomasterfrom
mh/Makefile-mktemp-darwin

Conversation

@fingolfin
Copy link
Copy Markdown
Member

It is not supported in macOS 13 userland which causes CI failures. For background info see here.

This is a subset of PR #60080 but unlike that, it only solves the specific issue causing trouble in CI right now, and does not attempt to resolve a bunch of other things -- which may very well all be fine and nice and desirable, but it's quite a bit more to review.

@fingolfin fingolfin requested a review from vtjnash November 10, 2025 12:00
@fingolfin fingolfin added the building Build system, or building Julia or its dependencies label Nov 10, 2025
@lgoettgens
Copy link
Copy Markdown
Contributor

@fingolfin
Copy link
Copy Markdown
Member Author

Indeed, thanks

@fingolfin fingolfin force-pushed the mh/Makefile-mktemp-darwin branch from 014bdd5 to d28fa40 Compare November 10, 2025 13:20
It is not supported in macOS 13 userland which causes
CI failures.

Also clean the temp files if necessary.
@fingolfin fingolfin force-pushed the mh/Makefile-mktemp-darwin branch from d28fa40 to 844bdd4 Compare November 10, 2025 13:20
@fingolfin fingolfin merged commit 3747300 into master Nov 10, 2025
7 checks passed
@fingolfin fingolfin deleted the mh/Makefile-mktemp-darwin branch November 10, 2025 21:33
@KristofferC KristofferC added the backport 1.13 Change should be backported to release-1.13 label Mar 12, 2026
KristofferC pushed a commit that referenced this pull request Mar 12, 2026
It is not supported in macOS 13 userland which causes CI failures. For
background info [see
here](#59980 (comment)).

This is a subset of PR #60080 but unlike that, it only solves the
specific issue causing trouble in CI right now, and does not attempt to
resolve a bunch of other things -- which may very well all be fine and
nice and desirable, but it's quite a bit more to review.

(cherry picked from commit 3747300)
@KristofferC KristofferC removed the backport 1.13 Change should be backported to release-1.13 label Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

building Build system, or building Julia or its dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants