OCaml Forge

Detail: [#940] Oasis ignores XCustom fields

Bugs: Browse | Download .csv | Monitor

[#940] Oasis ignores XCustom fields

Date:
2011-03-31 20:02
Priority:
3
State:
Open
Submitted by:
Roma Sokolov (larhat)
Assigned to:
Sylvain Le Gall (gildor-admin)
Product:
OASIS
Due in version:
0.2.1
Resolution:
Accepted As Bug
Severity:
minor
Version:
0.2.0
Component:
Plugin Custom
Operating System:
All
 
URL:
https://github.com/ocaml-batteries-team/batteries-included/blob/master/_oasis
Summary:
Oasis ignores XCustom fields

Detailed description
ocaml batteries has custom build system. There are XCustomConf, XCustomInstall and XCustomUninstall fields in _oasis. But oasis setup doesn't generate right setup.ml for these fields:
BaseSetup.configure = InternalConfigurePlugin.configure;
install = InternalInstallPlugin.install;
uninstall = InternalInstallPlugin.uninstall;

only XCustomBuild from XCustomXXX fields handles by oasis.


Followup

Message
Date: 2011-04-04 07:56
Sender: Sylvain Le Gall

You can leave this as a bug. Will correct it when possible.
Date: 2011-04-03 22:00
Sender: Roma Sokolov

Maybe, oasis should throw error, if somebody uses XCustom* fields without properly tool enabled?

It's consistent with plugin handling: you cann't use something like XMetaName if you're not enabled plugin META.

should be this submitted as feature request?
Date: 2011-03-31 20:57
Sender: Sylvain Le Gall

Use
InstallType: custom (0.2)
and
ConfType: custom (0.2)

(just as BuildType: custom (0.2))

Sylvain Le Gall

Attached Files:

Changes:

Field Old Value Date By
VersionNone2011-04-04 07:56gildor-admin
Severitymajor2011-04-04 07:56gildor-admin
ResolutionNone2011-04-04 07:56gildor-admin
assigned_tonone2011-03-31 20:57gildor-admin