Overview
Request 613094 accepted
- Created by markoschandras
- In state accepted
- Supersedes 612948
+%if 0%{?sle_version} || 0%{?suse_version}
=> if %?suse_version is sufficent: there is no product with a sle_version that does not also define a suse_version
Thank you for the suggestion. I will fix it in the next iteration if that's OK with you.
Why next? This weird construct is being introduced in this submission only. Cleaning it up right before it hits the product(s) sounds better
OK no problem. I was planning to clean it up from SLE first and then sync with Factory but I can do it the other way around
I think we now have upstream macros like %systemd_post in suse too.
I can have a look at it as well in the next iteration but I also need to make sure these macros work in maintained SLE distros so we don't deviate.
Request History
markoschandras created request
licensedigger accepted review
ok
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
staging-bot added openSUSE:Factory:Staging:adi:12 as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:12"
staging-bot accepted review
Picked openSUSE:Factory:Staging:adi:12
repo-checker accepted review
cycle and install check passed
dimstar accepted review
staging-bot accepted review
ready to accept
staging-bot approved review
ready to accept
dimstar_suse accepted request
Accept to openSUSE:Factory
