[Bug 669407] Review Request: ledmon - LED control app for Intel(R) storage controllers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=669407

Ondrej Vasik <ovasik@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ovasik@xxxxxxxxxx
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |ovasik@xxxxxxxxxx

--- Comment #2 from Ondrej Vasik <ovasik@xxxxxxxxxx> 2011-01-13 10:55:59 EST ---
I guess it was because of the "already existing dead project" with ledctl name
on sourceforge.
Personally, I think we could keep the package as ledctl (with %setup -n) ...
but if Jirka tends to rename the package, I'm not against it.

Few notes before real rereview:
Please remove the note about missing upstream URL, URL is now provided.

You moved ledctl from %_bindir to %_sbindir - is that intentional?... it may
break some scripts if someone already started with the %_bindir (I know, it is
VERY low probability, but possibly some wrapper with deprecation warning could
be considered).

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]