[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-24 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-10-24 12:37 EST ---
Imported and built!  As always, thanks for the review :)

-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-23 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-23 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-23 16:31 EST ---
OK, builds fine for me now; rpmlint is quiet and I don't see anything worth
complaining about.

* source files match upstream:
   48e5e4483e2847a563f02ec1d1f148e72c327bc623c5b59da41cada9590adc09  
   SQL-Translator-0.08001.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper (and there's a bunch of them)
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane.  See end for list.
* %check is present and all tests pass:
   All tests successful, 2 tests skipped.
   Files=43, Tests=1284, 66 wallclock secs (62.53 cusr +  2.89 csys = 65.42 CPU)
  The two skipped tests are:
   t/18ttschema-producer..skipped
 all skipped: Template v2.15 is is incompatible with SQL::Translator 0.08+
   t/33tt-table-producer..skipped
 all skipped: Template v2.15+ is incompatible with SQL::Translator 0.08+

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

APPROVED

Here's the full dependency list:
   perl(Parse::RecDescent::SQL::Translator::Parser::DB2::Grammar)
   perl(SQL::Translator) = 0.08001
   perl(SQL::Translator::Diff)
   perl(SQL::Translator::Filter::DefaultExtra)
   perl(SQL::Translator::Filter::Globals)
   perl(SQL::Translator::Filter::Names)
   perl(SQL::Translator::Parser) = 1.9
   perl(SQL::Translator::Parser::Access) = 1.3
   perl(SQL::Translator::Parser::DB2)
   perl(SQL::Translator::Parser::DB2::Grammar)
   perl(SQL::Translator::Parser::DBI) = 1.10
   perl(SQL::Translator::Parser::DBI::DB2)
   perl(SQL::Translator::Parser::DBI::MySQL) = 1.6
   perl(SQL::Translator::Parser::DBI::Oracle) = 1.2
   perl(SQL::Translator::Parser::DBI::PostgreSQL) = 1.10
   perl(SQL::Translator::Parser::DBI::SQLServer) = 1.6
   perl(SQL::Translator::Parser::DBI::SQLite) = 1.5
   perl(SQL::Translator::Parser::DBI::Sybase) = 1.6
   perl(SQL::Translator::Parser::Excel) = 1.14
   perl(SQL::Translator::Parser::MySQL) = 1.58
   perl(SQL::Translator::Parser::Oracle) = 1.29
   perl(SQL::Translator::Parser::PostgreSQL) = 1.48
   perl(SQL::Translator::Parser::SQLServer) = 1.6
   perl(SQL::Translator::Parser::SQLite) = 1.12
   perl(SQL::Translator::Parser::Storable) = 1.5
   perl(SQL::Translator::Parser::Sybase) = 1.10
   perl(SQL::Translator::Parser::XML) = 1.2
   perl(SQL::Translator::Parser::XML::SQLFairy) = 1.15
   perl(SQL::Translator::Parser::YAML) = 1.8
   perl(SQL::Translator::Parser::xSV) = 1.16
   perl(SQL::Translator::Producer) = 1.8
   perl(SQL::Translator::Producer::DB2) = 1.5
   perl(SQL::Translator::Producer::DiaUml) = 1.2
   perl(SQL::Translator::Producer::Diagram) = 1.13
   perl(SQL::Translator::Producer::Dumper) = 1.9
   perl(SQL::Translator::Producer::GraphViz) = 1.14
   perl(SQL::Translator::Producer::HTML) = 1.15
   perl(SQL::Translator::Producer::Latex) = 1.1
   perl(SQL::Translator::Producer::MySQL) = 1.52
   perl(SQL::Translator::Producer::Oracle) = 1.34
   perl(SQL::Translator::Producer::POD) = 1.5
   perl(SQL::Translator::Producer::PostgreSQL) = 1.29
   perl(SQL::Translator::Producer::SQLServer) = 1.7
   perl(SQL::Translator::Producer::SQLite) = 1.15
   perl(SQL::Translator::Producer::Storable) = 1.5
   perl(SQL::Translator::Producer::Sybase) = 1.9
   perl(SQL::Translator::Producer::TT::Base) = 1.6
   perl(SQL::Translator::Producer::TT::Table) = 1.3
   perl(SQL::Translator::Producer::TTSchema) = 1.11
   perl(SQL::Translator::Producer::XML) = 1.16
   perl(SQL::Translator::Producer::XML::SQLFairy) = 1.20
   perl(SQL::Translator::Producer::YAML) = 1.12
   perl(SQL::Translator::Schema) = 1.26
   perl(SQL::Translator::Schema::Constants) = 1.43
   perl(SQL::Translator::Schema::Constraint) = 1.21
   perl(SQL::Translator::Schema::Field) = 1.26
   perl(SQL::Translator::Schema::Graph)
   

[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-23 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-23 18:18 EST ---
New Package CVS Request
===
Package Name: perl-SQL-Translator
Short Description: Manipulate structured data definitions (SQL and more)
Owners: cweyl
Branches: FC-6 F-7 F-8 devel
InitialCC: perl-sig
Cvsextras Commits:

-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-23 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-24 00:32 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-10-14 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?([EMAIL PROTECTED]|
   |edu)|




--- Additional Comments From [EMAIL PROTECTED]  2007-10-14 16:47 EST ---
Yes, and updated.

* Sun Oct 14 2007 Chris Weyl [EMAIL PROTECTED] 0.08001-1
- updated to 0.08001
- update license tag
- nix errant perl(Producer::BaseTest) provides
- make description useful :)
- we now skip Template::Toolkit tests correctly, so stop disabling them

http://fedora.biggerontheinside.net/review/perl-SQL-Translator-0.08001-1.fc7.src.rpm
http://fedora.biggerontheinside.net/review/perl-SQL-Translator.spec


-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-09-05 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-05 20:33 EST ---
Chris, I know you've been around recently; did you still want to submit this?

-- 
Configure bugmail: https://bugzilla.redhat.com/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-07-29 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-29 14:09 EST ---
I was something from Chris in another ticket, so:
One final ping before I close this.

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-07-13 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-13 15:09 EST ---
I'm going to go ahead and close this out soon if there is no response.  There
are also several other open tickets from the same submitter; I'll start setting
them NEEDINFO and (later) closing them out if Chris doesn't give some indication
that he's still around.

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-07-06 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||edu)




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-06-18 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-18 15:12 EST ---
I see no record of perl-Text-RecordParser being built for rawhide, just F7.  I
can build it if you've no time.  I'm still completely confused by the ParseExcel
thing, though.


