This repository has been archived by the owner. It is now read-only.

brew install gnuplot failed when AquaTerm1.1.0.dmg is preinstalled #14647

Closed
lvnilesh opened this Issue Sep 2, 2012 · 19 comments

Comments

Projects
None yet
@lvnilesh

lvnilesh commented Sep 2, 2012

I wish there was an aquaterm brew formula. brew install aquaterm doesn't work so I manually installed http://downloads.sourceforge.net/project/aquaterm/AquaTerm/AquaTerm1.1.0.dmg and then tried brew install gnuplot but it failed.

Output from HOMEBREW_MAKE_JOBS=1 VERBOSE=1 brew install gnuplot https://gist.github.com/3603349

Output from ~/Library/Logs/Homebrew/config.log https://gist.github.com/3603370

@lvnilesh

This comment has been minimized.

Show comment
Hide comment
@lvnilesh

lvnilesh Sep 2, 2012

I uninstaled aquaterm via below steps and now brew install gnuplot succeeded.
rm -f /usr/local/lib/libaquaterm.* rm -rf /usr/local/include/aquaterm rm -rf /Applications/AquaTerm.app rm -rf /Library/Frameworks/AquaTerm.framework

lvnilesh commented Sep 2, 2012

I uninstaled aquaterm via below steps and now brew install gnuplot succeeded.
rm -f /usr/local/lib/libaquaterm.* rm -rf /usr/local/include/aquaterm rm -rf /Applications/AquaTerm.app rm -rf /Library/Frameworks/AquaTerm.framework

@lvnilesh

This comment has been minimized.

Show comment
Hide comment
@lvnilesh

lvnilesh Sep 2, 2012

I need aquaterm, however, I am using x11 as a stop gap by setting export GNUTERM=x11 in ~/.zshenv ( or ~/.bashprofile)

lvnilesh commented Sep 2, 2012

I need aquaterm, however, I am using x11 as a stop gap by setting export GNUTERM=x11 in ~/.zshenv ( or ~/.bashprofile)

@hesstobi

This comment has been minimized.

Show comment
Hide comment
@hesstobi

hesstobi Sep 4, 2012

got the same error here

hesstobi commented Sep 4, 2012

got the same error here

@isaiasmy

This comment has been minimized.

Show comment
Hide comment
@isaiasmy

isaiasmy Sep 4, 2012

The problem is related with some incompatibilities with aquaterm and Lion/Mountain lion. Your solution removes the aquaterm support, which is not desired in my case. There is a new release of aquaterm and some changes are necessary to compile applications with this new version. I don't have the expertise to create a formula with a proper patch. Nevertheless, i put some pointers just in case someone is able to do it:

http://permalink.gmane.org/gmane.comp.graphics.gnuplot.devel/10943
https://github.com/AquaTerm/AquaTerm/commits/master
https://trac.macports.org/ticket/34423
https://trac.macports.org/ticket/34346

isaiasmy commented Sep 4, 2012

The problem is related with some incompatibilities with aquaterm and Lion/Mountain lion. Your solution removes the aquaterm support, which is not desired in my case. There is a new release of aquaterm and some changes are necessary to compile applications with this new version. I don't have the expertise to create a formula with a proper patch. Nevertheless, i put some pointers just in case someone is able to do it:

http://permalink.gmane.org/gmane.comp.graphics.gnuplot.devel/10943
https://github.com/AquaTerm/AquaTerm/commits/master
https://trac.macports.org/ticket/34423
https://trac.macports.org/ticket/34346

@zigorsalvador

This comment has been minimized.

Show comment
Hide comment
@zigorsalvador

zigorsalvador Sep 20, 2012

I'm also encountering this issue.

Mac OS X 10.8.2 with AquaTerm 1.1.0 installed.

No problems without AquaTerm (i.e., using XQuartz).

However, AquaTerm is preferred.

I'm also encountering this issue.

Mac OS X 10.8.2 with AquaTerm 1.1.0 installed.

