This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 1009742 - Review Request: perl-Term-Menu - print an interactive menu
Review Request: perl-Term-Menu - print an interactive menu
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
unspecified Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
http://search.cpan.org/~dazjorz/Term-...
:
Depends On:
Blocks: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2013-09-18 23:32 EDT by Chris
Modified: 2016-02-08 08:57 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2016-02-08 08:57:33 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)

  None (edit)
Description Chris 2013-09-18 23:32:06 EDT
Spec URL: https://www.dropbox.com/s/tbujesxtmxknr39/perl-Term-Menu.spec
SRPM URL: https://www.dropbox.com/s/qtug6gu6xgn0178/perl-Term-Menu-0.09-1.fc18.src.rpm
Description: Simple perl module that makes it easy to print an interactive menu
Fedora Account System Username: cjohnston
Comment 1 Ralf Corsepius 2013-09-19 01:43:08 EDT
* Fails to build in mock.
Please add: 
BuildRequires:  perl(Test::More)

* With the BR: above added, the testsuite skips a test.
Adding 
BuildRequires:  perl(Test::Expect)
causes the skipped test to be run.

* The License: field needs to be
License:        GPL+ or Artistic

* Unless you intent to support EPEL5,
Group: ...
...
BuildRoot:      %{_tmppath}/%{name}-%{version}-build
...
rm -rf $RPM_BUILD_ROOT
can all be removed.

* Macro usage is inconsistent:
%__perl and %{__perl} are both being used.
Please only use one of these (I recommend using %{__perl})
Comment 2 Miroslav Suchý 2015-07-21 10:53:02 EDT
Chris, any progress? Are you still interrested in this package?
Comment 3 Miroslav Suchý 2016-02-08 08:57:33 EST
No response. Closing as dead review. If you ever want to continue, please resubmit.

Note You need to log in before you can comment on or make changes to this bug.