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

added inkscape formula #12989

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
Contributor

btel commented Jun 23, 2012

inkscape is a vector drawing program based on SVG format

Contributor

btel commented Jun 25, 2012

Thanks. I moved the inreplace lines to the patch as suggested.

pelson commented Jun 25, 2012

I've checked this out and it works like a charm of 10.7.4. Thanks!

Contributor

btel commented Jun 25, 2012

Good to hear this. Thanks for testing!

Contributor

NelsonMinar commented Jun 25, 2012

I successfully built and ran this, also on MacOS 10.7.4. The resulting UI is a little nicer rendering than the version inkscape.org provides as Inkscape-0.48.2-1-SNOWLEOPARD.dmg. Running it does give one GTK warning about a missing icon. I didn't have GTK on my system before so have installed no themes.

(inkscape:54527): Gtk-WARNING **: Could not find the icon 'object-visible'. The 'hicolor' theme
was not found either, perhaps you need to install it.
You can get a copy from:
http://icon-theme.freedesktop.org/releases
Contributor

btel commented Jun 27, 2012

This is a reported problem of inkscape: https://bugs.launchpad.net/inkscape/+bug/446842

Since it does not affect the user experience, I suggest to wait until it is fixed upstream.

scrax commented Jul 4, 2012

Hi, I have brew installed, but brew install inkscape tells me:
Error: No available formula for inkscape
how can I solve?
EDIT:
Found how:
brew install https://raw.github.com/btel/homebrew/3268dd1fea049fdd605717711e7ea1ecc1d6a576/Library/Formula/inkscape.rb

Hi,
the dependency lcms is missing. (the ./configure on inkscape fails). After a "brew install lcms", the formula works fine.

@btel wrote:

Since it does not affect the user experience, I suggest to wait until it is fixed upstream.

Hm, I don't understand this :-) If it does not affect the user experience, we don't need to wait until it's fixed upstream. Or am I just getting logic wrong here?

Contributor

2bits commented Jul 15, 2012

He means let them fix it upstream but don't patch the formula for it because it's cosmetic. @btel where did you get the patch for libpng? Would you add a link to the upstream commit or bug report in the formula comments before def patch please? Also squash to a single commit when you get a chance.

dmwelch commented Aug 9, 2012

FYI: I'm encountering this error, which may or may not be specific to my configuration (it's a fairly fresh install). The steps on the bug fixing checklist did not solve this. It looks like a syntax error, but then why am I'm the only one encountering this???


$ brew install btel/homebrew@3268dd1/Library/Formula/inkscape.rb
######################################################################## 100.0%
Error: /Library/Caches/Homebrew/Formula/inkscape.rb:4: syntax error, unexpected '<'

^
/Library/Caches/Homebrew/Formula/inkscape.rb:5: syntax error, unexpected '<'

^ /Library/Caches/Homebrew/Formula/inkscape.rb:7: syntax error, unexpected '<' ^ /Library/Caches/Homebrew/Formula/inkscape.rb:8: syntax error, unexpected '<' ^ /Library/Caches/Homebrew/Formula/inkscape.rb:8: syntax error, unexpected tIDENTIFIER, expecting $end ^ Please report this bug: https://github.com/mxcl/homebrew/wiki/bug-fixing-checklist /usr/local/Library/Homebrew/formula.rb:379:in `require' /usr/local/Library/Homebrew/formula.rb:379:in`factory' /usr/local/Library/Homebrew/extend/ARGV.rb:14:in `formulae' /usr/local/Library/Homebrew/extend/ARGV.rb:14:in`map' /usr/local/Library/Homebrew/extend/ARGV.rb:14:in `formulae' /usr/local/Library/Homebrew/cmd/install.rb:23:in`install' /usr/local/bin/brew:80:in `send' /usr/local/bin/brew:80

EDIT: This is the correct command (Thanks, @mistydemeo)
$ brew install -v https://raw.github.com/btel/homebrew/master/Library/Formula/inkscape.rb

Contributor

mistydemeo commented Aug 9, 2012

@dmwelch To install from url, use the "raw" version of the formula (you can get the link for that from "View file"), not the HTML version.

Owner

MikeMcQuaid commented Aug 9, 2012

Inkscape offers a binary so don't think we should pull this.

dmwelch commented Aug 9, 2012

