Overview

Request 718735 superseded

submit o2locktop package for inital version

Loading...

Dominique Leuenberger's avatar

+%doc COPYING => must be %license COPYING (possibly not for SLE11, as that was not supported there)


Gang He's avatar
author source maintainer

Hello Dimstar, how do I change this part code for handle all cases? The diff looks like?

-%doc COPYING README.md DESIGN.md

%if 0%{?suse_version} >= 1200 %license COPYING %else %doc COPYING %endif %doc README.md DESIGN.md

right?

Thanks Gang


Dominique Leuenberger's avatar

yes, that should be all right


Gang He's avatar
author source maintainer

Thanks, please see the new request https://build.opensuse.org/request/show/718769


Gang He's avatar
author source maintainer

The diff looks like, -%doc COPYING README.md DESIGN.md +%if 0%{?suse_version} >= 1200 +%license COPYING +%else +%doc COPYING +%endif +%doc README.md DESIGN.md


Dominique Leuenberger's avatar

+BuildRequires: python

Do we really want to introduce new python2 code? Py2 is dead.. please port this application to python3


Gang He's avatar
author source maintainer

Hello Dimstar,

Since we want to use o2locktop(only python scripts) in different distributions (from SLE11sp4 to SLE12/SLE15, and TW), we should depend on Python or other words? as you know, in some distributions, there is only python2?

Thanks Gang


Dominique Leuenberger's avatar

IIRC, SLE11 had no python3 yet - SLE12 and up have py3

Python2 is declared end-of-life per Jan 1, 2020 (so less than 6 months)

See https://pythonclock.org/ (which is why I believe the TW package and wherever it is introduced with py3 available, must be py3.


Gang He's avatar
author source maintainer

Hello Dimstar, please see my new request https://build.opensuse.org/request/show/718735

Thanks Gang

Request History
Gang He's avatar

ganghe created request

submit o2locktop package for inital version


Staging Bot's avatar

staging-bot added as a reviewer

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


Staging Bot's avatar

staging-bot accepted review

Picked openSUSE:Factory:Staging:adi:28


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


Gang He's avatar

ganghe superseded request

superseded by 718769

openSUSE Build Service is sponsored by