Opened 10 years ago

Closed 9 years ago

#44671 closed enhancement (fixed)

Coin: Install framework by default and remove variants

Reported by: mbrethen Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: kurthindenburg (Kurt Hindenburg)
Port: Coin

Description

By default Coin is set up as a Mac OS X framework. In addition:

1) The '--enable-next-minor' configure option was removed by the developer; therefore I have removed the 'devel' variant.

2) The '--enable-threadsafe' configure option was removed by the developer; I have removed this variant.

3) Removed the manpages variant; this doesn't build with the framework.

4) fontconfig, freetype, zlib and bzip2 dependencies added (they were always required).

5) qt4-x11 is ancient and no longer builds; removed this option.

Attachments (1)

Coin.diff (3.9 KB) - added by mbrethen 10 years ago.

Download all attachments as: .zip

Change History (19)

comment:1 Changed 10 years ago by mbrethen

Changed framework to a subport. Manpages variant works with non-framework install only.

Edit: see also Ticket #44679

Last edited 10 years ago by mbrethen (previous) (diff)

comment:2 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:3 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

css (sorry I don't know your real name), do you have an opinion on these patches? Did you want to add openmaintainer for these ports?

comment:4 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Mark, I had to change the below in your patch - otherwise it was trying to download coin- which it couldn't find.

bitbucket.setup     Coin3D coin 3.1.3 Coin-$

to

bitbucket.setup     Coin3D Coin 3.1.3 Coin-$

comment:5 Changed 10 years ago by mbrethen

That's odd, it worked for me.

I just checked, the download url is https://bitbucket.org/Coin3D/coin/downloads/Coin-3.1.3.tar.gz

Last edited 10 years ago by mbrethen (previous) (diff)

comment:6 Changed 10 years ago by mf2k (Frank Schima)

Cc: css@… removed
Owner: changed from macports-tickets@… to css@…
Version: 2.3.1

comment:7 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

bitbucket.setup Coin3D coin 3.1.3 Coin-$

https://bitbucket.org/Coin3D/coin/downloads/coin-3.1.3.tar.gz

bitbucket.setup Coin3D Coin 3.1.3 Coin-$

https://bitbucket.org/Coin3D/Coin/downloads/Coin-3.1.3.tar.gz

Only the bottom one works - I see your patch didn't actually change the bitbucket line - I wonder since I'm running macports trunk if something's changed.

comment:8 Changed 10 years ago by mbrethen

The URL I posted does work, both that you list do not. The filename is incorrect in the first, and the "coin" directory in the second should not be capitalized.

comment:9 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Ryan fixed the distfile download in #44103

comment:10 Changed 10 years ago by mbrethen

So, are we waiting a response from css to commit?

comment:11 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Is coin and coin-frameworks suppose to be able to be installed at the same time?

If so, this needs fixed - if not, they should be marked as conflicts

Error: Failed to activate Coin-framework: Image error: /opt/local/bin/coin-config is being used by the active Coin port.  Please deactivate this port first, or use 'port -f activate Coin-framework' to force the activation.

comment:12 Changed 10 years ago by mbrethen

You would install one or the other, not both. I'll fix it.

Changed 10 years ago by mbrethen

Attachment: Coin.diff added

comment:13 Changed 10 years ago by mbrethen

Coin Portfile patch updated; Adds 'conflicts' keyword.

comment:14 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

I'm not sure the subport is correct - it doesn't work for me - I'll try to find time to test more

comment:15 Changed 9 years ago by mbrethen

What error are you getting?

Edit: I forgot to upload framework-prefix.patch; I'll do It later today.

Last edited 9 years ago by mbrethen (previous) (diff)

comment:16 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

After an re-install, it works now - user error

Is there any reason that +manpages is only for coin?

sudo port install coin-framework +manpages

comment:17 Changed 9 years ago by mbrethen

As I recall there was a problem with installing manpages as a framework. I don't remember specifics, but it would take some patching to fix.

Edit: for a history see ticket #26265

Last edited 9 years ago by mbrethen (previous) (diff)

comment:18 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

thanks r127233

Note: See TracTickets for help on using tickets.