Opened 11 years ago

Closed 9 years ago

#39755 closed defect (fixed)

libemf @1.0.4: update to 1.0.7

Reported by: ballapete (Peter "Pete" Dyballa) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: haspatch Cc: kurthindenburg (Kurt Hindenburg)
Port: libemf

Description

The upgraded version 1.0.7 of libEMF can be built in 64 bit. This succeeded on my Snow Leopard Mac with Sandy Bridge processor. 

checksums       md5 f1011f5cc254aa228be78704fe5f9960 \
                  sha1 ef61f8d73b6e68785b973cbbe8c449138f40b740 \
                  rmd160 cad46f73982f5e22fee51e6278053dcb5ea29b98

Attachments (2)

Portfile​ (1.4 KB) - added by ballapete (Peter "Pete" Dyballa) 11 years ago.
Updated Portfile
Portfile.diff​ (1.8 KB) - added by ballapete (Peter "Pete" Dyballa) 11 years ago.
diff -u /opt/local/var/macports/sources/lil.fr.rsync.macports.org/release/tarballs/ports/graphics/libemf/Portfile Portfile

Download all attachments as: .zip

Change History (11)

Changed 11 years ago by ballapete (Peter "Pete" Dyballa)

Attachment: Portfile​ added

Updated Portfile

comment:1 Changed 11 years ago by larryv (Lawrence Velázquez)

Keywords: 64-bit removed
Summary: Upgrade of libEMF to version 1.0.7 → libemf @1.0.4: update to 1.0.7

Please attach a unified diff of the Portfile.

comment:2 Changed 11 years ago by ballapete (Peter "Pete" Dyballa)

I could not test the Portfile on any other 64 bit hardwire. To make it simple I commented a dozen lines I do not understand.

Changed 11 years ago by ballapete (Peter "Pete" Dyballa)

Attachment: Portfile.diff​ added

diff -u /opt/local/var/macports/sources/lil.fr.rsync.macports.org/release/tarballs/ports/graphics/libemf/Portfile Portfile

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

So you're saying it now builds 64-bit? That's good news.

What about the PowerPC-specific directives you commented out? Are you sure they're no longer needed?

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

Keywords: haspatch added

comment:5 in reply to:  3 Changed 11 years ago by ballapete (Peter "Pete" Dyballa)

Replying to ryandesign@…:

So you're saying it now builds 64-bit? That's good news.

Definitely!

What about the PowerPC-specific directives you commented out? Are you sure they're no longer needed?

I made a test on PPC Leopard (G4, Mac OS X 10.5.8) and it shows that the commented out code is still necessary, otherwise the compiler complains that there is no CPU support for this hardware. It seems that for PPC (32-bit HW) a particular #define is necessary – and for PPC64 maybe another one. I cannot test this because I do not have a G5.

​http://sourceforge.net/mailarchive/forum.php?set=custom&viewmonth=&viewday=&forum_name=libemf-devel&style=nested&max_rows=75&submit=Change+View mentions patches for PPC64 – after the Vietnamese spam. Maybe they're all integrated now.

comment:6 in reply to:  3 Changed 11 years ago by ballapete (Peter "Pete" Dyballa)

Replying to ryandesign@…:

So you're saying it now builds 64-bit? That's good news.

What about the PowerPC-specific directives you commented out? Are you sure they're no longer needed?

On a PPC Tiger (Mac OS X 10.4.11), G4, the out commented part has to be commented in in order to perform the build. Here the same effect is visible that the default compiler does not provide the proper defines that libEMF would understand.

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

Cc: khindenburg@… added

Cc Me!

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

What's the status of this patch? It works fine on Mavericks. I can't test it on anything else.

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

Resolution: → fixed
Status: new → closed

I went ahead updated to 1.0.8 r147759

Note: See TracTickets for help on using tickets.