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

Correctly construct arguments for freebsd-update #1

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

dgeo
Copy link
Contributor

@dgeo dgeo commented Apr 12, 2024

Lost the original PR, but this patch ensures that freebsd-update args are correctly split

@arrowd
Copy link

arrowd commented Apr 13, 2024

You should have rights to merge PRs into this repo. Or are you requesting a review with this PR?

@dgeo
Copy link
Contributor Author

dgeo commented Apr 17, 2024

@arrowd seems I'm missing the "merge" button…
I see: Only those with [write access](https://docs.github.com/articles/what-are-the-different-access-permissions) to this repository can merge pull requests.
From git command: ERROR: Permission to freebsd/iocage.git denied to dgeo

@dgeo dgeo closed this Apr 17, 2024
@dgeo dgeo reopened this Apr 17, 2024
@arrowd
Copy link

arrowd commented Apr 18, 2024

@dgeo This should be fixed now. At least it is fixed for me.

@dgeo
Copy link
Contributor Author

dgeo commented Apr 18, 2024

Thank you !

@dgeo dgeo merged commit fc1d821 into freebsd:master Apr 18, 2024
0 of 8 checks passed
@dgeo dgeo deleted the fix-arguments branch April 18, 2024 14:40
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