Projects
New Ticket     Wiki     Browse Source     Timeline     Roadmap     Bug Reports     Search

Ticket #12597 (closed enhancement: fixed)

Opened 15 months ago

Last modified 7 months ago

ImageMagick: add +rsvg variant

Reported by: bulk@… Owned by: ryandesign@…
Priority: Normal Milestone: Port Enhancements
Component: ports Version: 1.5.0
Keywords: Cc: jmr@…
Port:

Description

Hi there,

I'd love to see a "+rsvg" variant for image magick. All it needs to do is add a dependency on 'librsvg' and ImageMagick does the rest (no additional commands/configure flags needed.. it auto detects it).

thanks,

--nickg

Change History

Changed 12 months ago by css@…

  • milestone set to Port Enhancements

Changed 9 months ago by ryandesign@…

  • owner changed from macports-tickets@… to ryandesign@…
  • summary changed from RFE ImageMagick: add +rsvg variant to ImageMagick: add +rsvg variant

I'll assign this to me, since I'm the ImageMagick maintainer. Not saying I'll resolve it immediately though.

Additional configure arguments are needed. Auto-detecting other packages is not appropriate for software installed using MacPorts. It would mean that two users would "magically" get two different versions of ImageMagick, simply if one user already had librsvg installed and the other did not. We don't want that. We want everyone who builds a port with a given set of variants to get the same software (modulo major OS versions and processor architectures). If ImageMagick auto-detects librsvg if installed, we need to add configure arguments to disable librsvg explicitly unless this new +rsvg variant is chosen.

Changed 8 months ago by bulk@…

My point was it works just like all the other libraries in ImageMagick. assuming wikitext doesn't mangle the text, it's something like the following:

Add to configure.args:

  --without-rsvg

Since right now, it WILL use rsvg if you install librsvg first

add somewhere:

variant rsvg description {Include librsvg library support} {
    depends_lib-append  port:librsvg
    configure.args-delete    --without-rsvg
    configure.args-append   --with-rsvg
}

I hope this helps,

thanks,

-nickg

Changed 7 months ago by ryandesign@…

  • cc jmr@… added

Has duplicate #15036 which has a patch which looks a lot like what bulk@modp wrote above.

Changed 7 months ago by ryandesign@…

  • status changed from new to closed
  • resolution set to fixed

Committed the patch from #15036 in r36122. Thanks everyone.

Note: See TracTickets for help on using tickets.