Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 186811 - Review Request: libnfnetlink - Netfilter netlink userspace library
Summary: Review Request: libnfnetlink - Netfilter netlink userspace library
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 186887 186892
TreeView+ depends on / blocked
 
Reported: 2006-03-26 19:18 UTC by Paul P Komkoff Jr
Modified: 2011-11-28 18:35 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-11 13:19:24 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Paul P Komkoff Jr 2006-03-26 19:18:39 UTC
Spec Name or Url: http://stingr.net/l/fe/libnfnetlink.spec
SRPM Name or Url: http://stingr.net/l/fe/libnfnetlink-0.0.14-1.src.rpm
Description:
libnfnetlink is a userspace library that provides some low-level
nfnetlink handling functions.  It is used as a foundation for other, netfilter
subsystem specific libraries such as libnfnetlink_conntrack, libnfnetlink_log
and libnfnetlink_queue.

Comment 1 Paul P Komkoff Jr 2006-03-26 20:14:30 UTC
This is the 1st prerequisite to conntrack userspace tool. Second will be
libnfnetlink_conntrack.

Comment 2 Jochen Schmitt 2006-03-28 18:44:54 UTC
Good:
+ Local build works.
+ rpmlint has not complaints for SRPM.
+ Mock build worked fine.

Bad:
- rpmlint complaints on libnfnetlink:
  E: libnfnetlink library-without-ldconfig-postin /usr/lib/libnfnetlink.so.0.0.0
  E: libnfnetlink library-without-ldconfig-postun /usr/lib/libnfnetlink.so.0.0.0
- rpm don't contains verbatin copy of the license.
- devel rpm contains static libraries.

Comment 3 Paul P Komkoff Jr 2006-03-28 21:14:59 UTC
I've updated spec and srpm wrt ldconfig and --disable-static.
And wrt license text - original distribution don't contain it too.

Comment 4 Stefan Neufeind 2006-04-01 21:21:22 UTC
Unfortunately I can't judge about the spec in detail. But I did build and try
the three related packages (libnfnetlink, libnetfilter_conntrack and conntrack).
They did build and install fine, conntrack seems to work fine as well.

Comment 5 Paul Howarth 2006-04-25 15:35:49 UTC
Please bump the release number with package revision, even during review. The
SRPM at the posted URL does not contain the same spec file as the one at the
posted spec URL (still missing ldconfig scriptlets).

Comment 7 Jochen Schmitt 2006-04-26 18:15:22 UTC
Please take the license text from www.gnu.org an poke the upstream to include a
written license text.

Comment 8 Paul P Komkoff Jr 2006-04-26 20:04:20 UTC
Is it serious, real, critical showstopper? It wil enhance review period for
another couple of weeks because of "upstream" being on vacation.

Comment 9 Paul Howarth 2006-04-27 07:48:44 UTC
(In reply to comment #8)
> Is it serious, real, critical showstopper? It wil enhance review period for
> another couple of weeks because of "upstream" being on vacation.

No, it's not a blocker. It's a "SHOULD" in the package review guidelines
(http://fedoraproject.org/wiki/Packaging/ReviewGuidelines).


Comment 10 Jochen Schmitt 2006-04-27 15:38:25 UTC
You may be able to download the current CPL text from 
http://www.gnu.org/licenses/gpl.txt and add is to your package.

Then you may poke the upstream to include a verbatin copy of the license in 
the upstream package.

When this may be happen, you can create an updated version of your package, 
which use the text from the upstream package instead of the text from 
www.gnu.org

Comment 11 Stefan Neufeind 2006-05-08 11:44:27 UTC
So to my understanding there is nothing really preventing a release in extras,
since the gpl-text can be added to the package without it being upstream as well
for now. Or is there something I could do to actually move this topic (release
in extras) forward?

Comment 12 Paul Howarth 2006-05-08 16:42:54 UTC
(In reply to comment #11)
> So to my understanding there is nothing really preventing a release in extras,
> since the gpl-text can be added to the package without it being upstream as well
> for now. Or is there something I could do to actually move this topic (release
> in extras) forward?

Jochen is asking you to include http://www.gnu.org/licenses/gpl.txt as an
additional source file, to include this file as %doc in your spec, and also to
try to get upstream to include a copy of the license text distributed with their
source. Whilst there is no requirement in the package review guidelines for the
license text to be packaged if upstream do not provide it, and poking upstream
is only a *should* rather than a *must* in the guidelines, if you want to get
this package approved sooner rather than later, I'd do as Jochen asks.

Comment 14 Jochen Schmitt 2006-05-10 17:57:07 UTC
I'm happy to APPOROVE your package.

Comment 15 Christian Iseli 2006-12-31 00:13:33 UTC
(In reply to comment #14)
> I'm happy to APPOROVE your package.

Please do block FE-ACCEPT when you approve a package.
Thanks.


Comment 16 Paul P Komkoff Jr 2009-10-13 12:22:43 UTC
Package Change Request
======================
Package Name: libnfnetlink
New Branches: EL-4 EL-5
Owners: stingray jrussek

Comment 17 Kevin Fenzi 2009-10-13 16:35:02 UTC
jrussek doesn't seem to be in the packager group. 

Please add a new request and reset the flag?

Comment 18 Paul P Komkoff Jr 2011-11-27 22:19:43 UTC
Package Change Request
======================
Package Name: libnfnetlink
New Branches: EL-6
Owners: stingray jrussek

Comment 19 Gwyn Ciesla 2011-11-28 13:10:45 UTC
jrussek still isn't in the packager group.  What Kevin meant was submit a new request, either after jrussek is in the packager group, or without jrussek.

Comment 20 Paul P Komkoff Jr 2011-11-28 18:22:37 UTC
Package Change Request
======================
Package Name: libnfnetlink
New Branches: EL-4 EL-5 EL-6
Owners: stingray

Comment 21 Gwyn Ciesla 2011-11-28 18:35:54 UTC
Git done (by process-git-requests).


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