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 1089110 - Review Request: nield - Network Interface Events Logging Daemon
Summary: Review Request: nield - Network Interface Events Logging Daemon
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-NEEDSPONSOR
TreeView+ depends on / blocked
 
Reported: 2014-04-18 01:32 UTC by Tetsumune KISO
Modified: 2020-01-19 14:26 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-01-19 14:26:59 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Tetsumune KISO 2014-04-18 01:32:12 UTC
Spec URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield-0.5.0-1.fc20.spec
SRPM URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield-0.5.0-1.fc20.src.rpm
Description: nield(Network Interface Events Logging Daemon) is a tool that receives notifications from the kernel through the netlink socket, and generates logs related to link state, neighbor cache(ARP,NDP), IP address(IPv4,IPv6), route, FIB rules, traffic control.
Fedora Account System Username: t2mune

Comment 1 Christopher Meng 2014-04-18 03:07:16 UTC
FAS username invalid.

SPEC name invalid.

Use your real name in bugzilla.

Read carefully:

https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group

Comment 2 Christopher Meng 2014-04-18 03:12:31 UTC
SPEC file:

1. Packager: xxxx <txxx>

Remove that.

2. ExclusiveArch: %{ix86} x86_64

Why?

3. ${RPM_BUILD_ROOT}

Bad.

See https://fedoraproject.org/wiki/Packaging:Guidelines#Using_.25.7Bbuildroot.7D_and_.25.7Boptflags.7D_vs_.24RPM_BUILD_ROOT_and_.24RPM_OPT_FLAGS

4. cp contrib/fedora/nield.service ${RPM_BUILD_ROOT}/%{_unitdir}

Use install -pm644 to do that.

5. %clean

Remove this section please.

6. %files
/usr/sbin/nield

/usr/sbin --> %{_sbindir}

7. %doc /usr/share/man/man8/nield.8.gz

No %doc for manpages please.

Also list manpages by:

%{_mandir}/man8/nield.8*

8. * Thu Apr 17 2014 t2mune - 0.5.0-1

Who are you? Name? Email address? I assume you are not a troll.

Comment 3 Christopher Meng 2014-04-18 03:14:21 UTC
Also,

9. [ -d ${RPM_BUILD_ROOT}/%{_unitdir} ] || mkdir -p ${RPM_BUILD_ROOT}/%{_unitdir}

You don't need to use script here, just

mkdir -p %{buildroot}%{_unitdir}

is enough.

Comment 4 Tetsumune KISO 2014-04-18 04:36:44 UTC
Hi Christopher,

Thank you for your comments.
I've posted a new version, taking account of them:

Spec URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield.spec
SRPM URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield-0.5.0-1.fc20.src.rpm
Description: nield(Network Interface Events Logging Daemon) is a tool that receives notifications from the kernel through the netlink socket, and generates logs related to link state, neighbor cache(ARP,NDP), IP address(IPv4,IPv6), route, FIB rules, traffic control.
Fedora Account System Username: t2mune

Comment 5 Volker Fröhlich 2014-04-18 06:08:44 UTC
Please use spaces after parentheses!

License is GPLv2+, run licensecheck on the source code to see why! As you are the developer, you should state the license in every file, not just a random few.

The changelog format is not correct yet: https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs

Use the version macro in the Source line. You can also use the name macro there.

You must include %doc COPYING in the files section of your spec file and I recommend adding "ChangeLog" as well.

Comment 6 Volker Fröhlich 2014-04-18 06:11:54 UTC
I meant _before_!

Comment 7 Tetsumune KISO 2014-04-18 09:37:48 UTC
Hi Volker,

Tyank you for comments.

> Please use spaces after parentheses!
Could you give me more details?

Comment 8 Volker Fröhlich 2014-04-18 09:49:15 UTC
Oh, just "Blah blah (something)" instead of "Blah blah(something)".

Comment 9 Tetsumune KISO 2014-04-18 11:35:46 UTC
I uploaded new files:

Spec URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield.spec
SRPM URL: http://sourceforge.net/projects/nield/files/nield-0.5.0/nield-0.5.0-1.fc20.src.rpm
Description: nield (Network Interface Events Logging Daemon) is a tool that receives notifications from the kernel through the netlink socket, and generates logs related to link state, neighbor cache (ARP,NDP), IP address (IPv4,IPv6), route, FIB rules, traffic control.
Fedora Account System Username: t2mune

Comment 10 Tetsumune KISO 2014-04-19 17:51:41 UTC
(In reply to Volker Fröhlich from comment #5)
> License is GPLv2+, run licensecheck on the source code to see why! As you
> are the developer, you should state the license in every file, not just a
> random few.

This package include a part of files in linux-headers (GPL),
but there are no mention of a license and copyright in some files.
What would be the best thing to do?

Comment 11 Christopher Meng 2014-04-20 12:30:16 UTC
Please also follow this carefully:

https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

Comment 12 Tetsumune KISO 2014-04-28 23:38:18 UTC
Updated so as not to include kernel header files.
Could you review them?

Spec URL: http://sourceforge.net/projects/nield/files/nield-0.5.1/fedora/nield.spec
SRPM URL: http://sourceforge.net/projects/nield/files/nield-0.5.1/fedora/nield-0.5.1-1.fc20.src.rpm
Description: The Network Interface Events Logging Daemon is a tool that receives
notifications from the kernel through the netlink socket, and generates logs
related to link state, neighbor cache (ARP,NDP), IP address (IPv4,IPv6), route,
FIB rules, traffic control.
Fedora Account System Username: t2mune

And I ran licensecheck and rpmlint.
"netlink" is not a spelling error.

[t2mune@fedora20 ~]$ licensecheck nield-0.5.1/*
nield-0.5.1/src/tcmsg_filter.c: GPL (v2 or later)
nield-0.5.1/src/debug.c: GPL (v2 or later)
nield-0.5.1/src/log.c: GPL (v2 or later)
nield-0.5.1/src/ndmsg.c: GPL (v2 or later)
nield-0.5.1/src/nlmsg.c: GPL (v2 or later)
nield-0.5.1/src/rtmsg.c: GPL (v2 or later)
nield-0.5.1/src/nield.h: GPL (v2 or later)
nield-0.5.1/src/nield.c: GPL (v2 or later)
nield-0.5.1/src/tcmsg_qdisc.c: GPL (v2 or later)
nield-0.5.1/src/frhdr.c: GPL (v2 or later)
nield-0.5.1/src/ifimsg.c: GPL (v2 or later)
nield-0.5.1/src/ifamsg.c: GPL (v2 or later)
nield-0.5.1/src/list.h: GPL (v2 or later)
nield-0.5.1/src/rtnetlink.h: GPL (v2 or later)
nield-0.5.1/src/tcamsg.c: GPL (v2 or later)
[t2mune@fedora20 ~]$ rpmlint rpmbuild/SPECS/nield.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[t2mune@fedora20 ~]$ rpmlint rpmbuild/SRPMS/nield-0.5.1-1.fc20.src.rpm 
nield.src: W: spelling-error %description -l en_US netlink -> net link, net-link, linnet
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Comment 13 Christopher Meng 2014-04-29 00:23:15 UTC
Read carefully:

https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

Find which step you forgot to follow, then do it.

Comment 14 Volker Fröhlich 2015-09-23 12:12:20 UTC
Tetsumune, any progress concerning a sponsorship?

Comment 15 Tetsumune KISO 2015-09-24 12:30:57 UTC
No progress.

Comment 16 Volker Fröhlich 2020-01-19 14:26:59 UTC
No activity in 4 years, closing!


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