Bug 987717 - Review Request: perl-GraphViz2 - A wrapper for AT&T's Graphviz
Review Request: perl-GraphViz2 - A wrapper for AT&T's Graphviz
Status: NEW
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On: 979653 979654 979656 979657 979660 979666 979668 1122050
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-23 21:20 EDT by Christopher Meng
Modified: 2014-07-22 19:29 EDT (History)
4 users (show)

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


Attachments (Terms of Use)

  None (edit)
Description Christopher Meng 2013-07-23 21:20:55 EDT
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
Comment 1 Peter Oliver 2013-10-02 18:22:44 EDT
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-%{version}.tgz"?
Comment 2 Christopher Meng 2013-11-04 04:02:07 EST
(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-%{version}.tgz"?

Sorry for the late response, just saw it from SPAM...

Don't worry ;) I will update the URLs once all deps are resolved.
Comment 3 Petr Pisar 2014-02-25 08:28:39 EST
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.
Comment 4 Christopher Meng 2014-02-25 08:30:57 EST
(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.
Comment 5 Lubomir Rintel 2014-04-28 08:54:00 EDT
What's the status of this? How is this different from perl-GraphViz (you my want to clarify that in %description).
Comment 6 Christopher Meng 2014-04-28 11:19:57 EDT
(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.
Comment 7 Christopher Meng 2014-07-13 00:31:21 EDT
New deps again.

perl(DBIx::Admin::TableInfo) with perl(Moo) + perl(Data::Dumper::Concise).
Comment 8 Christopher Meng 2014-07-13 00:43:05 EDT
Adding Upstream releases monitoring entries, with Blocks soon.

perl(Moo) and perl(Data::Dumper::Concise) are dated in the repo.
Comment 9 Petr Pisar 2014-07-22 03:53:02 EDT
(In reply to Christopher Meng from comment #0)
> Spec URL: http://cicku.me/perl-GraphViz2.spec

404 Not Found
Comment 10 Petr Pisar 2014-07-22 10:47:21 EDT
I submitted DBIx::Admin::TableInfo for review as bug #1122050.

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