gv 3.7.2 #7611

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@RuiPereira
Contributor

GNU gv allows to view and navigate through
PostScript and PDF documents on an X display by
providing a graphical user interface for the
Ghostscript interpreter.

@jacknagel jacknagel commented on an outdated diff Sep 13, 2011
Library/Formula/gv.rb
@@ -0,0 +1,36 @@
+require 'formula'
+
+class Gv < Formula
+ url 'ftp://ftp.mirrorservice.org/pub/gnu/gv/gv-3.7.2.tar.gz'
jacknagel
jacknagel Sep 13, 2011 Contributor

This should be http://ftpmirror.gnu.org/gv/gv-3.7.2.tar.gz as we're trying to use automatic mirror selection for GNU downloads now.

I probably need to make the GNU site regex in brew audit less dumb; most of these URLs have /gnu/ in common.

Contributor

URL fixed.

Contributor

Not seeing it, did you push the correct commit?

Contributor

I did a rebase to with my original commit and the fix, the new commit is c09a014

Contributor

Ohhh sorry, I had only assumed it was a ftp: vs http:. Fixing...

Contributor

Thanks, will test & merge tomorrow.

Contributor

Should be OK now.

Contributor
adamv commented Sep 14, 2011

I don't see an xaw3d formula.

Contributor
adamv commented Sep 14, 2011

Also the gs dep should be ghostscript.

Contributor

I don't see an xaw3d formula.

Whoops, missed that. Though I see there is a pull request for it.

Contributor

So I see the required progression here is imake -> xaw3d -> gv. Will take a look.

@jacknagel jacknagel commented on an outdated diff Sep 14, 2011
Library/Formula/gv.rb
@@ -0,0 +1,36 @@
+require 'formula'
+
+class Gv < Formula
+ url 'http://ftpmirror.gnu.org/gv/gv-3.7.2.tar.gz'
+ homepage 'http://www.gnu.org/s/gv/'
+ md5 'eb47d465755b7291870af66431c6f2e1'
+
+ depends_on 'gs'
+ depends_on 'xaw3d'
+
jacknagel
jacknagel Sep 14, 2011 Contributor

I see after the build that we get this:

rmdir: /usr/local/Cellar/gv/3.7.2/share/gv/safe-gs-workdir (empty)

so we should probably add

skip_clean 'share/gv/safe-gs-workdir'

here.

@jacknagel jacknagel commented on an outdated diff Sep 14, 2011
Library/Formula/gv.rb
@@ -0,0 +1,36 @@
+require 'formula'
+
+class Gv < Formula
+ url 'http://ftpmirror.gnu.org/gv/gv-3.7.2.tar.gz'
+ homepage 'http://www.gnu.org/s/gv/'
+ md5 'eb47d465755b7291870af66431c6f2e1'
+
+ depends_on 'gs'
jacknagel
jacknagel Sep 14, 2011 Contributor

And yeah, as Adam said this should be spelled out ghostscript.

@RuiPereira RuiPereira gv 3.7.2
GNU gv allows to view and navigate through
PostScript and PDF documents on an X display by
providing a graphical user interface for the
Ghostscript interpreter.
4addf79
Contributor

Fixed. And yes, sorry for the all imake < xaw3d < gv dependency, I should have made that clear in the pull request.

Contributor

Pulled, thanks.

@jacknagel jacknagel closed this Sep 15, 2011
@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.