Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0007219OCamltools (ocaml{lex,yacc,dep,debug,...})public2016-04-11 11:042017-02-23 16:50
Reporterwhitequark 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityalways
StatusacknowledgedResolutionopen 
PlatformOSOS Version
Product Version4.02.3 
Target VersionFixed in Version 
Summary0007219: ocamlmklib should respect OCAMLPARAM
Description... specifically, at least ccopts and cclib, but probably other variables too.

This makes it very annoying to pass -I/-L paths to native dependencies through e.g. OASIS.
Tagsjunior_job
Attached Files

- Relationships

-  Notes
(0015715)
gasche (developer)
2016-04-11 22:29

I think this is a fine feature suggestion, but the implementation is a bit tedious because ocamlmklib currently implements its own bare-metal argument handling logic, sensibly simpler than the rest of the compiler. That means you cannot just call the compiler OCAMLPARAM-parsing function and be done with it; the best way to go is probably to reimplement the command-line argument parsing logic of ocamlmklib.

Would anyone be interested in proposing a patch to do this?
(0015716)
whitequark (developer)
2016-04-11 22:32

What I did for now is implemented an OCAMLMKLIBPARAM (https://github.com/whitequark/opam-cross-android/blob/master/packages/ocaml-android32.4.02.3/files/patches/ocamlmklib.patch [^]). But this is rather suboptimal.
(0015717)
whitequark (developer)
2016-04-11 22:35

FWIW ocamlmklib's options are sufficiently different from ocamlc's that I'm not sure if it even makes sense to reuse Compenv.read_OCAMLPARAM; what I would do is to call Compenv.parse_args, and then manually reimplement the parameters relevant for ocamlmklib, of which there are very few.

- Issue History
Date Modified Username Field Change
2016-04-11 11:04 whitequark New Issue
2016-04-11 22:29 gasche Note Added: 0015715
2016-04-11 22:29 gasche Status new => acknowledged
2016-04-11 22:29 gasche Tag Attached: junior_job
2016-04-11 22:32 whitequark Note Added: 0015716
2016-04-11 22:35 whitequark Note Added: 0015717
2016-04-14 17:36 doligez Target Version => 4.03.1+dev
2016-12-07 16:37 doligez Category OCaml tools (ocaml{lex,yacc,dep,browser,debug}) => OCaml tools (ocaml{lex,yacc,dep,debug})
2017-02-16 14:01 doligez Target Version 4.03.1+dev => undecided
2017-02-23 16:45 doligez Category OCaml tools (ocaml{lex,yacc,dep,debug}) => tools (ocaml{lex,yacc,dep,debug,...})
2017-02-23 16:50 doligez Severity minor => feature
2017-02-23 16:50 doligez Target Version undecided =>


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker