https://bugzilla.redhat.com/show_bug.cgi?id=987717
Bug ID: 987717 Summary: Review Request: perl-GraphViz2 - A wrapper for AT&T's Graphviz Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: cickumqt@gmail.com QA Contact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org
Spec URL: http://cicku.me/perl-GraphViz2.spec SRPM URL: http://cicku.me/perl-GraphViz2-2.14-1.fc20.src.rpm Description: This module provides a Perl interface to the amazing Graphviz, an open source graph visualization tool from AT&T. Fedora Account System Username: cicku
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |979666, 979668, 979660, | |979654
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |979653, 979656
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |979657
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Bug 987717 depends on bug 979660, which changed state.
Bug 979660 Summary: Review Request: perl-XML-Bare - Minimal XML parser implemented via a C state engine https://bugzilla.redhat.com/show_bug.cgi?id=979660
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Bug 987717 depends on bug 979668, which changed state.
Bug 979668 Summary: Review Request: perl-HTML-Entities-Interpolate - Call HTML::Entities::encode_entities via a hash within a string https://bugzilla.redhat.com/show_bug.cgi?id=979668
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Peter Oliver mavit@mavit.org.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mavit@mavit.org.uk
--- Comment #1 from Peter Oliver mavit@mavit.org.uk --- There's a newer upstream version of this module, 2.22, and version 2.14 is no-longer on CPAN proper. Worth setting Source0 to "http://backpan.perl.org/authors/id/R/RS/RSAVAGE/GraphViz2-%%7Bversion%7D.tgz...
https://bugzilla.redhat.com/show_bug.cgi?id=987717
--- Comment #2 from Christopher Meng cickumqt@gmail.com --- (In reply to Peter Oliver from comment #1)
There's a newer upstream version of this module, 2.22, and version 2.14 is no-longer on CPAN proper. Worth setting Source0 to "http://backpan.perl.org/authors/id/R/RS/RSAVAGE/GraphViz2-%%7Bversion%7D.tgz...
Sorry for the late response, just saw it from SPAM...
Don't worry ;) I will update the URLs once all deps are resolved.
https://bugzilla.redhat.com/show_bug.cgi?id=987717 Bug 987717 depends on bug 979666, which changed state.
Bug 979666 Summary: Review Request: perl-Text-Xslate - Scalable template engine https://bugzilla.redhat.com/show_bug.cgi?id=979666
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ppisar@redhat.com
--- Comment #3 from Petr Pisar ppisar@redhat.com --- Besides the spec file is not for current upstream version 2.27, it does not require /usr/bin/dot (graphviz), although the run() method executes it.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Whiteboard| |NotReady
--- Comment #4 from Christopher Meng cickumqt@gmail.com --- (In reply to Petr Pisar from comment #3)
Besides the spec file is not for current upstream version 2.27, it does not require /usr/bin/dot (graphviz), although the run() method executes it.
Cleanup soon, will finish it.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lkundrak@v3.sk
--- Comment #5 from Lubomir Rintel lkundrak@v3.sk --- What's the status of this? How is this different from perl-GraphViz (you my want to clarify that in %description).
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng cickumqt@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Whiteboard|NotReady |
--- Comment #6 from Christopher Meng cickumqt@gmail.com --- (In reply to Lubomir Rintel from comment #5)
What's the status of this? How is this different from perl-GraphViz (you my want to clarify that in %description).
Good catch.
perl-G and perl-G2 are from the same author. perl-G is deprecated now:
http://search.cpan.org/dist/GraphViz/
You can take a deep look:
"Interface to AT&T's GraphViz. Deprecated. See GraphViz2"
That's why I packaged perl-G2. Last year I was trying to package many dependencies, then back to G2 itself I couldn't find any free time.
I will clean up my old reviews soon.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
--- Comment #7 from Christopher Meng i@cicku.me --- New deps again.
perl(DBIx::Admin::TableInfo) with perl(Moo) + perl(Data::Dumper::Concise).
https://bugzilla.redhat.com/show_bug.cgi?id=987717
--- Comment #8 from Christopher Meng i@cicku.me --- Adding Upstream releases monitoring entries, with Blocks soon.
perl(Moo) and perl(Data::Dumper::Concise) are dated in the repo.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
--- Comment #9 from Petr Pisar ppisar@redhat.com --- (In reply to Christopher Meng from comment #0)
Spec URL: http://cicku.me/perl-GraphViz2.spec
404 Not Found
https://bugzilla.redhat.com/show_bug.cgi?id=987717
--- Comment #10 from Petr Pisar ppisar@redhat.com --- I submitted DBIx::Admin::TableInfo for review as bug #1122050.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Christopher Meng i@cicku.me changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |1122050
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1122050 [Bug 1122050] Review Request: perl-DBIx-Admin-TableInfo - A wrapper for DBI's table_info(), column_info(), *_key_info()
https://bugzilla.redhat.com/show_bug.cgi?id=987717 Bug 987717 depends on bug 1122050, which changed state.
Bug 1122050 Summary: Review Request: perl-DBIx-Admin-TableInfo - Wrapper for DBI's table_info(), column_info(), *_key_info() https://bugzilla.redhat.com/show_bug.cgi?id=1122050
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Emmanuel Seyman emmanuel@seyman.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |emmanuel@seyman.fr
--- Comment #11 from Emmanuel Seyman emmanuel@seyman.fr --- Okay, folks.
I now need perl-Graphviz2 in Fedora (it's in the dependency chain of perl-Mojolicious-Plugin-DbicSchemaViewer). Given that the spec file is no longer available and that last comment on this bug is nearly 4 years old, I've decided to treat this review as abandoned.
I've opened bug #1592136 with my attempt at a package.
https://bugzilla.redhat.com/show_bug.cgi?id=987717
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED See Also| |https://bugzilla.redhat.com | |/show_bug.cgi?id=1592136 Blocks| |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed| |2018-06-18 02:58:00
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug.
package-review@lists.fedoraproject.org