Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 219056 - Review Request: iscsitarget - iSCSI Enterprise Target
Summary: Review Request: iscsitarget - iSCSI Enterprise Target
Status: CLOSED DUPLICATE of bug 197867
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2006-12-10 05:01 UTC by David Nalley
Modified: 2007-11-30 22:11 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-07-06 19:01:24 UTC

Attachments (Terms of Use)

Description David Nalley 2006-12-10 05:01:44 UTC
Spec URL:
Description: IET is software (specifically a kernel module, daemon, and management app) for building a iSCSI storage system on Linux) This is my first package, and I need a sponsor. I am sure there are a plethora of mistakes, please feel free to point out my problems. 

rpmlint on the application RPM complains about documentation permissions, (0644) but that makes sense to me for documentation. rpmlint also complains about lack of capitalization in the first letter of the package name, but what is present is correct for the package, though many places have to capitalize, so if I need to change that, please let me know. 

rpmlint on the kernel module complains that depmod is being run against something other than the running kernel, which appears to be incorrect (both from reading the spec file and seeing what was built and whats running.

Comment 1 David Nalley 2006-12-10 05:29:52 UTC
I missed adding the URL to source, just changed that new spec and srpm are: 
Spec URL:

Comment 2 Wilmer Jaramillo M. 2007-01-11 23:45:48 UTC
I look only some things, you should use the rpm directory macros %{_sbindir} and
%{_initrddir}[1] and the command 'ln' must create with '-sf' parameter for a
clean re-install output, you should delete of top the _docdir macro and the
Packager tag should not be used[2].


Comment 3 David Nalley 2007-01-31 20:22:33 UTC
Ok, fixed here:
Spec URL:

Thanks for the review! 

Comment 4 Marek Mahut 2007-06-10 19:19:37 UTC
- specfile is properly named, is cleanly written and uses macros consistently.
    put \n between each entry in %changelog

- description is not OK
    %description is too ample, rpmlint is complaining about
summary-not-capitalized, but it's ok as it is the name of the software

- permissions are not correct
   both iscsitarget.spec and iscsitarget-0.4.14.tar.gz are 777
   /usr/share/doc/iscsitarget-0.4.14 is 644, but this is ok

Comment 5 Jerry Amundson 2007-07-05 16:50:14 UTC
Perhaps this duplication has already been discussed?
Bugzilla Bug 197867: Review Request: iscsitarget (includes kernel module)

Comment 6 Jason Tibbitts 2007-07-06 19:01:24 UTC
I'm closing this as a duplicate, since that's what should have been done long
ago.  Perhaps the submitters could work together.

*** This bug has been marked as a duplicate of 197867 ***

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