[Bug 194566] Review Request: 915resolution

bugzilla at redhat.com bugzilla at redhat.com
Tue Aug 1 22:48:24 UTC 2006


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

Summary: Review Request: 915resolution


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





------- Additional Comments From cweyl at alumni.drew.edu  2006-08-01 18:39 EST -------
(In reply to comment #10)
> Bad
> ---
> 
> It really does need to be enclosed in a wrapper with an example script for use
> in %{_sysconfdir} - as it stands, it's one of those applications that you
> install and wonder why you did - at least with something in %{_sysconfdir} users
> will know what to do (or where to look!)
>
> Fix the issue in BAD and I'm happy to approve it.

Is hdparm a service?  They both perform the same sort of function -- run once at
boot to tweak various interface settings of a physical device.

If someone has installed this package, then either they know what they're doing
(and can find README.fedora), or they don't and this package shouldn't try to do
anything with their system.  If they do, they're still going to have to
configure it based on the particulars of their hardware, so the amount of
user-work is the same either way.  The package-supplied README is quite detailed
in the particulars of the proper use of the program.

A full-blown sysv-style service wrapper around would turn this very simple
package into a much more complex beast, with addtional scripts, documentation,
additional bits in the spec, etc, than it warrants.  It introduces additional
complexity at a net of questionable benefit -- especially when we consider that
this package, while useful to those who need its functionality _now_, is
predestined to have a very limited lifespan.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list