search

ocurrent / obuilder

Experimental "docker build" alternative using btrfs/zfs snapshots
Apache License 2.0
60 stars 17 forks source link

Support Fmt.cli and Logs.cli #74

Closed MisterDA closed 3 years ago

MisterDA commented 3 years ago

The first commit is just a switch to Cmdliner.Arg.&, I find it cleaner and it's more in line with Cmdliner's documentation, but I can remove it if you prefer. (EDIT: removed). The second introduces support for Fmt.cli, that's the --color=always|never|auto flag, and support for setting logs verbosity through the command line. This introduces a bit of noise in the help page, but helps development by not having to patch the source to get more logs. The new options are only available through sub-commands.

One thing: in setup_log, maybe setting both

  Logs.set_level level;
  Logs.Src.set_level Obuilder.log_src level;

is redundant, I'm not sure.

talex5 commented 3 years ago

Thanks!