Opened 11 years ago

Last modified 2 years ago

#23040 assigned defect

ocamlmklib doesn't understand -arch

Reported by: jeremyhu (Jeremy Huddleston Sequoia) Owned by: pmetzger (Perry E. Metzger)
Priority: Normal Milestone:
Component: ports Version: 1.8.1
Keywords: Cc: achim.passen@…, fh@…, cooljeanius (Eric Gallager)
Port: ocaml

Description

Building ocamlimages, we execute ocamlmklib from ocaml which doesn't understand our LDFLAGS (including -arch)

ocamlmklib -oc ci_jpeg -L/opt/local/lib -arch x86_64 jpegread.o jpegwrite.o -ljpeg
Unknown option -arch
Don't know what to do with x86_64
Usage: ocamlmklib [options] <.cmo|.cma|.cmx|.cmxa|.ml|.mli|.o|.a|.obj|.lib|.dll files>
Options are:
  -cclib <lib>   C library passed to ocamlc -a or ocamlopt -a only
  -ccopt <opt>   C option passed to ocamlc -a or ocamlopt -a only

Attachments (2)

Portfile-caml-sqlite3.diff (1.2 KB) - added by fh@… 9 years ago.
Portfile-caml-sqlite3.2.diff (577 bytes) - added by fh@… 9 years ago.
Better handling of arch flags as suggested by jmr@…

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by achim.passen@…

Cc: achim.passen@… added

Cc Me!

comment:2 Changed 10 years ago by ryandesign (Ryan Schmidt)

Has duplicate #24215.

Changed 9 years ago by fh@…

Attachment: Portfile-caml-sqlite3.diff added

comment:3 Changed 9 years ago by fh@…

Path built following #22168 and #71393 to remove the -arch flag that is not supported by ocamlmklib. The package now builds and installs.

Notice that the patch also upgrades to version 1.5.9

comment:4 Changed 9 years ago by fh@…

Cc: fh@… added

Cc Me!

comment:5 Changed 9 years ago by mww@…

Resolution: fixed
Status: newclosed

thanks, commited!

comment:6 Changed 9 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: closedreopened

r76475 only fixed caml-sqlite3. This is a general problem; and in fact the description for this ticket only specifically mentions ocamlimages.

comment:7 Changed 9 years ago by jmroot (Joshua Root)

Fixed camlimages in r77600. I'd suggest doing it this way for all ports that require it, since clearing the archflags as was done for caml-sqlite3 breaks build_arch support entirely.

Changed 9 years ago by fh@…

Better handling of arch flags as suggested by jmr@…

comment:8 Changed 7 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:9 Changed 4 years ago by mf2k (Frank Schima)

Owner: mww@… deleted
Status: reopenedassigned

See #53012.

comment:10 Changed 2 years ago by pmetzger (Perry E. Metzger)

Owner: set to pmetzger
Note: See TracTickets for help on using tickets.