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

docker-mender-convert: fix mktemp invocation for macOS/*BSD #430

Merged
merged 1 commit into from Feb 23, 2022

Conversation

TheYoctoJester
Copy link
Contributor

Changelog: fix compatibility of docker-mender-convert with macOS/*BSD

@mender-test-bot
Copy link
Contributor

@TheYoctoJester, Let me know if you want to start the integration pipeline by mentioning me and the command "start pipeline".


my commands and options
You can trigger a pipeline on multiple prs with:
  • mentioning me and start pipeline --pr mender/127 --pr mender-connect/255

You can cherry pick to a given branch or branches with:

  • mentioning me and:
 cherry-pick to:
 * 1.0.x
 * 2.0.x

@lluiscampos
Copy link
Contributor

@TheYoctoJester As our polite CI is reminding you, the commit is missing the Changelog. You wrote it here in the GitHub Pull Request description, but needs to be in the commit itself :) Do git commit --amend, fix it, and then force-push your branch.

Changelog: fix compatibility of docker-mender-convert with macOS/*BSD

The macOS/*BSD versions of mktemp do not support the -p flag
for passing in a path prefix. Fixing this by making $(pwd)
part of the template (-t). Based on a suggestion at [1].

[1]: https://hub.mender.io/t/bug-convert-a-raw-disk-image-in-macos-fails-mktemp-illegal-option-p/4428/3?u=theyoctojester

Signed-off-by: Josef Holzmayr <josef.holzmayr@northern.tech>
@TheYoctoJester
Copy link
Contributor Author

@lluiscampos done.

@lluiscampos lluiscampos self-assigned this Feb 23, 2022
@lluiscampos lluiscampos merged commit d6ffdcd into mendersoftware:master Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants