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@xxxxxxxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review