[Bug 773502] Review Request: perl-Spreadsheet-read - universal API to read any spreadsheet

[bugzilla at redhat.com]

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

--- Comment #1 from Michael Schwendt <mschwendt at gmail.com> 2012-01-12 05:16:47 EST ---
[Although I know a bit of Perl, I'm not a Perl packager, so I'm not up-to-date
on Fedora Perl Packaging].


* A plain src.rpm rebuild is interactive here:

$ rpmbuild --rebuild perl-Spreadsheet-Read-0.45-1.fc15.src.rpm
[...]
+ cd Spreadsheet-Read-0.45
+ LANG=C
+ export LANG
+ unset DISPLAY
+ /usr/bin/perl Makefile.PL INSTALLDIRS=vendor
Do you want to install 'xlscat' (Convert Spreadsheet to plain text or CSV) ? 
[n] 


That's a blocker. Several more setup questions are asked. This ought to be
automatic with a default build config being applied.



* There's a test-suite included, suitable for a %check section. "make test"
does something, but requires _at least_ perl-Test-Simple perl-Test-NoWarnings
and then still warns about further build requirements as well as skipped tests.

-- 
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.