No problems without AquaTerm (i.e., using XQuartz).

However, AquaTerm is preferred.

@Sharpie Sharpie closed this in 627bb7c Sep 22, 2012

@Sharpie

This comment has been minimized.

Show comment
Hide comment
@Sharpie

Sharpie Sep 22, 2012

Contributor

Allright, AquaTerm is now disabled by default in the Gnuplot build. Anyone who wants it will have to figure out how to make it work and submit a pull request.

Contributor

Sharpie commented Sep 22, 2012

Allright, AquaTerm is now disabled by default in the Gnuplot build. Anyone who wants it will have to figure out how to make it work and submit a pull request.

snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this issue Dec 17, 2012

gnuplot: Disable aquaterm by default
Adds a MacPorts patch that allows `--without-aquaterm` as a configuration
option and then invokes this option by default.

Fixes #14647.
@justinkb

This comment has been minimized.

Show comment
Hide comment
@justinkb

justinkb Dec 29, 2012

Contributor

I have this working now. Will add a commit and pull request in a bit.

Would a proper fix require that aquaterm is actually built with homebrew (I have it working with a Xcode compiled AquaTerm.app and AquaTerm.framework)? This will take a bit longer, but I can probably do it.

PS. Should the gnuplot build be building the X11 terminal even if with-x isn't set? Of course, this doesn't really hurt, but it's confusing behaviour that it will build it just because you have XQuartz installed, even when not specifying you want it.

Contributor

justinkb commented Dec 29, 2012

I have this working now. Will add a commit and pull request in a bit.

Would a proper fix require that aquaterm is actually built with homebrew (I have it working with a Xcode compiled AquaTerm.app and AquaTerm.framework)? This will take a bit longer, but I can probably do it.

PS. Should the gnuplot build be building the X11 terminal even if with-x isn't set? Of course, this doesn't really hurt, but it's confusing behaviour that it will build it just because you have XQuartz installed, even when not specifying you want it.

@ydzhou

This comment has been minimized.

Show comment
Hide comment
@ydzhou

ydzhou Jan 24, 2013

Please do pull a request to fix aquaterm issue. Really miss aquaterm.

ydzhou commented Jan 24, 2013

Please do pull a request to fix aquaterm issue. Really miss aquaterm.

@milktrader milktrader referenced this issue in mbaz/Gaston.jl Mar 5, 2013

Closed

gaston_demo() fails on Mac #3

@acornejo

This comment has been minimized.

Show comment
Hide comment
@acornejo

acornejo Jul 17, 2013

The latest version of aquaterm and gnuplot play nice together.

Could we get gnuplot with aquaterm back? The following works:

diff --git a/Library/Formula/gnuplot.rb b/Library/Formula/gnuplot.rb
index ef60286..262920f 100644
--- a/Library/Formula/gnuplot.rb
+++ b/Library/Formula/gnuplot.rb
@@ -17,6 +17,7 @@ class Gnuplot < Formula
   option 'tests',  'Verify the build with make check (1 min)'
   option 'without-emacs', 'Do not build Emacs lisp files'
   option 'latex',  'Build with LaTeX support'
+  option 'aquaterm', 'Build with AquaTerm support'

   if build.head?
     depends_on :automake
@@ -53,6 +54,7 @@ class Gnuplot < Formula
     args << '--enable-qt'             if build.include? 'qt'
     args << '--without-lua'           if build.include? 'nolua'
     args << '--without-lisp-files'    if build.include? 'without-emacs'
+    args << '--with-aquaterm'         if build.include? 'aquaterm'

     if build.include? 'latex'
       args << '--with-latex'

The latest version of aquaterm and gnuplot play nice together.

Could we get gnuplot with aquaterm back? The following works:

