New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #27724 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

libcudd needs some more -m32s

Reported by: peteg42@… Owned by: mij@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: haspatch Cc: ryandesign@…
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

libcudd-build_arch.diff (7.1 KB) - added by ryandesign@… 2 years ago.
proposed patch

Change History

comment:1 follow-up: ↓ 2 Changed 2 years ago by ryandesign@…

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

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 2 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 2 years ago by ryandesign@…

proposed patch

comment:3 Changed 2 years ago by ryandesign@…

  • Keywords haspatch added

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

comment:4 Changed 2 years ago by ryandesign@…

  • Status changed from new to closed
  • Resolution set to fixed

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

Note: See TracTickets for help on using tickets.