Bug 2281543 - Review Request: kalm - Breathing techniques
Summary: Review Request: kalm - Breathing techniques
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL: https://apps.kde.org/kalm/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-05-19 19:11 UTC by Yaakov Selkowitz
Modified: 2024-05-30 00:33 UTC (History)
3 users (show)

Fixed In Version: kalm-24.05.0-1.fc41
Clone Of:
Environment:
Last Closed: 2024-05-30 00:33:27 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 7461871 to 7486931 (1.42 KB, patch)
2024-05-24 18:57 UTC, Fedora Review Service
no flags Details | Diff

Description Yaakov Selkowitz 2024-05-19 19:11:46 UTC
Spec URL: https://yselkowitz.fedorapeople.org/kalm.spec
SRPM URL: https://yselkowitz.fedorapeople.org/kalm-24.04.90-1.fc41.src.rpm
Description: Kalm can teach you different breathing techniques.
Fedora Account System Username: yselkowitz

Comment 1 Fedora Review Service 2024-05-19 19:44:04 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7461871
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2281543-kalm/fedora-rawhide-x86_64/07461871-kalm/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Steve Cossette 2024-05-19 19:49:56 UTC
Two suggestions, one mandatory and one superficial:

1- (Mandatory) You're missing licenses in there, according to reuse:

* Used licenses: BSD-2-Clause, CC-BY-SA-4.0, CC0-1.0, LGPL-2.1-or-later
(Those licenses are also in the project's LICENSES folder)

2- (Superficial) I think your e-mail is mistyped in the clog :)

Comment 3 Steve Cossette 2024-05-19 19:54:30 UTC
(In reply to Steve Cossette from comment #2)
> Two suggestions, one mandatory and one superficial:
> 
> 1- (Mandatory) You're missing licenses in there, according to reuse:
> 
> * Used licenses: BSD-2-Clause, CC-BY-SA-4.0, CC0-1.0, LGPL-2.1-or-later
> (Those licenses are also in the project's LICENSES folder)
> 
> 2- (Superficial) I think your e-mail is mistyped in the clog :)

Nevermind #2 :)

Comment 4 Yaakov Selkowitz 2024-05-19 19:57:12 UTC
(In reply to Steve Cossette from comment #2)
> 1- (Mandatory) You're missing licenses in there, according to reuse:
> 
> * Used licenses: BSD-2-Clause, CC-BY-SA-4.0, CC0-1.0, LGPL-2.1-or-later
> (Those licenses are also in the project's LICENSES folder)

Files that are used only for CI and/or build infrastructure are not installed and therefore their licenses need not be included.

Comment 5 Yaakov Selkowitz 2024-05-24 18:53:05 UTC
Spec URL: https://yselkowitz.fedorapeople.org/kalm.spec
SRPM URL: https://yselkowitz.fedorapeople.org/kalm-24.05.0-1.fc41.src.rpm
Description: Kalm can teach you different breathing techniques.
Fedora Account System Username: yselkowitz

Comment 6 Fedora Review Service 2024-05-24 18:57:05 UTC
Created attachment 2034992 [details]
The .spec file difference from Copr build 7461871 to 7486931

Comment 7 Fedora Review Service 2024-05-24 18:57:07 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7486931
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2281543-kalm/fedora-rawhide-x86_64/07486931-kalm/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 8 Neal Gompa 2024-05-30 00:07:31 UTC
Review notes:

* Package follows Fedora Packaging Guidelines
* Package builds and installs
* Package licensing is correctly handled and recorded
* No serious issues from rpmlint

PACKAGE APPROVED.

Comment 9 Fedora Admin user for bugzilla script actions 2024-05-30 00:09:03 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/kalm


Note You need to log in before you can comment on or make changes to this bug.