[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
https://bugzilla.redhat.com/show_bug.cgi?id=957622 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||perl-Twiggy-0.1021-3.fc17 Resolution|--- |ERRATA Last Closed||2013-06-19 22:33:43 --- Comment #10 from Fedora Update System --- perl-Twiggy-0.1021-3.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AZpddFSzmy&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
https://bugzilla.redhat.com/show_bug.cgi?id=957622 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- perl-Twiggy-0.1021-3.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JESJHA5LMH&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
https://bugzilla.redhat.com/show_bug.cgi?id=957622 --- Comment #8 from Fedora Update System --- perl-Twiggy-0.1021-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/perl-Twiggy-0.1021-3.fc17 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=qUyisXvghU&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
https://bugzilla.redhat.com/show_bug.cgi?id=957622 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=qeP5CTh4aU&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
https://bugzilla.redhat.com/show_bug.cgi?id=957622 Bug 957622 depends on bug 959188, which changed state. Bug 959188 Summary: Review Request: perl-Plack-Middleware-Deflater - Compress response body with Gzip or Deflate https://bugzilla.redhat.com/show_bug.cgi?id=959188 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=zAV6GUBJ8R&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? | Flags||fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5lUh6m9RIi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 --- Comment #7 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=JeuB2pN902&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 Robin Lee changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Robin Lee --- New Package SCM Request === Package Name: perl-Twiggy Short Description: AnyEvent HTTP server for PSGI (like Thin) Owners: cheeselee Branches: f17 f18 f19 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hfxFsNo8em&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 Petr Šabata changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #5 from Petr Šabata --- Ok, spec clean, Plack::Middleware::Deflater review done -- time to approve this too :) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=snMC0XQ0Zi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 --- Comment #4 from Robin Lee --- Thanks! Change: - Remove duplicated BR Spec URL: http://cheeselee.fedorapeople.org/perl-Twiggy.spec SRPM URL: http://cheeselee.fedorapeople.org/perl-Twiggy-0.1021-3.fc18.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CwbrmhbQ9B&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 --- Comment #3 from Petr Šabata --- Alright, better. You have some duplicate BRs now, though. I'm going to review Plack::Middleware::Deflater too... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=YhKpfk2THI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 Robin Lee changed: What|Removed |Added Depends On||959188 --- Comment #2 from Robin Lee --- Changes: - Revise BuildRequires and Requires - Include the eg directory to %%doc Spec URL: http://cheeselee.fedorapeople.org/perl-Twiggy.spec SRPM URL: http://cheeselee.fedorapeople.org/perl-Twiggy-0.1021-2.fc18.src.rpm I also submitted a review request for Plack::Middleware::Deflater. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4S8v9shGgl&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 --- Comment #1 from Petr Šabata --- The package uses bundled inc::Module::Install, either buildrequire all its actual dependencies (preferred, listed below) or buildrequire perl(inc::Module::Install) and delete tne inc directory. perl(base) perl(Carp) perl(Config) perl(Cwd) perl(ExtUtils::MM_Unix) perl(Fcntl) perl(File::Find) perl(File::Path) perl(FindBin) perl(Pod::Text) perl(strict) perl(vars) In addition to those, you're missing many other used in Twiggy code and/or the test suite (some are shared with inc::Module::Install): perl(AnyEvent::AIO) perl(AnyEvent::Handle) perl(AnyEvent::Socket) perl(AnyEvent::Util) perl(base) perl(Carp) perl(constant) perl(Errno) perl(FindBin) perl(HTTP::Request) perl(HTTP::Request::Common) perl(HTTP::Response) perl(IO::AIO) perl(IO::Handle) perl(IO::Socket::INET) perl(IO::Uncompress::Gunzip) perl(LWP::UserAgent) perl(parent) perl(Plack::App::File) perl(Plack::HTTPParser) perl(Plack::Loader) perl(Plack::Request) perl(Plack::Test::Suite) perl(Plack::Util) perl(Scalar::Util) perl(Socket) perl(warnings) Consider packaging Plack::Middleware::Deflater too, so the deflater.t tests get executed. You can drop perl(HTTP::Status) and perl(Try::Tiny) from your runtime dep list. RPM finds those automatically. perl(AnyEvent::AIO) and perl(IO::AIO), on the other hand, don't get auto-detected and you need to require them explicitly. Oh, and perhaps some users would find the examples in the 'eg' directory useful?... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LNfiKJE7ah&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 957622] Review Request: perl-Twiggy - AnyEvent HTTP server for PSGI (like Thin)
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=957622 Petr Šabata changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com Assignee|nob...@fedoraproject.org|psab...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=f1GS3GwQCy&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review