[Bug 744340] Review Request: targetcli - Configuration shell for kernel target subsystem

bugzilla at redhat.com bugzilla at redhat.com
Fri Nov 4 12:23:49 UTC 2011


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=744340

--- Comment #9 from Tomasz Torcz <zdzichu at irc.pl> 2011-11-04 08:23:48 EDT ---
Created attachment 531753
  --> https://bugzilla.redhat.com/attachment.cgi?id=531753
/etc/modules-load.d/targetcli.conf

Two little nitpicks:
1) you don't need mkdir -p %{buildroot}%{_sysconfdir}/rc.d/init.d in spec
anymore
2) module loading from scripts should be avoided, and done by dropping list of
modules in /etc/modules-load.d/ dir.

But those are really details. Apart from this, thanks for packaging this!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the package-review mailing list