MacChen01 commented on code in PR #1143:
URL: https://github.com/apache/bigtop/pull/1143#discussion_r1276989580
##########
bigtop-packages/src/rpm/ranger/SPECS/ranger.spec:
##########
@@ -120,7 +120,10 @@ Requires: coreutils, insserv
# CentOS 5 does not have any dist macro
# So I will suppose anything that is not Mageia or a SUSE will be a
RHEL/CentOS/Fedora
-%if %{!?suse_version:1}0 && %{!?mgaversion:1}0
+# using the openeuler-lsb replace redhat-lsb in openEuler
+%if 0%{?openEuler}
+Requires: coreutils, openeuler-lsb
+%elif %{!?suse_version:1}0 && %{!?mgaversion:1}0
Review Comment:
1. I got a error using **ranger 2.4.0** when smoke test in
fedora-36/Rockylinux/openEuler, the error log is:
2. The cause is python3 support, the code is:
ranger-release-ranger-2.4.0\security-admin\scripts\db_setup.py
Ranger fixed the bug by the commit, and ranger master branch toke it.
https://github.com/apache/ranger/commit/f6b69ca5846de9a7dfced70ae8e403388ecb89fe
3. With the commit
https://github.com/apache/ranger/commit/f6b69ca5846de9a7dfced70ae8e403388ecb89fe,
smoke test is successful in openEuler/Fedora-36/Rockylinux, the log is:
**_openEuler smoketest ok_**
**_Fedora-36 smoketest ok_**
**_Rockylinux smoketest ok_**
4. The patch we can push it after you review and agree.
@guyuqi
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
