Overview

Request 1043106 accepted

- avoid file conflict on SLE15 and other older distributions


Dominique Leuenberger's avatar

17+%if %{?suse_version} && %{suse_version} > 1500

three errors in one line:

  • %if %{?suse_version} => if suse_version is not defined, you get an error (invalid syntax)
  • %{suse_version} > 1500 if suse_version is not defined, this is still being 'expanded' (well, not) and you get a string vs 1500 comparison
  • %{?suse_version} && %{suse_version} > 1500 in what case can suse_version be > 1500 and not be defined?

What you actually wanted to use is %if 0%{?suse_version} > 1500

Request History
Dirk Mueller's avatar

dirkmueller created request

- avoid file conflict on SLE15 and other older distributions


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar_suse added openSUSE:Factory:Staging:adi:35 as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:adi:35"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked "openSUSE:Factory:Staging:adi:35"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:adi:35 got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:adi:35 got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:adi:35 got accepted.

openSUSE Build Service is sponsored by