Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=953379 Bug ID: 953379 Summary: Review Request: tipcutils - Utils package required to configure TIPC Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: unspecified Assignee: nobody@xxxxxxxxxxxxxxxxx Reporter: lsm5@xxxxxxxxxxxxxxxxx QA Contact: extras-qa@xxxxxxxxxxxxxxxxx CC: erik.hugne@xxxxxxxxxxxx, lsm5@xxxxxxxxxxxxxxxxx, misc@xxxxxxxx, notting@xxxxxxxxxx, package-review@xxxxxxxxxxxxxxxxxxxxxxx Depends On: 854256 Blocks: 177841 (FE-NEEDSPONSOR) Category: --- +++ This bug was initially created as a clone of Bug #854256 +++ Spec URL: http://marvin.hb0da.org/~dev/tipcutils.spec SRPM URL: http://marvin.hb0da.org/~dev/tipcutils-2.0.3-0.src.rpm Description: Hi. Fedora seems to be missing a package containing the userspace tools necessary to configure TIPC (/net/tipc). This package contains the tipc-config program, and also tipc-pipe, which is a netcat-like program that runs over TIPC links. Fedora Account System Username: ehugne --- Additional comment from Michael Scherer on 2012-09-06 01:59:41 EDT --- Hi, since you are not in the packager group, you need a sponsor first http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group On your spec, there is various issue with it : - it doesn have a changelog - you should use macro for path, rather than for well know executable - BuildArch: x86_64 is wrong, unless there is a reason and then it should be explained - %{_builddir}/%{name}-%{version}/configure you should use the macro %configure, or if it doesn't work, ./configure - the license file should be shipped as %doc - BuildRoot is not needed, %clean eithe,r as seen on https://fedoraproject.org/wiki/Packaging:Guidelines#.25clean ( the rest of the rules are also on this page, so I recommend to read it carefully ) There is surely other stuff to fix, but I do not want to overwhelm you for the first comment :) --- Additional comment from Erik Hugne on 2012-09-07 10:47:23 EDT --- Thanks for the prompt feedback. I have adressed your comments, and have one open question regarding the license. Is it required to have this as a separate file? All files in tipcutils have the license prepended to them. --- Additional comment from Erik Hugne on 2012-09-24 05:07:08 EDT --- The srpm/spec links still from the first request still apply: Spec URL: http://marvin.hb0da.org/~dev/tipcutils.spec SRPM URL: http://marvin.hb0da.org/~dev/tipcutils-2.0.3-0.src.rpm --- Additional comment from Lokesh Mandvekar on 2013-04-13 02:08:13 EDT --- Hi Erik, Well, first off thanks for helping me use TIPC 2.0 with UDP bearers sometime back. Are you still interested in proceeding with this review request? If you are (and I'm hoping you would be), I'd like to help with the package review, coz this program is useful to me too. I'm not a sponsor and I'm in process of getting my first package reviewed. * The Release number needs to be incremented everytime you make a change to the spec file, along with a Changelog entry for it. * Fedora requires that files not be directly installed to /sbin. /usr/sbin is preferable in this case. Check this out: https://fedoraproject.org/wiki/Packaging:Guidelines#Filesystem_Layout * In the %files section, use macros for directory names (for eg. %{_bindir} instead of /usr/bin) * Run rpmlint on the spec, srpm and binary rpm files and post their outputs. * Run the srpm file through koji and provide a link to the build here. Koji HOWTOs: https://fedoraproject.org/wiki/Using_the_Koji_build_system?rd=Koji/UsingKoji * The Source URL isn't accessible anymore. Could you please check that out? There might be other issues (but I'm sorta new to reviewing myself, and will revisit this as and when I learn new stuff). --- Additional comment from Lokesh Mandvekar on 2013-04-13 02:13:03 EDT --- macros to be used in %files: https://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25files_section --- New Request --- I would like to assume maintainership of this package and have already talked to the upstream developers about this. FAS username: lsm5 Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/tipcutils.spec SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/tipcutils-2.0.5-1.fc20.src.rpm Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5269869 $ rpmlint SPECS/tipcutils.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint SRPMS/tipcutils-2.0.5-1.fc20.src.rpm tipcutils.src: W: spelling-error %description -l en_US userspace -> user space, user-space, users pace 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint RPMS/x86_64/tipcutils-2.0.5-1.fc20.x86_64.rpm tipcutils.x86_64: W: spelling-error %description -l en_US userspace -> user space, user-space, users pace 1 packages and 0 specfiles checked; 0 errors, 1 warnings. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bWhFnrWJ0K&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review