Overview
I've send an annoucement mail to factory about this package https://lists.opensuse.org/opensuse-factory/2018-07/msg00193.html
@agraul Just build against :Update for Leap/SLE and you can use %license everywhere.
As he builds just the same as is in the project utilities, not much of his choice there
+%if ( 0%{?suse_version} == 1315 && 0%{?sle_version} <= 120200 ) || ! 0%{?is_opensuse}
That would also negatively impact SLE15 packages
Either use only %license
or make the condition less error prone like this:
%if 0%{?suse_version} >= 1500 %license LICENSE.md %else %doc LICENSE.md %endif
+# MANUAL BEGIN +Requires: bash >= 3.2 +# MANUAL END
This should not be needed - by having a shebang pointing to /bin/bash
(which you have, due to fix-shebang.patch), the rpm get's an automatic dependency on /bin/bash - which in turn is provided by the package bash (the only downside there being it not versioned, but openSUSE used bash 4.0 since 2009; so I'd say this is safe to be ignored)
Request History
agraul created request
licensedigger accepted review
ok
staging-bot added openSUSE:Factory:Staging:adi:54 as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:54"
staging-bot accepted review
Picked openSUSE:Factory:Staging:adi:54
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto added repo-checker as a reviewer
Please review build success
factory-auto accepted review
Check script succeeded
repo-checker accepted review
cycle and install check passed
mrdocs accepted review
Thanks!
staging-bot accepted review
ready to accept
staging-bot approved review
ready to accept
dimstar_suse accepted request
Accept to openSUSE:Factory