Bug 1264482 - Review Request: compat-nettle27 - Compat package with nettle libraries
Summary: Review Request: compat-nettle27 - Compat package with nettle libraries
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Pete Walter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-09-18 14:56 UTC by Kalev Lember
Modified: 2015-09-19 13:08 UTC (History)
2 users (show)

Fixed In Version: compat-nettle27-2.7.1-1.fc23
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-09-19 13:08:25 UTC
Type: ---
Embargoed:
walter.pete: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Kalev Lember 2015-09-18 14:56:17 UTC
Spec URL: https://kalev.fedorapeople.org/compat-nettle27.spec
SRPM URL: https://kalev.fedorapeople.org/compat-nettle27-2.7.1-1.fc22.src.rpm
Description:
This is an ABI compatibility package for 3rd party programs that don't follow Fedora release schedule closely. The intention is to keep this package during the F23 lifetime to give 3rd parties time to rebuild their stuff, and retire and obsolete it in F24.

Fedora Account System Username: kalev

Comment 1 Kalev Lember 2015-09-18 14:58:13 UTC
For reference, here's the diff to the F22 nettle package: http://paste.fedoraproject.org/268945

I'd like to keep the diff to the original package down to the minimum in order to be able to easily share fixes when needed, so if possible please don't pick on the indentation and things like that that are also present in the original F22 package.

Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=11137312

Comment 2 Pete Walter 2015-09-18 16:15:17 UTC
Looks good to me, except for two small things I noticed:

1) this comment,

# Explicitly conflict with older libvpx packages that ship libraries
# with the same soname as this compat package

should read "nettle" instead of "libvpx" here.

2) if you want you can remove "Group: Development/Libraries" line (both from the compat package and regular nettle) as nothing in Fedora uses group tags any more.

Comment 3 Pete Walter 2015-09-18 16:24:30 UTC
Anyway, the two issues above are minor and don't block the review. Feel free to fix them before importing if you want.

I am following https://fedoraproject.org/wiki/Package_Renaming_Process here and won't do a full review.

- rpmlint output:

compat-nettle27.src:10: W: macro-in-comment %{version}
compat-nettle27.src:9: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9)
compat-nettle27.src: W: invalid-url Source0: nettle-2.7.1-hobbled.tar.gz
compat-nettle27.x86_64: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 4 warnings.

- licensing: OK, matches nettle
- naming: OK
- package installs correctly in Fedora 23 side by side with nettle
- Obsoletes and Provides: not required in this case
- Conflicts use looks right to me

fedora-review has nothing interesting to say and I won't post its output here.

Everything seems fine to me, so the package is:

APPROVED

Comment 4 Kalev Lember 2015-09-18 16:49:47 UTC
Thanks for the quick review, Pete!

New Package SCM Request
=======================
Package Name: compat-nettle27
Short Description: Compat package with nettle libraries
Upstream URL: http://www.lysator.liu.se/~nisse/nettle/
Owners: kalev
Branches: f23
InitialCC:

Comment 5 Gwyn Ciesla 2015-09-19 12:36:24 UTC
Git done (by process-git-requests).

Comment 6 Kalev Lember 2015-09-19 13:08:25 UTC
Package imported and built.


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