diff --git a/Library/Formula/gnuplot.rb b/Library/Formula/gnuplot.rb
index ef60286..262920f 100644
--- a/Library/Formula/gnuplot.rb
+++ b/Library/Formula/gnuplot.rb
@@ -17,6 +17,7 @@ class Gnuplot < Formula
   option 'tests',  'Verify the build with make check (1 min)'
   option 'without-emacs', 'Do not build Emacs lisp files'
   option 'latex',  'Build with LaTeX support'
+  option 'aquaterm', 'Build with AquaTerm support'

   if build.head?
     depends_on :automake
@@ -53,6 +54,7 @@ class Gnuplot < Formula
     args << '--enable-qt'             if build.include? 'qt'
     args << '--without-lua'           if build.include? 'nolua'
     args << '--without-lisp-files'    if build.include? 'without-emacs'
+    args << '--with-aquaterm'         if build.include? 'aquaterm'

     if build.include? 'latex'
       args << '--with-latex'
@lvnilesh

This comment has been minimized.

Show comment
Hide comment
@lvnilesh

lvnilesh Aug 16, 2013

+1 for getting gnuplot with aquaterm back.

+1 for getting gnuplot with aquaterm back.

@manphiz

This comment has been minimized.

Show comment
Hide comment
@manphiz

manphiz Aug 16, 2013

Contributor

Will review a PR for aquaterm support. Please note that if aquaterm is from 3rd party, a requirement should be added to check its availability.

Contributor

manphiz commented Aug 16, 2013

Will review a PR for aquaterm support. Please note that if aquaterm is from 3rd party, a requirement should be added to check its availability.

@Globegitter

This comment has been minimized.

Show comment
Hide comment
@Globegitter

Globegitter Sep 1, 2013

Just needing that now, so 👍

Just needing that now, so 👍

@pavel-popov

This comment has been minimized.

Show comment
Hide comment
@pavel-popov

pavel-popov Sep 29, 2013

Someone, reopen Issue, please!

Someone, reopen Issue, please!

@LandonSchropp

This comment has been minimized.

Show comment
Hide comment

+1

@albertz

This comment has been minimized.

Show comment
Hide comment
@albertz

albertz Nov 9, 2013

Contributor

Any update here?

Contributor

albertz commented Nov 9, 2013

Any update here?

@Cueball

This comment has been minimized.

Show comment
Hide comment
@Cueball

Cueball Nov 11, 2013

+1 Just jumped from MacPorts to homebrew...broken AquaTerm isn't a deal-breaker, but it'd be nice to have. Will keep poking around based on the info from acornejo...

Cueball commented Nov 11, 2013

+1 Just jumped from MacPorts to homebrew...broken AquaTerm isn't a deal-breaker, but it'd be nice to have. Will keep poking around based on the info from acornejo...

@rvega

This comment has been minimized.

Show comment
Hide comment

rvega commented Mar 28, 2014

+1

:)

@mw9

This comment has been minimized.

Show comment
Hide comment
@mw9

mw9 Mar 28, 2014

Contributor

It is now working with AT 1.1.1.

On Friday, 28 March 2014, Rafael Vega notifications@github.com wrote:

+1

:)

Reply to this email directly or view it on GitHubhttps://github.com/Homebrew/homebrew/issues/14647#issuecomment-38937736
.

Contributor

mw9 commented Mar 28, 2014

It is now working with AT 1.1.1.

On Friday, 28 March 2014, Rafael Vega notifications@github.com wrote:

+1

:)

Reply to this email directly or view it on GitHubhttps://github.com/Homebrew/homebrew/issues/14647#issuecomment-38937736
.

@rvega

This comment has been minimized.

Show comment
Hide comment
@rvega

rvega Mar 29, 2014

Yes, sorry, I didn't read the caveat that states AT must be installed first. I assumed Homebrew would install it as a dependency.
Sorry for the noise.

rvega commented Mar 29, 2014

Yes, sorry, I didn't read the caveat that states AT must be installed first. I assumed Homebrew would install it as a dependency.
Sorry for the noise.

@Homebrew Homebrew 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.