[Bug 1095662] Review Request: perl-Sys-Detect-Virtualization - Detect if a UNIX system is running as a virtual machine

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

 



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



--- Comment #2 from David Dick <ddick@xxxxxxxx> ---
(In reply to Petr Šabata from comment #1)
> Missing BRs:
> perl(Config)
> perl(ExtUtils::MM_Unix)

Done.

> Your package always run-requires Parse::DMIDecode; you want to remove line
> 41.

*ouch* Done.

> Are the explicit run-requires in the virtdetect package necessary?

It was the only way i knew to remove the requires for Getopt::Long, POSIX, etc
from perl-Sys-Detect-Virtualization (where they are not required) and keep them
in virtdetect (where they are).  Suggestions?

> It might not be obvious to the user that virtdetect is a subpackage of this
> one; maybe packaging the README in its own %doc would be a good idea?

Okay.  Along the same sort of line, i thought including the Changelog would be
useful too?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
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]