Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#36938 closed defect (fixed)

vtk5 +carbon fails to build in Mountain Lion

Reported by: ajacobo@… Owned by: adfernandes (Andrew Fernandes)
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: mountainlion Cc:
Port: vtk5

Description (last modified by ryandesign (Ryan Carsten Schmidt))

I'm trying to build vtk5 with the +carbon +python27 variant and I get

Error: org.macports.build for port vtk5 returned: command execution failed

The +tcltk +x11 +python variant builds fine, but the carbon variant doesn't work neither in the stable nor the devel versions of vtk.

Any help? I attach the main.log

Attachments (1)

main.log.bz2 (201.5 KB) - added by ryandesign (Ryan Carsten Schmidt) 11 years ago.
ajacobo's log

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)
Keywords: mountainlion added
Owner: changed from macports-tickets@… to adfernandes@…
Summary: Vtk5 +carbon +python fails to build in Mountain Lionvtk5 +carbon fails to build in Mountain Lion

Please compress large files before attaching them. I've removed your log and attached a compressed version of it.

The log says:

error: vtkCarbonRenderWindow does not work in 64 bit

Sounds like we might need

supported_archs i386 ppc

in the carbon variant. One would think this issue would affect Lion and Snow Leopard users as well.

Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: main.log.bz2 added

ajacobo's log

comment:2 in reply to:  1 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ryandesign@…:

Sounds like we might need

supported_archs i386 ppc

in the carbon variant. One would think this issue would affect Lion and Snow Leopard users as well.

Doing that leads to a different error message on Mountain Lion, which is not surprising since Carbon is deprecated in Mountain Lion. It is an obsolete API that should not be used anymore.

comment:3 Changed 11 years ago by adfernandes (Andrew Fernandes)

Resolution: fixed
Status: newclosed

comment:4 Changed 11 years ago by jmroot (Joshua Root)

It would be even nicer to not define the carbon variant on darwin 12+, rather than making it error out.

comment:5 Changed 11 years ago by adfernandes (Andrew Fernandes)

True, but then we get lint warnings that the x11 and tcltk variants conflict with the non-existing carbon variant.

Given that carbon is really, really deprecated, my feeling is that it's better to remove it entirely from your variant schemes, rather than have it silently dropped. This is never mind the fact that only 32-bit builds are allowed.

Note: See TracTickets for help on using tickets.