Pino Toscano
2019-Apr-01 15:04 UTC
[Libguestfs] [PATCH] OCaml tools: fix 3999 -> 3339 typo
RFC 3339 is the actual RFC for date/time strings. Typo found by Martin 'eagle eyes' Kletzander. Fixes commit f79129b8dc92470e3a5597daf53c84038bd6859e. --- common/mltools/tools_utils-c.c | 4 ++-- common/mltools/tools_utils.ml | 4 ++-- lib/guestfs.pod | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/common/mltools/tools_utils-c.c b/common/mltools/tools_utils-c.c index b015dcace..3b80091c0 100644 --- a/common/mltools/tools_utils-c.c +++ b/common/mltools/tools_utils-c.c @@ -39,7 +39,7 @@ extern value guestfs_int_mllib_inspect_decrypt (value gv, value gpv, value keysv); extern value guestfs_int_mllib_set_echo_keys (value unitv); extern value guestfs_int_mllib_set_keys_from_stdin (value unitv); -extern value guestfs_int_mllib_rfc3999_date_time_string (value unitv); +extern value guestfs_int_mllib_rfc3339_date_time_string (value unitv); /* Interface with the guestfish inspection and decryption code. */ int echo_keys = 0; @@ -108,7 +108,7 @@ guestfs_int_mllib_set_keys_from_stdin (value unitv) } value -guestfs_int_mllib_rfc3999_date_time_string (value unitv) +guestfs_int_mllib_rfc3339_date_time_string (value unitv) { CAMLparam1 (unitv); char buf[64]; diff --git a/common/mltools/tools_utils.ml b/common/mltools/tools_utils.ml index de42df600..127180225 100644 --- a/common/mltools/tools_utils.ml +++ b/common/mltools/tools_utils.ml @@ -32,7 +32,7 @@ and key_store_key external c_inspect_decrypt : Guestfs.t -> int64 -> (string * key_store_key) list -> unit = "guestfs_int_mllib_inspect_decrypt" external c_set_echo_keys : unit -> unit = "guestfs_int_mllib_set_echo_keys" "noalloc" external c_set_keys_from_stdin : unit -> unit = "guestfs_int_mllib_set_keys_from_stdin" "noalloc" -external c_rfc3999_date_time_string : unit -> string = "guestfs_int_mllib_rfc3999_date_time_string" +external c_rfc3339_date_time_string : unit -> string = "guestfs_int_mllib_rfc3339_date_time_string" type machine_readable_fn = { pr : 'a. ('a, unit, string, unit) format4 -> 'a; @@ -93,7 +93,7 @@ let log_as_json msgtype msg | Some { pr } -> let json = [ "message", JSON.String msg; - "timestamp", JSON.String (c_rfc3999_date_time_string ()); + "timestamp", JSON.String (c_rfc3339_date_time_string ()); "type", JSON.String msgtype; ] in pr "%s\n" (JSON.string_of_doc ~fmt:JSON.Compact json) diff --git a/lib/guestfs.pod b/lib/guestfs.pod index 3c1d635c5..af944ddb7 100644 --- a/lib/guestfs.pod +++ b/lib/guestfs.pod @@ -3295,7 +3295,7 @@ within a single line, below it is indented for readability): C<type> can be: C<message> for progress messages, C<info> for information messages, C<warning> for warning messages, and C<error> for error message. -C<timestamp> is the L<RFC 3999|https://www.ietf.org/rfc/rfc3339.txt> +C<timestamp> is the L<RFC 3339|https://www.ietf.org/rfc/rfc3339.txt> timestamp of the message. In addition to that, a subset of these tools support an extra string -- 2.20.1
Martin Kletzander
2019-Apr-01 15:12 UTC
Re: [Libguestfs] [PATCH] OCaml tools: fix 3999 -> 3339 typo
On Mon, Apr 01, 2019 at 05:04:47PM +0200, Pino Toscano wrote:>RFC 3339 is the actual RFC for date/time strings. >Typo found by Martin 'eagle eyes' Kletzander. >You spelt "angel" wrong.
Richard W.M. Jones
2019-Apr-01 16:42 UTC
Re: [Libguestfs] [PATCH] OCaml tools: fix 3999 -> 3339 typo
On Mon, Apr 01, 2019 at 05:04:47PM +0200, Pino Toscano wrote:> RFC 3339 is the actual RFC for date/time strings. > Typo found by Martin 'eagle eyes' Kletzander. > > Fixes commit f79129b8dc92470e3a5597daf53c84038bd6859e.Looks good, ACK. Rich.> common/mltools/tools_utils-c.c | 4 ++-- > common/mltools/tools_utils.ml | 4 ++-- > lib/guestfs.pod | 2 +- > 3 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/common/mltools/tools_utils-c.c b/common/mltools/tools_utils-c.c > index b015dcace..3b80091c0 100644 > --- a/common/mltools/tools_utils-c.c > +++ b/common/mltools/tools_utils-c.c > @@ -39,7 +39,7 @@ > extern value guestfs_int_mllib_inspect_decrypt (value gv, value gpv, value keysv); > extern value guestfs_int_mllib_set_echo_keys (value unitv); > extern value guestfs_int_mllib_set_keys_from_stdin (value unitv); > -extern value guestfs_int_mllib_rfc3999_date_time_string (value unitv); > +extern value guestfs_int_mllib_rfc3339_date_time_string (value unitv); > > /* Interface with the guestfish inspection and decryption code. */ > int echo_keys = 0; > @@ -108,7 +108,7 @@ guestfs_int_mllib_set_keys_from_stdin (value unitv) > } > > value > -guestfs_int_mllib_rfc3999_date_time_string (value unitv) > +guestfs_int_mllib_rfc3339_date_time_string (value unitv) > { > CAMLparam1 (unitv); > char buf[64]; > diff --git a/common/mltools/tools_utils.ml b/common/mltools/tools_utils.ml > index de42df600..127180225 100644 > --- a/common/mltools/tools_utils.ml > +++ b/common/mltools/tools_utils.ml > @@ -32,7 +32,7 @@ and key_store_key > external c_inspect_decrypt : Guestfs.t -> int64 -> (string * key_store_key) list -> unit = "guestfs_int_mllib_inspect_decrypt" > external c_set_echo_keys : unit -> unit = "guestfs_int_mllib_set_echo_keys" "noalloc" > external c_set_keys_from_stdin : unit -> unit = "guestfs_int_mllib_set_keys_from_stdin" "noalloc" > -external c_rfc3999_date_time_string : unit -> string = "guestfs_int_mllib_rfc3999_date_time_string" > +external c_rfc3339_date_time_string : unit -> string = "guestfs_int_mllib_rfc3339_date_time_string" > > type machine_readable_fn = { > pr : 'a. ('a, unit, string, unit) format4 -> 'a; > @@ -93,7 +93,7 @@ let log_as_json msgtype msg > | Some { pr } -> > let json = [ > "message", JSON.String msg; > - "timestamp", JSON.String (c_rfc3999_date_time_string ()); > + "timestamp", JSON.String (c_rfc3339_date_time_string ()); > "type", JSON.String msgtype; > ] in > pr "%s\n" (JSON.string_of_doc ~fmt:JSON.Compact json) > diff --git a/lib/guestfs.pod b/lib/guestfs.pod > index 3c1d635c5..af944ddb7 100644 > --- a/lib/guestfs.pod > +++ b/lib/guestfs.pod > @@ -3295,7 +3295,7 @@ within a single line, below it is indented for readability): > C<type> can be: C<message> for progress messages, C<info> for > information messages, C<warning> for warning messages, and C<error> > for error message. > -C<timestamp> is the L<RFC 3999|https://www.ietf.org/rfc/rfc3339.txt> > +C<timestamp> is the L<RFC 3339|https://www.ietf.org/rfc/rfc3339.txt> > timestamp of the message. > > In addition to that, a subset of these tools support an extra string > -- > 2.20.1 > > _______________________________________________ > Libguestfs mailing list > Libguestfs@redhat.com > https://www.redhat.com/mailman/listinfo/libguestfs-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org
Reasonably Related Threads
- [PATCH 0/4] OCaml tools: output messages as JSON machine
- [PATCH 4/4] OCaml tools: output messages into JSON for machine readable
- [PATCH] mltools, options: support --allow-discards when decrypting LUKS devices
- [PATCH v2 4/4] OCaml tools: output messages into JSON for machine readable
- [PATCH v2 0/4] OCaml tools: output messages as JSON machine