Skip to content

Add EESSI client container for macOS-26.x#266

Merged
boegel merged 5 commits intoEESSI:mainfrom
TopRichard:macOS-Tahoe-Ubuntu-22.04
Mar 13, 2026
Merged

Add EESSI client container for macOS-26.x#266
boegel merged 5 commits intoEESSI:mainfrom
TopRichard:macOS-Tahoe-Ubuntu-22.04

Conversation

@TopRichard
Copy link

No description provided.

@boegel
Copy link
Contributor

boegel commented Mar 10, 2026

@TopRichard Is the container recipe you're adding here at all specific to macOS?

If so, can you point out there? I guess that should be highlighted with a comment.

If not, we should probably rename the recipe to not even mention macOS? (and I would personally go with macOS-26.x, not Tahoe, but that's a tiny detail)

@TopRichard
Copy link
Author

TopRichard commented Mar 11, 2026

@TopRichard Is the container recipe you're adding here at all specific to macOS?

If so, can you point out there? I guess that should be highlighted with a comment.

If not, we should probably rename the recipe to not even mention macOS? (and I would personally go with macOS-26.x, not Tahoe, but that's a tiny detail)

Yes, the recipe uses the macOS user ID and group ID to allow proper bind mounting.

@TopRichard TopRichard changed the title Add EESSI client container for macOS Tahoe Add EESSI client container for macOS-26.x Mar 11, 2026
@TopRichard TopRichard force-pushed the macOS-Tahoe-Ubuntu-22.04 branch from 3ad8db4 to 9581d16 Compare March 11, 2026 08:24
@boegel
Copy link
Contributor

boegel commented Mar 13, 2026

@TopRichard The Dockerfile should be renamed to also use macOS-26.x?

(and maybe macOS-26 is nicer, but now I'm really nitpicking...)

Copy link
Contributor

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel boegel merged commit 72c01b6 into EESSI:main Mar 13, 2026
37 checks passed
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.

2 participants