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

Fix compose.sh on Linux #1972

Closed
wants to merge 1 commit into from
Closed

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere commented May 7, 2024

@maxandersen I don't get why the variable was commented out, probably a mistake?

As things are, your fix just makes it work for Mac, but no longer on Linux.

With this patch it works on Linux again, but I'll let someone else check it works on Mac...

@yrodiere yrodiere requested a review from maxandersen May 7, 2024 10:59
Copy link

github-actions bot commented May 7, 2024

🙈 The PR is closed and the preview is expired.

@maxandersen
Copy link
Member

maxandersen commented May 7, 2024

lets just revert it - using latest podman version that honors/ignores :z is better anyway.

opened #1973

@gsmet gsmet closed this May 10, 2024
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.

3 participants