-
Notifications
You must be signed in to change notification settings - Fork 269
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
add "yast_ftp_server" to support SLE12sp5 #8901
Conversation
@foursixnine @mimi1vx Could you please review it? |
tests/yast2_cmd/yast_ftp_server.pm
Outdated
@@ -110,7 +110,7 @@ sub run { | |||
if (is_sle('15+')) { | |||
assert_script_run 'yast ftp-server startup socket'; | |||
} | |||
if (is_sle('<=12-sp4')) { | |||
if (is_sle('<15')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is fine, but just to be explicit, I'd personally use:
if (is_sle('<15')) { | |
if (is_sle('<=12-sp5')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed as suggested.
tests/yast2_cmd/yast_ftp_server.pm
Outdated
@@ -153,7 +153,7 @@ sub run { | |||
if (is_sle('15+')) { | |||
assert_script_run 'yast ftp-server startup socket'; | |||
} | |||
if (is_sle('<=12-sp4')) { | |||
if (is_sle('<15')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed as suggested.
a2f50b9
to
d9565aa
Compare
d9565aa
to
1b18447
Compare
This module is a suite of tests for yast ftp server, this update fixed the lack support of SLE12sp5.
Needles: no
Verification run: