Opened 11 years ago

Closed 11 years ago

#40884 closed defect (fixed)

libsodium @0.4.5: universal build fails: error: Unsupported byte ordering

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: tresni (Brian Hartvigsen)
Priority: Normal Milestone:
Component: ports Version: 2.2.0
Keywords: Cc:
Port: libsodium

Description

On Mavericks with Xcode 5.0.1, libsodium fails to build when the universal variant is requested.

Attachments (3)

main.log (49.3 KB) - added by ryandesign (Ryan Carsten Schmidt) 11 years ago.
libsodium_nouniversal.diff (564 bytes) - added by tresni (Brian Hartvigsen) 11 years ago.
libsodium+muniversal.diff (552 bytes) - added by tresni (Brian Hartvigsen) 11 years ago.

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: main.log added

comment:1 Changed 11 years ago by tresni (Brian Hartvigsen)

Teach me not to try build +universal... Okay, so I know this issue, testing a possible workaround/patch. Talking to upstream maintainer as well.

Changed 11 years ago by tresni (Brian Hartvigsen)

Attachment: libsodium_nouniversal.diff added

comment:2 Changed 11 years ago by tresni (Brian Hartvigsen)

According to upstream, we should not be doing universal builds anyway. This has not been tested and it is uncertain how the library would actually compile/work in that case.

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

Removing the universal variant from libsodium would mean having to remove the universal variant from its dependents zmq-devel and libchloride as well.

I would rather not remove the universal variant. If upstream doesn't want us building universal all at once, we can probably use the muniversal portgroup to build separately for each arch.

Last edited 11 years ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)

comment:4 Changed 11 years ago by tresni (Brian Hartvigsen)

I've tested and it compiles fine using muniversal PortGroup on 10.8.5 (Not on Mavericks yet...) Upstream says that method of compiling/building should be fine. New patch file coming.

Changed 11 years ago by tresni (Brian Hartvigsen)

Attachment: libsodium+muniversal.diff added

comment:5 Changed 11 years ago by g5pw (Aljaž Srebrnič)

Thanks, commited in r112769.

comment:6 Changed 11 years ago by g5pw (Aljaž Srebrnič)

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