-
Notifications
You must be signed in to change notification settings - Fork 258
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
ULC: build the universal linux client also for rpm based OS. #1824
base: master
Are you sure you want to change the base?
Conversation
33d2195
to
678c4fa
Compare
930aff9
to
c77727c
Compare
.matrix.yml
Outdated
IMAGE: "rhel8" | ||
BUILD_SCRIPT: CD/rpm/build-ulc.sh | ||
FINISH_SCRIPT: CD/rpm/finish-ulc.sh | ||
CUSTOM_TEST_IMAGES: [ "rhel8", "rhel9" ] |
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.
CUSTOM_TEST_IMAGES: [ "rhel8", "rhel9" ] | |
CUSTOM_TEST_IMAGES: [ Rocky-8, Fedora-36, Fedora-37, SLE-15_sp6 ] |
7cfb674
to
aa56690
Compare
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.
we need to address a few things in the spec to make it work cross-distro (i.e. on SUSE and maybe others, too)
%if 0%{?suse_version} | ||
Requires(pre): shadow | ||
%else | ||
Requires(pre): shadow-utils | ||
%endif |
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.
we cannot have distro-dependant Requires in the ULC package
%if 0%{?suse_version} | ||
Requires(pre): shadow | ||
%else | ||
Requires(pre): glibc-common | ||
Requires(pre): shadow-utils | ||
%endif |
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 here, dependencies cannot depend on the distro
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.
I think we can remove the requirement on glibc-common and replace the shadow/shadow-utils requirement with
Requires: /usr/sbin/useradd
Requires: /usr/sbin/groupadd
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.
maybe it makes sense to replace the other package dependencies (i.e. sed, gawk, grep, etc.) by program dependencies, too
fe0adb5
to
3c96a1b
Compare
3c96a1b
to
2ae6e33
Compare
Thank you for contributing to the Bareos Project!
Please check
If you have any questions or problems, please give a comment in the PR.
Helpful documentation and best practices
Checklist for the reviewer of the PR (will be processed by the Bareos team)
Make sure you check/merge the PR using
devtools/pr-tool
to have some simple automated checks run and a proper changelog record added.General
Source code quality
Tests