-
Notifications
You must be signed in to change notification settings - Fork 278
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 kernel-default-devel for sle-micro #20436
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -294,6 +294,17 @@ sub setup_network { | |
} | ||
} | ||
|
||
sub install_ltp_extra_dep_package { | ||
if (is_transactional) { | ||
if ((get_var('FLAVOR', '') =~ /Base-RT-Updates|Base-RT|Base-RT-encrypted/)) { | ||
zypper_install_available("kernel-rt-devel"); | ||
} | ||
else { | ||
zypper_install_available("kernel-default-devel"); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. devel package should be installed by default as dependency, so not need to install it in this way for normal kernel-default and kernel-rt has some other poo to fix ltp build for rt flavor. i think we dont need this PR at all There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For kernel-default-devel: For kernel-rt: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is not specifically about kernel-rt but about adding multibuild support to ltp package in ibs/obs to have specific ltp version build against particular kernel. Test should work fine on kernel-default and just skip it on rt flavor for now. |
||
} | ||
} | ||
} | ||
|
||
sub run { | ||
my $self = shift; | ||
my $inst_ltp = get_var 'INSTALL_LTP'; | ||
|
@@ -363,6 +374,8 @@ sub run { | |
} | ||
} | ||
|
||
install_ltp_extra_dep_package(); | ||
|
||
log_versions 1; | ||
|
||
zypper_call('in efivar') if is_sle('12+') || is_opensuse; | ||
|
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.
are you aware about
install_package
from package utils?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.
No idea when i create PR, but now i got it, thanks :)