Opened 14 years ago

Closed 14 years ago

#27724 closed defect (fixed)

libcudd needs some more -m32s

Reported by: peteg42@… Owned by: mij@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: haspatch Cc: ryandesign (Ryan Carsten Schmidt)
Port: libcudd

Description

In the Portfile for libcudd, in the post-build hook, the compiler invocations require the -m32 flag. Otherwise it fails to build on Snow Leopard.

Attachments (1)

libcudd-build_arch.diff (7.1 KB) - added by ryandesign (Ryan Carsten Schmidt) 14 years ago.
proposed patch

Download all attachments as: .zip

Change History (5)

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

Cc: ryandesign@… added
Owner: changed from macports-tickets@… to mij@…

Build succeeds for me on Snow Leopard. If it fails for you, please attach the main.log.

I do notice, though, that the port does not support switching build_arch.

comment:2 in reply to:  1 Changed 14 years ago by peteg42@…

Replying to ryandesign@…:

Build succeeds for me on Snow Leopard. If it fails for you, please attach the main.log.

I do notice, though, that the port does not support switching build_arch.

Ah, so it does. Thanks Ryan!

My problem was and is that I need a 32-bit version of it. I had hacked in some -m32s to get it to build before and so was out of sync with MacPorts. Hacking in those extra -m32s revived my earlier hack.

Perhaps we can leave this bug as a request to support both arch variants.

cheers peter

Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: libcudd-build_arch.diff added

proposed patch

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

Keywords: haspatch added

I've attached a patch to fix this. Mij, may I commit it?

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

Resolution: fixed
Status: newclosed

The maintainer gave approval for this change via email, so I committed it in r74491.

Note: See TracTickets for help on using tickets.