-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-06-18 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-06-18 15:48 EST ---
(In reply to comment #3)
 I see no record of perl-Text-RecordParser being built for rawhide, just F7.  I
 can build it if you've no time.  I'm still completely confused by the 
 ParseExcel
 thing, though.

I believe this caused by not following the upstream version number:

   upstream   : Spreadsheet-ParseExcel-0.32
   RPM package: perl-Spreadsheet-ParseExcel-0.3200

Look at the provides list

  $ rpm -qp --provides perl-Spreadsheet-ParseExcel-0.3200-1.fc8.i386.rpm 
  perl(Spreadsheet::ParseExcel) = 0.32
  ...
  perl-Spreadsheet-ParseExcel = 0.3200-1.fc8


It provides perl(Spreadsheet::ParseExcel) = 0.32 and not
perl(Spreadsheet::ParseExcel) = 0.3200.  And 0.32 is less than 0.2622.

PS - There several perl packages that have started changing the upstream version
in order to avoid bumping the epoch.

jpo

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-06-12 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-12 11:55 EST ---
I'd certainly think it'd be in rawhide  Odd.  Very odd.

I'll look into this also -- but have been super busy last week and this one, so
it might be a couple days.

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-06-09 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-10 01:32 EST ---
Hmm, not building for me (rawhide, x86_64) due to missing dependencies:

No Package Found for perl(Text::RecordParser) = 0.02
No Package Found for perl(Spreadsheet::ParseExcel) = 0.2602

Shouldn't the former be in rawhide by now?  (My mirror is up to date.)
The latter, I've no idea, because it's certainly there
(perl-Spreadsheet-ParseExcel-0:0.3200-1.fc8.i386).  If I remove the '= 0.2602
bit it finds it.  I'm at a loss.

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-05-16 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||perl-SQL-Translator




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-05-16 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


Bug 240333 depends on bug 240177, which changed state.

Bug 240177 Summary: Review Request: perl-Text-RecordParser - Read 
record-oriented files
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240177

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 240333] Review Request: perl-SQL-Translator - Manipulate structured data definitions (SQL and more)

2007-05-16 Thread bugzilla
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: perl-SQL-Translator - Manipulate structured data 
definitions (SQL and more)
Alias: perl-SQL-Translator

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


Bug 240333 depends on bug 240207, which changed state.

Bug 240207 Summary: Review Request: perl-Class-Base - Useful base class for 
deriving other modules
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240207

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review