On Wed, Nov 09, 2022 at 02:43:52PM -0600, Eric Blake wrote:
@@ -51,7 +51,9 @@ typedef enum exit_code {
OK = 0,
ERROR = 1, /* all script error codes are mapped to this */
MISSING = 2, /* method missing */
- RET_FALSE = 3 /* script exited with code 3 meaning false */
+ RET_FALSE = 3, /* script exited with code 3 meaning false */
+ /* Adjust methods.c:sh_dump_plugin when defining new codes */
+ /* 4-7 is reserved since 1.8; handle like ERROR for now */
} exit_code;
extern exit_code call (const char **argv)
diff --git a/plugins/sh/methods.c b/plugins/sh/methods.c
index 5c1ee5f8..4a75a3a0 100644
--- a/plugins/sh/methods.c
+++ b/plugins/sh/methods.c
@@ -1,5 +1,5 @@
/* nbdkit
- * Copyright (C) 2018-2021 Red Hat Inc.
+ * Copyright (C) 2018-2022 Red Hat Inc.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are
@@ -58,6 +58,10 @@ sh_dump_plugin (void)
const char *args[] = { script, method, NULL };
CLEANUP_FREE_STRING string o = empty_vector;
+ /* Dump information about the sh/eval features */
+ printf ("max_known_status=%d\n", RET_FALSE);
It also took me a double-take here to realise that this prints the
maximum known status number instead of a boolean of some kind. So as
Laszlo suggests this might be better with some enum sentinel to record
the first reserved value.
Anyway it's all good so:
Reviewed-by: Richard W.M. Jones <rjones(a)redhat.com>
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming and virtualization blog:
http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html