Bug 2041246 - Review Request: kalk - kalk is a convergent calculator for Plasma.
Summary: Review Request: kalk - kalk is a convergent calculator for Plasma.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-01-16 18:37 UTC by Onuralp SEZER
Modified: 2023-03-18 12:24 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-03-18 12:24:00 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Onuralp SEZER 2022-01-16 18:37:26 UTC
Spec URL: https://pagure.io/kalk_rpm/raw/master/f/kalk.spec
SRPM URL: https://pagure.io/kalk_rpm/blob/master/f/kalk-21.12-1.fc35.src.rpm
Description: kalk is a convergent calculator for Plasma.
Fedora Account System Username: thunderbirdtr

Comment 1 Neal Gompa 2022-01-22 17:46:29 UTC
Initial spec review:

> %description
> %{summary}

This should be "%{summary}." in the description simply to make rpmlint shut up.

> %prep
> %autosetup -n %{name}-%{version}

"-n %{name}-%{version}" is superfluous, since that's the default. Drop it.

Also, license files are not captured with a "%license" stanza. Add "%license LICENSES/*" to the file list.

Comment 2 Justin Zobel 2022-02-06 00:40:01 UTC
MR added.

Comment 3 Onuralp SEZER 2022-02-06 06:32:51 UTC
Spec file updated.

(In reply to Neal Gompa from comment #1)
> Initial spec review:
> 
> > %description
> > %{summary}
> 
> This should be "%{summary}." in the description simply to make rpmlint shut
> up.
> 
> > %prep
> > %autosetup -n %{name}-%{version}
> 
> "-n %{name}-%{version}" is superfluous, since that's the default. Drop it.
> 
> Also, license files are not captured with a "%license" stanza. Add "%license
> LICENSES/*" to the file list.

Comment 4 Justin Zobel 2022-03-11 00:56:55 UTC
Passes fedora-review but needs licenses added, have notified Onuralp.

Comment 5 Neal Gompa 2022-03-14 22:42:53 UTC
Everything looks gravy, so...

PACKAGE APPROVED.

Comment 6 Tomas Hrcka 2022-03-15 07:01:04 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/kalk

Comment 7 Package Review 2023-03-18 12:24:00 UTC
Package is now in repositories, closing review.


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