Opened 12 years ago

Closed 12 years ago

#33111 closed update (fixed)

Update of py27-enable

Reported by: petrus.hyvonen@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: mf2k (Frank Schima), jjstickel@…, jjstickel (Jonathan Stickel)
Port: py-enable

Description

Attached is an updated version of the Enthought Enable package py27-enable

Attachments (6)

Portfile (1.3 KB) - added by petrus.hyvonen@… 12 years ago.
Portfile.2 (1.4 KB) - added by petrus.hyvonen@… 12 years ago.
py enable file in unified python way
py-enable_Portfile (1.6 KB) - added by jjstickel (Jonathan Stickel) 12 years ago.
enable_clang_patch.diff (1.6 KB) - added by jjstickel (Jonathan Stickel) 12 years ago.
no-64-bit-quartz.diff (628 bytes) - added by petrus.hyvonen@… 12 years ago.
Patch corresponding to https://github.com/enthought/enable/pull/47/files#diff-0
Portfile.3 (1.6 KB) - added by petrus.hyvonen@… 12 years ago.
Updated portfile to include 64-bit patch

Download all attachments as: .zip

Change History (21)

Changed 12 years ago by petrus.hyvonen@…

Attachment: Portfile added

comment:1 Changed 12 years ago by mf2k (Frank Schima)

Cc: macsforever2000@… added
Version: 2.0.3

The existing py26-enable port needs to use the new unified python portgroup and then a py27 can easily be added. Please submit a patch for that from the existing py26-enable port. I may get to this myself soon, but your assistance will make this go faster.

comment:2 Changed 12 years ago by jjstickel@…

Cc: jjstickel@… added

Cc Me!

Changed 12 years ago by petrus.hyvonen@…

Attachment: Portfile.2 added

py enable file in unified python way

comment:3 Changed 12 years ago by petrus.hyvonen@…

The patches that was in 3.6 version I commented out in the uploaded portfile. Are they needed and in that case they need to be redone?

comment:4 Changed 12 years ago by jjstickel (Jonathan Stickel)

Cc: jjstickel@… added

Cc Me!

comment:5 Changed 12 years ago by jjstickel (Jonathan Stickel)

I modified some parts of the Portfile, including adding a license and sha256 checksum. I am fairly sure the patches are not needed anymore, and so I deleted them.

comment:6 in reply to:  1 Changed 12 years ago by jjstickel (Jonathan Stickel)

Replying to macsforever2000@…:

Please submit a patch for that from the existing py26-enable port.

Much has changed, and so the patch is quite long. Would you like it uploaded in addition to the Portfile?

comment:7 Changed 12 years ago by mf2k (Frank Schima)

Port: py-enable added

I suppose it's fine not to. To be clear, what other ports/tickets need to be updated along with this one? I'll try to get to this soon.

comment:8 Changed 12 years ago by jjstickel (Jonathan Stickel)

All of the existing enthought ports should be updated in one go. I think tickets/patches exist for all of them now. I will summarize on ticket #33108 (py27-traits).

comment:9 Changed 12 years ago by petrus.hyvonen@…

After upgrading to new xcode 4.3 compiler, and a complete reinstall when macports 2.0.4 is out, I do not manage to get py27-enable to build again.

First error is:

In file included from kiva/agg/agg-24/font_freetype/agg_font_freetype.cpp:18: In file included from kiva/agg/agg-24/font_freetype/agg_font_freetype.h:29: kiva/agg/agg-24/include/agg_scanline_u.h:467:30: error: type 'base_type' (aka 'agg::scanline_u8') is not a direct or virtual base of 'scanline32_u8_am<AlphaMask>'

scanline32_u8_am() : base_type(), m_alpha_mask(0) {}

But I get same error when I try to make enable package standalone withot ports, so maybe an issue with the package itself.

comment:10 in reply to:  9 Changed 12 years ago by jjstickel (Jonathan Stickel)

Replying to petrus.hyvonen@…:

After upgrading to new xcode 4.3 compiler, and a complete reinstall when macports 2.0.4 is out, I do not manage to get py27-enable to build again.

I just got a previously used Macbook Pro with Lion. Using Xcode 4.2, I get the same error when building py27-enable. On my Snow-Leopard machine with Xcode 3.2.6, it still builds fine. Therefore, it looks like the error is related to Xcode. You might want to ask on the enthought-dev or EPD-users list.

comment:11 Changed 12 years ago by erik.tollerud@…

I encountered the same problem and traced it to its source - it seems to be caused by Xcode switching from gcc to clang at 4.something or other. Turns out clang is pickier about a few things that cause compilation of agg 2.4 (which is bundled inside enable) to fail.

https://github.com/enthought/enable/pull/46 has the fix - it's been applied to the github version of enable, but if you want to get this working now, you could just apply that patch.

comment:12 in reply to:  11 Changed 12 years ago by jjstickel (Jonathan Stickel)

Replying to erik.tollerud@…:

I encountered the same problem and traced it to its source - it seems to be caused by Xcode switching from gcc to clang at 4.something or other. Turns out clang is pickier about a few things that cause compilation of agg 2.4 (which is bundled inside enable) to fail.

https://github.com/enthought/enable/pull/46 has the fix - it's been applied to the github version of enable, but if you want to get this working now, you could just apply that patch.

Thanks for this information. I'll shortly upload a patchfile and revised portfile. For some strange reason I do not understand, the patchfile downloaded from github would not apply cleanly. I had to manually make the edits and create a patchfile. I suspect something to do with the encoding or newline characters.

Changed 12 years ago by jjstickel (Jonathan Stickel)

Attachment: py-enable_Portfile added

Changed 12 years ago by jjstickel (Jonathan Stickel)

Attachment: enable_clang_patch.diff added

comment:13 Changed 12 years ago by petrus.hyvonen@…

Note also this one: https://github.com/enthought/enable/pull/47

This was needed for me to get it working.

comment:14 in reply to:  13 Changed 12 years ago by jjstickel (Jonathan Stickel)

Replying to petrus.hyvonen@…:

Note also this one: https://github.com/enthought/enable/pull/47

This was needed for me to get it working.

Petrus, I'll let you provide the patchfile and update the Porftile for this one. Thanks.

Changed 12 years ago by petrus.hyvonen@…

Attachment: no-64-bit-quartz.diff added

Changed 12 years ago by petrus.hyvonen@…

Attachment: Portfile.3 added

Updated portfile to include 64-bit patch

comment:15 Changed 12 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.