add OpenBSD 6.2 #54
Merged
add OpenBSD 6.2 #54
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mitsururike
reviewed
Oct 19, 2017
scripts/openbsd/init.sh
Outdated
| @@ -3,9 +3,12 @@ | |||
| set -e | |||
| set -x | |||
|
|
|||
| sudo tee /etc/pkg.conf <<EOF | |||
| installpath = ftp.openbsd.org | |||
| # pkg.conf has been replaces with installurl since 6.1 | |||
mitsururike
reviewed
Oct 19, 2017
scripts/openbsd/init.sh
Outdated
| @@ -42,7 +45,7 @@ fi | |||
| # XXX this patch should be applied in our ports tree, but we desparately need | |||
| # package builder first. remove this when the builder is ready. | |||
| # | |||
| if [ `ansible --version | head -n 1 | cut -d' ' -f 2` == '2.3.2.0' ]; then | |||
| if [ `ansible --version | head -n 1 | cut -d' ' -f 2` == '2.3.2.0' -a `uname -r` != '6.2' ]; then | |||
There was a problem hiding this comment.
our ports tree has been patched for 6.2?
There was a problem hiding this comment.
have not looked at the patches directory yet, but the package in OPENBSD_6_2 has passed specs that check the bug in question.
mitsururike
reviewed
Oct 19, 2017
reallyenglish_spec/packages_spec.rb
Outdated
| @@ -15,24 +15,27 @@ | |||
| end | |||
| when "openbsd" | |||
| # XXX RE_5_9 does not have the latest ansible yet | |||
| if os[:release].to_f >= 6.0 | |||
| if os[:release].to_f >= 6.0 && os[:release].to_f < 6.2 | |||
There was a problem hiding this comment.
i do not understand why 6.2 and above are excluded.
added 3 commits
October 19, 2017 18:18
of course, it is still messy. it will be fixed once we have package builders and ansible ports are updated to the same one. also: * fix a typo, `uanme`
|
all boxes have passed the tests. |
added 2 commits
October 19, 2017 20:25
this installs bsd.mp on OpenBSD
what is tested here is the MP kernel has been installed
|
all boxes have passed the tests. |
mitsururike
reviewed
Oct 20, 2017
|
i have been working on another feature branch. the tests you have mentioned would be implemented in the branch |
mitsururike
approved these changes
Oct 20, 2017
|
f71aa2c adds the tests |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
this PR add OpenBSD 6.2.
also:
bsd.mp, fixes install bsd.mp on all OpenBSD boxes #55as a result, all OpenBSD boxes will be updated