Bug 81868 - [maintainer update] lang/ocaml : take maintainership and adding some options
Summary: [maintainer update] lang/ocaml : take maintainership and adding some options
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Thierry Thomas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-03 17:40 UTC by Burelle Marwan
Modified: 2005-06-08 22:20 UTC (History)
1 user (show)

See Also:


Attachments
new-ocaml-ports (3.51 KB, text/plain)
2005-06-03 17:40 UTC, Burelle Marwan
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Burelle Marwan 2005-06-03 17:40:01 UTC

Add to ocaml's port 2 options and take maintainership, since OCaml port has none.

First, WITH_COMPILER_LIB, in order to install compiler libs (additionnal libs needed to build programs manipulating OCaml modules, like the ocaml/cduce interface for example.)

Second, WITH_OPT_DEFAULT, telling the port to replace ocamlc and other tools by there native and faster version (ocamlc.opt and other) (bytecode version are preserved under the name [tool_name].byte, .opt version are also preserved.)
Comment 1 Thierry Thomas freebsd_committer freebsd_triage 2005-06-04 23:11:19 UTC
State Changed
From-To: open->feedback


1) Since compiler libs are needed to build other ports, it 
would be better to have it defined by default, and to replace 

.if defined(WITH_COMPILER_LIB) 

by 

.if !defined(WITHOUT_COMPILER_LIB) 

2) When installed as a port with WITH_OPT_DEFAULT, optimized 
code is a link; however, when installed as a package, the code 
is duplicated: '@exec ln' should be used. 



Comment 2 Thierry Thomas freebsd_committer freebsd_triage 2005-06-04 23:11:19 UTC
Responsible Changed
From-To: freebsd-ports-bugs->thierry


Take it.
Comment 3 Thierry Thomas freebsd_committer freebsd_triage 2005-06-08 22:18:34 UTC
State Changed
From-To: feedback->closed


Committed, thanks! 

Note: with some modifications (.for / .endfor). 

WITH_OPT_DEFAULT not removed, since it is not activated by 
default.