@MikeMcQuaid The binary doesn't work on Lion and the nightly Mac build isn't actively supported by the project. Building from source requires a lot of Macports/Fink workarounds (hence Homebrew).

@mistydemeo Thanks for the heads up, that makes a lot more sense!

Contributor

pwnall commented Aug 11, 2012

@MikeMcQuaid Inkscape requires Xquartz on OSX 10.8, and the current Xquartz release isn't working very well for me. I'd love to see a homebrew version that works well.

@btel I tried installing this on OSX 10.8, and got the following error during ./configure.

checking GNU compiler version... 
configure: error: gcc >= 3.0 is needed to compile inkscape

I have XCode 4.4.1 and the updated command-line tools installed.

My config.log is at https://gist.github.com/3321353

Owner

MikeMcQuaid commented Aug 11, 2012

We're not going to make our own XQuartz build. If this doesn't work on Lion then I can see an argument in pulling it. Pretty sure I used Inkscape in binary form on Lion somehow though?

Owner

MikeMcQuaid commented Aug 11, 2012

Can someone test this build and let me know what doesn't work on 10.7/10.8: http://inkscape.org/download/

Contributor

2bits commented Aug 11, 2012

I don't think he's asking for an XQuartz because XQuartz doesn't work well, rather he's asking for an inkscape because the inkscape binary that's built against XQuartz doesn't work well. @pwnall please clarify if I'm incorrect. @MikeMcQuaid I'm not sure how to test their precompiled binary vs. one we build. Anyone have a suggestion to expose something that's broken?

Contributor

2bits commented Aug 11, 2012

@btel please run brew audit inkscape and fix whatever it complains about. Does this need:

depends_on :x11
Owner

MikeMcQuaid commented Aug 11, 2012

Ah, I see. I'd like to know what doesn't work well with their binary. @2bits if it runs under Lion/Mountain Lion that would be a good tea. Don't have XQuartz installed (and aren't going to install it) on Mountain Lion so a bit limited here.

Contributor

NelsonMinar commented Aug 11, 2012

The Inkscape binary at http://inkscape.org/download/ (Inkscape-0.48.2-1-SNOWLEOPARD.dmg) works fine on Lion 10.7.4 with XQuartz. At least it launches fine, I didn't test a lot. The homebrew formula also works fine but has a nicer UI look. Sample screenshots; top is Inkscape's distribution, bottom is homebrew. http://imgur.com/a/dxn1q

Contributor

NelsonMinar commented Aug 11, 2012

I had the same problem pwnall had when building. I believe the root cause is ld: unknown option: -z. I worked around by building with brew install --use-gcc

Owner

MikeMcQuaid commented Aug 11, 2012

The bottom one looks awful, I assume you mean the top is Homebrew?

Contributor

NelsonMinar commented Aug 11, 2012

The bottom one with the greyer background is Homebrew. I prefer it because the font kerning is a little less like a deranged child's refrigerator magnets than the screenshot at top. Also the font hinting is better. And the UI controls look better. Presumably all of this is just a matter of GTK themes, something I know nothing about.

btel's formula is useful and contains significant patches to make Inkscape buildable in Homebrew. I hope it is included.

Owner

MikeMcQuaid commented Aug 11, 2012

We don't include GUI packages that have working OS X binaries I'm afraid so I'm closing this. It's a good candidate for a brew tap. Sorry!

Contributor

pwnall commented Aug 15, 2012

@MikeMcQuaid I hope you'll reconsider.

Small technical detail: inkscape can be used as a command-line tool.
http://inkscape.org/doc/inkscape-man.html

That aside, Homebrew currently packages MacVim (which is great, the brew version is much better than the binary on their site). I hope Inkscape and Gimp will deserve similar exceptions to the GUI rule, because they're very useful for Web development.

Thanks so much!

Contributor

btel commented Aug 21, 2012

Hi, I am back from holidays. I am sorry to see this closed.

I agree with @pwnall - I tend to use inkscape as a command line tool and not for GUI. The binaries do not install the symlinks. Furthermore, the binaries seem to lag behind the release cycle (current version is 0.48.3 and binaries are for 0.48.2 only).

@MikeMcQuaid Is it possible to have this reopened? The response for the request was very positive so far (see comments).

@adamv adamv reopened this Aug 21, 2012

Contributor

adamv commented Aug 21, 2012

Reopened.

maddanio commented Sep 7, 2012

did you ever consider building it with the native ui? it has become kinda abandondend I think, but having it in brew may motivate people to help finishing it?

http://wiki.inkscape.org/wiki/index.php/NativeUIMacOsX

Contributor

2bits commented Sep 7, 2012

@btel welcome back. Please rebase on master and squash to a single commit, plus fix the things that are wrong in Homebrew-0.9.3, like ENV.x11 There is no reason this won't fit somewhere in Homebrew.

I would like this working in homebrew too! Especially a native version (without Xquartz). Given that gimp already offers a native version, I am hoping doing the same for inkscape isn't too hard?

I especially need the console version of inkscape, since I use that to convert svg to pdf (and the output of svg2pdf isn't good enough for complex drawings).

Contributor

2bits commented Sep 11, 2012

Would this need a dep on librsvg? If it can work without XQuartz, that would be a appreciated by some people.

Contributor

Sharpie commented Sep 11, 2012

Would this need a dep on librsvg?

Yes. librsvg is required if I recall.

If it can work without XQuartz, that would be a appreciated by some people.

Unfortunately, building GTK+ against Quartz instead of X11 is problematic as some programs bug out when using a Quartz-based GTK+ build. Now that we have taps, this probably won't be supported as an option in mxcl/master as it is a choice that carries enough of a support burden that it should be maintained by a dedicated group of people who know the software well.

On the other hand, GTK+ 3.x can switch off between Quartz and X11 based on user preference. So, once Inkscape and GIMP become gtk3 compatible, we should have an easy path to Quartz-enabled apps while retaining X11 support for those programs that need it.

Contributor

2bits commented Sep 11, 2012

It's not XQuartz vs X11, it's with or without X. Because superenv doesn't bring in X11 from gtk's depends_on :x11, we might be able to get this to work without X. Am I missing something?

Contributor

jacknagel commented Sep 11, 2012

"Quartz" and "XQuartz" are different things.

Contributor

2bits commented Sep 11, 2012

I thought that distinction is Aqua vs X11.

I think jack & 2bit are correct: XQuartz ~ X11 and Quartz ~ Aqua.

I think a lot of people (including myself) would be interested in a "native inkscape", that is with no XQuartz or X11 dependencies (i.e. using the quartz backend of gtk). Sharpie's post is self explanatory I believe, I don't understand the confusion.

Contributor

Sharpie commented Sep 11, 2012

Am I missing something?

Yes. GTK+, version 2.x can either build with an X11 backend (requiring Apple X11 or XQuartz) or a Quartz backend (which is entirely native and does not run through X11.app). Several programs that use GTK+ for their GUI run fine when the X11 backend is used but are unstable when the Quartz backend is used.

So, getting it to work without X involves building a version of GTK+ that sets other programs up to fail and therefore is an unlikely candidate for an option in mxcl/master.

Contributor

jacknagel commented Sep 11, 2012

Also, to clarify:

Because superenv doesn't bring in X11 from gtk's depends_on :x11

It didn't initially, but that was fixed: b644ed4

Carreau commented Oct 14, 2012

Hi there.

Just wanted to say that this formulae works great on OS X 10.7.4 with the --use-gcc flag.

It has the huge avantage that Inkscape extension works out of the box, which is not the case with binary on the official website.

Also, it feel snappier. Even if this affirmation has nothing scientific.

Contributor

2bits commented Oct 14, 2012

Thanks for the info. I think we are waiting for @btel to update his commit and rebase it with the suggested changes. HB has itself changed a lot in the last couple of months.

Contributor

bpierre commented Oct 24, 2012

I get this error during the configure:

checking for png_read_info in -lpng... no
configure: error: libpng >= 1.2 is needed to compile inkscape

But libpng is already installed:

$ brew install libpng
Error: libpng-1.5.13 already installed
Contributor

samueljohn commented Dec 3, 2012

@btel, I'd love to have Inkscape in homebrew, would you update this pull request? I can test it on Xcode-only systems.

Contributor

derekbrokeit commented Dec 13, 2012

I'd love to see inkscape made available. I would suggest adding perl and python options for an inkscape install.

Also, when I tried to install the above brew, it ran into this error:

checking for zlib.h... yes
checking for Perl development environment... skipped
checking for Python development environment... skipped
checking for LCMS... no
configure: error: --enable-lcms was specified, but appropriate LittleCms development packages could not be found

I suppose is should depend on little-cms

Contributor

adamv commented Dec 16, 2012

Needs to be squashed to a single commit for review!

mgold commented Jan 8, 2013

New version of Inkscape is out and should be updated here.

Tried putting together a formula based on brew create http://sourceforge.net/projects/inkscape/files/inkscape/0.48.4/inkscape-0.48.4.tar.gz and what's here, and aborted when it started to look like it was trying to install half of GNOME. Went crying all the way back to the slightly-out-of-date .dmg.

As has been said, Inkscape is a bit heavyweight for homebrew, but if you can get it to work....

Contributor

samueljohn commented Jan 10, 2013

In my opinion inkscape would still be a welcome addition just beacuse it is so heavy weight that a simple configure && make install does not work.

In the end we could provide a bottle to avoid too long build times.

Owner

MikeMcQuaid commented Jan 10, 2013

They provide binaries from their site!

mgold commented Jan 10, 2013

Not quite. The current .dmg available from their site is two point-point revisions out of date (0.48.2 instead of 0.48.4). Yes, they have tarballs, but isn't the whole point of a package manager that you don't have to do that yourself?

Owner

MikeMcQuaid commented Feb 1, 2013

@adamv @mistydemeo @mxcl @jacknagel @Sharpie Let's vote on this: who wants it to go in or out (want to get it merged or closed)? I'm out.

Contributor

adamv commented Feb 1, 2013

If it compiles I'd allow it in, for sentimental reasons, but I don't have a strong opinion. Just want to see a single commit for review.

Contributor

btel commented Feb 1, 2013

Ok. I squashed the commits and updated to new inkscape. Sorry it took so long!

dmwelch commented Feb 1, 2013

In. Yea. WhateverŠ ;-)

