dbuenzli / logs

Logging infrastructure for OCaml
http://erratique.ch/software/logs
ISC License
86 stars 19 forks source link

Update the install layout to have one library per directory. #48

Closed mefyl closed 1 year ago

mefyl commented 1 year ago

This would fix the following issue with dune and has been discussed here.

dbuenzli commented 1 year ago

I think that should work. Could you just opam pin logs --dev and confirm everything's right for you. Here's the layout:

> tree $(opam var lib)/logs
/Users/dbuenzli/.opam/4.14.0/lib/logs
├── META
├── browser
│   ├── logs_browser.a
│   ├── logs_browser.cma
│   ├── logs_browser.cmi
│   ├── logs_browser.cmti
│   ├── logs_browser.cmx
│   ├── logs_browser.cmxa
│   ├── logs_browser.cmxs
│   └── logs_browser.mli
├── cli
│   ├── logs_cli.a
│   ├── logs_cli.cma
│   ├── logs_cli.cmi
│   ├── logs_cli.cmti
│   ├── logs_cli.cmx
│   ├── logs_cli.cmxa
│   ├── logs_cli.cmxs
│   └── logs_cli.mli
├── fmt
│   ├── logs_fmt.a
│   ├── logs_fmt.cma
│   ├── logs_fmt.cmi
│   ├── logs_fmt.cmti
│   ├── logs_fmt.cmx
│   ├── logs_fmt.cmxa
│   ├── logs_fmt.cmxs
│   ├── logs_fmt.mli
│   └── logs_fmt_top_init.ml
├── logs.a
├── logs.cma
├── logs.cmi
├── logs.cmti
├── logs.cmx
├── logs.cmxa
├── logs.cmxs
├── logs.mli
├── logs_top_init.ml
├── lwt
│   ├── logs_lwt.a
│   ├── logs_lwt.cma
│   ├── logs_lwt.cmi
│   ├── logs_lwt.cmti
│   ├── logs_lwt.cmx
│   ├── logs_lwt.cmxa
│   ├── logs_lwt.cmxs
│   └── logs_lwt.mli
├── opam
├── threaded
│   ├── logs_threaded.a
│   ├── logs_threaded.cma
│   ├── logs_threaded.cmi
│   ├── logs_threaded.cmti
│   ├── logs_threaded.cmx
│   ├── logs_threaded.cmxa
│   ├── logs_threaded.cmxs
│   └── logs_threaded.mli
└── top
    ├── logs_top.a
    ├── logs_top.cma
    ├── logs_top.cmx
    ├── logs_top.cmxa
    ├── logs_top.cmxs
    └── logs_top_init_ml

6 directories, 58 files
mefyl commented 1 year ago

That does the trick indeed, thank you very much 🙏