Opened 10 years ago

Closed 10 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 10 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 10 years ago by mbrethen

What error are you getting?

If Coin is installed, you have to remove it before installing the framework subport.

Version 1, edited 10 years ago by mbrethen (previous) (next) (diff)

comment:16 Changed 10 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 10 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 10 years ago by mbrethen (previous) (diff)

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

Resolution: fixed
Status: newclosed

thanks r127233

Note: See TracTickets for help on using tickets.