On 2/1/13 4:03 AM, "Bartosz Telenczuk" notifications@github.com wrote:

Ok. I squashed the commits and updated to new inkscape. Sorry it took so long!


Reply to this email directly or view it on GitHub
mxcl#12989 (comment) .

Contributor

adamv commented Feb 3, 2013

This compiled on Snow Leopard, but running it spun up the CPU and no UI ever appeared in X.

Contributor

btel commented Feb 3, 2013

@adamv I can reproduce the problem on Mountain Lion (previously I haven't tried opening a window). I guess this was introduced by the new stable version of inkscape (or one of the dependencies). I reported it on inkscape bug tracker: https://bugs.launchpad.net/inkscape/+bug/1114392

I also added some missing dependencies to the formula.

Contributor

adamv commented Feb 3, 2013

Contributor

btel commented Feb 3, 2013

Done! It works now on my system.

Contributor

adamv commented Feb 3, 2013

Will try when I get a chance, thanks.

Contributor

btel commented Feb 3, 2013

I also fixed the coding style issues.

@adamv adamv closed this in d44b3f6 Feb 3, 2013

I too would like to see a native update on this. Wouldn't take much effort, but I'm not terribly confident on modifying the current formula without some play time to get it right. Has anyone tried modification yet of the formula to get http://wiki.inkscape.org/wiki/index.php/NativeUIMacOsX working?

Carreau commented Mar 15, 2013

@ylluminate the wiki page hasn't been edited in almost 3 years, and last time I tried, I was unable to build the native version by hand.

+1 to native build recipie. Anyone know if there are barriers to this (e.g. unmaintained dependancies) ?
[EDIT] I also just learnt that GIMP has a native mac package - since it has many similar dependancies it seems likely that inkscape could also be built natively.

To get install the theme and get rid of the error:

(inkscape:40141): Gtk-WARNING **: Could not find the icon 'object-visible'. The 'hicolor' theme
was not found either, perhaps you need to install it.

do:

brew install hicolor-icon-theme
Contributor

samueljohn commented Sep 4, 2013

Okay, I added the hicolor-icon-theme to the dependencies of inkscape in 41c71cb.

scpeters pushed a commit to scpeters/homebrew that referenced this pull request Oct 30, 2013

inkscape: depends_on hicolor-icon-theme
As in the discussion [1] that theme is recommended to avoid
a warning if incskape.

[1]: Homebrew#12989 (comment)

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