-
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
Do not check license for Web&Scripting module on SLE15 #4756
Conversation
lib/registration.pm
Outdated
@@ -22,7 +22,7 @@ use strict; | |||
|
|||
use testapi; | |||
use utils qw(addon_decline_license assert_screen_with_soft_timeout); | |||
use version_utils qw(is_sle sle_version_at_least is_sle12_hdd_in_upgrade); | |||
use version_utils qw(is_sle is_sle is_sle12_hdd_in_upgrade); |
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.
duplicit is_sle
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, thanks
push(@addons_with_license, 'sdk') unless sle_version_at_least('15'); | ||
my @addons_with_license = qw(ha geo we live rt idu ids lgm hpcm ses); | ||
# Development tools and Web-Scripting modules do not have license in SLE 15 | ||
push(@addons_with_license, 'sdk') unless 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.
just for info (no change needed) - using if is_sle('<15')
is sometimes safer in case you share code with opensuse because it also checks if distri=sle
lib/registration.pm
Outdated
@@ -567,7 +567,7 @@ sub fill_in_reg_server { | |||
sub scc_deregistration { | |||
my (%args) = @_; | |||
$args{version_variable} //= 'VERSION'; | |||
if (sle_version_at_least('12-SP1', version_variable => $args{version_variable})) { | |||
if (is_sle('12-SP1+', version_variable => $args{version_variable})) { |
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.
there is no version_variable
argument in is_sle function (not implemented yet) ...this won't work :(
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.
Ouch, then I'll partially revert this sle_version_at_least changes. Thanks!
On SLE 15, we don't have any license for Web and Scripting module, but expect it. See [poo#33943](https://progress.opensuse.org/issues/33943).
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.
Feel free to merge after travis
On SLE 15, we don't have any license for Web and Scripting module, but
expect it.
See poo#33943.
Verification run.