From e6a5ef9508b8a86a2948f35a4482e927e5edf833 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Dimino?= Date: Tue, 3 Jul 2018 10:49:18 +0100 Subject: [PATCH] Add the lib_root and libexec_root install sections (#947) Signed-off-by: Jeremie Dimino --- CHANGES.md | 2 + doc/dune-files.rst | 2 + src/install.ml | 189 +++++++++++++++++++++++++-------------------- src/install.mli | 26 ++++--- 4 files changed, 125 insertions(+), 94 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 427426fe..d12e9e75 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -108,6 +108,8 @@ next - Do not require opam-installer anymore (#941, @diml) +- Add the `lib_root` and `libexec_root` install sections (#947, @diml) + 1.0+beta20 (10/04/2018) ----------------------- diff --git a/doc/dune-files.rst b/doc/dune-files.rst index 23daa2db..26391c1b 100644 --- a/doc/dune-files.rst +++ b/doc/dune-files.rst @@ -603,7 +603,9 @@ The syntax is as follows: manual. The following sections are available: - ``lib`` +- ``lib_root`` - ``libexec`` +- ``libexec_root`` - ``bin`` - ``sbin`` - ``toplevel`` diff --git a/src/install.ml b/src/install.ml index 40ab01d1..ee937fb5 100644 --- a/src/install.ml +++ b/src/install.ml @@ -3,7 +3,9 @@ open Import module Section = struct type t = | Lib + | Lib_root | Libexec + | Libexec_root | Bin | Sbin | Toplevel @@ -18,109 +20,130 @@ module Section = struct let compare : t -> t -> Ordering.t = compare let to_string = function - | Lib -> "lib" - | Libexec -> "libexec" - | Bin -> "bin" - | Sbin -> "sbin" - | Toplevel -> "toplevel" - | Share -> "share" - | Share_root -> "share_root" - | Etc -> "etc" - | Doc -> "doc" - | Stublibs -> "stublibs" - | Man -> "man" - | Misc -> "misc" + | Lib -> "lib" + | Lib_root -> "lib_root" + | Libexec -> "libexec" + | Libexec_root -> "libexec_root" + | Bin -> "bin" + | Sbin -> "sbin" + | Toplevel -> "toplevel" + | Share -> "share" + | Share_root -> "share_root" + | Etc -> "etc" + | Doc -> "doc" + | Stublibs -> "stublibs" + | Man -> "man" + | Misc -> "misc" let of_string = function - | "lib" -> Some Lib - | "libexec" -> Some Libexec - | "bin" -> Some Bin - | "sbin" -> Some Sbin - | "toplevel" -> Some Toplevel - | "share" -> Some Share - | "share_root" -> Some Share_root - | "etc" -> Some Etc - | "doc" -> Some Doc - | "stublibs" -> Some Stublibs - | "man" -> Some Man - | "misc" -> Some Misc - | _ -> None + |"lib" -> Some Lib + |"lib_root" -> Some Lib_root + |"libexec" -> Some Libexec + |"libexec_root" -> Some Libexec_root + |"bin" -> Some Bin + |"sbin" -> Some Sbin + |"toplevel" -> Some Toplevel + |"share" -> Some Share + |"share_root" -> Some Share_root + |"etc" -> Some Etc + |"doc" -> Some Doc + |"stublibs" -> Some Stublibs + |"man" -> Some Man + |"misc" -> Some Misc + | _ -> None let t = let open Sexp.Of_sexp in enum - [ "lib" , Lib - ; "libexec" , Libexec - ; "bin" , Bin - ; "sbin" , Sbin - ; "toplevel" , Toplevel - ; "share" , Share - ; "share_root" , Share_root - ; "etc" , Etc - ; "doc" , Doc - ; "stublibs" , Stublibs - ; "man" , Man - ; "misc" , Misc + [ "lib" , Lib + ; "lib_root" , Lib_root + ; "libexec" , Libexec + ; "libexec_root" , Libexec_root + ; "bin" , Bin + ; "sbin" , Sbin + ; "toplevel" , Toplevel + ; "share" , Share + ; "share_root" , Share_root + ; "etc" , Etc + ; "doc" , Doc + ; "stublibs" , Stublibs + ; "man" , Man + ; "misc" , Misc ] let should_set_executable_bit = function - | Lib -> false - | Libexec -> true - | Bin -> true - | Sbin -> true - | Toplevel -> false - | Share -> false - | Share_root -> false - | Etc -> false - | Doc -> false - | Stublibs -> true - | Man -> false - | Misc -> false + | Lib + | Lib_root + | Toplevel + | Share + | Share_root + | Etc + | Doc + | Man + | Misc + -> false + | Libexec + | Libexec_root + | Bin + | Sbin + | Stublibs + -> true module Paths = struct type t = - { lib : Path.t - ; libexec : Path.t - ; bin : Path.t - ; sbin : Path.t - ; toplevel : Path.t - ; share : Path.t - ; share_root : Path.t - ; etc : Path.t - ; doc : Path.t - ; stublibs : Path.t - ; man : Path.t + { lib : Path.t + ; lib_root : Path.t + ; libexec : Path.t + ; libexec_root : Path.t + ; bin : Path.t + ; sbin : Path.t + ; toplevel : Path.t + ; share : Path.t + ; share_root : Path.t + ; etc : Path.t + ; doc : Path.t + ; stublibs : Path.t + ; man : Path.t } let make ~package ~destdir ?(libdir=Path.relative destdir "lib") () = let package = Package.Name.to_string package in - { bin = Path.relative destdir "bin" - ; sbin = Path.relative destdir "sbin" - ; toplevel = Path.relative libdir "toplevel" - ; share_root = Path.relative libdir "share" - ; stublibs = Path.relative libdir "lib/stublibs" - ; man = Path.relative destdir "man" - ; lib = Path.relative libdir package - ; libexec = Path.relative libdir package - ; share = Path.relative destdir ("share/" ^ package) - ; etc = Path.relative destdir ("etc/" ^ package) - ; doc = Path.relative destdir ("doc/" ^ package) + let lib_root = libdir in + let libexec_root = libdir in + let share_root = Path.relative destdir "share" in + let etc_root = Path.relative destdir "etc" in + let doc_root = Path.relative destdir "doc" in + { lib_root + ; libexec_root + ; share_root + ; bin = Path.relative destdir "bin" + ; sbin = Path.relative destdir "sbin" + ; man = Path.relative destdir "man" + ; toplevel = Path.relative libdir "toplevel" + ; stublibs = Path.relative libdir "stublibs" + ; lib = Path.relative lib_root package + ; libexec = Path.relative libexec_root package + ; share = Path.relative share_root package + ; etc = Path.relative etc_root package + ; doc = Path.relative doc_root package } let get t section = match section with - | Lib -> t.lib - | Libexec -> t.libexec - | Bin -> t.bin - | Sbin -> t.sbin - | Toplevel -> t.toplevel - | Share -> t.share - | Share_root -> t.share_root - | Etc -> t.etc - | Doc -> t.doc - | Stublibs -> t.stublibs - | Man -> t.man - | Misc -> invalid_arg "Install.Paths.get" + | Lib -> t.lib + | Lib_root -> t.lib_root + | Libexec -> t.libexec + | Libexec_root -> t.libexec_root + | Bin -> t.bin + | Sbin -> t.sbin + | Toplevel -> t.toplevel + | Share -> t.share + | Share_root -> t.share_root + | Etc -> t.etc + | Doc -> t.doc + | Stublibs -> t.stublibs + | Man -> t.man + | Misc -> invalid_arg"Install.Paths.get" end end diff --git a/src/install.mli b/src/install.mli index 6d7f120d..2ce0920a 100644 --- a/src/install.mli +++ b/src/install.mli @@ -5,7 +5,9 @@ open Stdune module Section : sig type t = | Lib + | Lib_root | Libexec + | Libexec_root | Bin | Sbin | Toplevel @@ -27,17 +29,19 @@ module Section : sig type section = t type t = - { lib : Path.t - ; libexec : Path.t - ; bin : Path.t - ; sbin : Path.t - ; toplevel : Path.t - ; share : Path.t - ; share_root : Path.t - ; etc : Path.t - ; doc : Path.t - ; stublibs : Path.t - ; man : Path.t + { lib : Path.t + ; lib_root : Path.t + ; libexec : Path.t + ; libexec_root : Path.t + ; bin : Path.t + ; sbin : Path.t + ; toplevel : Path.t + ; share : Path.t + ; share_root : Path.t + ; etc : Path.t + ; doc : Path.t + ; stublibs : Path.t + ; man : Path.t } val make