<div dir="ltr">> I'm not sure what editor you use, but either setting it up so that there is a > '\n' in the end of the file (at least by default unless you remove it or > something) would probably be favourable.<div> </div><div>I'm sorry. I'll check the settings of my editor.</div><div> </div><div>> I, personally, prefer the usual `--enable-rust=yes|check|no`</div>> semantics<div> </div><div>I see. I'll fix this.</div><div> </div><div>Thank you for your comments!</div><div> </div><div>Regards,</div><div>Hiroyuki</div></div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">2019年7月3日(水) 16:58 Martin Kletzander <<a href="mailto:mkletzan@redhat.com">mkletzan@redhat.com</a>>: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Jul 02, 2019 at 10:14:19PM +0900, Hiroyuki Katsura wrote: >From: Hiroyuki_Katsura <<a href="mailto:hiroyuki.katsura.0513@gmail.com" target="_blank">hiroyuki.katsura.0513@gmail.com</a>> > >--- > Makefile.am | 4 ++++ > <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 3 +++ > generator/Makefile.am | 3 +++ > generator/<a href="http://bindtests.ml" rel="noreferrer" target="_blank">bindtests.ml</a> | 3 +++ > generator/bindtests.mli | 1 + > generator/<a href="http://main.ml" rel="noreferrer" target="_blank">main.ml</a> | 5 +++++ > generator/<a href="http://rust.ml" rel="noreferrer" target="_blank">rust.ml</a> | 34 ++++++++++++++++++++++++++++++++++ > generator/rust.mli | 19 +++++++++++++++++++ > m4/guestfs-rust.m4 | 30 ++++++++++++++++++++++++++++++ > rust/Cargo.toml | 6 ++++++ > rust/Makefile.am | 29 +++++++++++++++++++++++++++++ > rust/src/.gitkeep | 0 > rust/tests/.gitkeep | 0 > 13 files changed, 137 insertions(+) > create mode 100644 generator/<a href="http://rust.ml" rel="noreferrer" target="_blank">rust.ml</a> > create mode 100644 generator/rust.mli > create mode 100644 m4/guestfs-rust.m4 > create mode 100644 rust/Cargo.toml > create mode 100644 rust/Makefile.am > create mode 100644 rust/src/.gitkeep > create mode 100644 rust/tests/.gitkeep > [...] >diff --git a/generator/rust.mli b/generator/rust.mli >new file mode 100644 >index 000000000..4fef55d4e >--- /dev/null >+++ b/generator/rust.mli >@@ -0,0 +1,19 @@ >+(* libguestfs >+ * Copyright (C) 2009-2019 Red Hat Inc. >+ * >+ * This program is free software; you can redistribute it and/or modify >+ * it under the terms of the GNU General Public License as published by >+ * the Free Software Foundation; either version 2 of the License, or >+ * (at your option) any later version. >+ * >+ * This program is distributed in the hope that it will be useful, >+ * but WITHOUT ANY WARRANTY; without even the implied warranty of >+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >+ * GNU General Public License for more details. >+ * >+ * You should have received a copy of the GNU General Public License >+ * along with this program; if not, write to the Free Software >+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA >+*) >+ >+val generate_rust: unit -> unit >\ No newline at end of file I'm not sure what editor you use, but either setting it up so that there is a '\n' in the end of the file (at least by default unless you remove it or something) would probably be favourable. >diff --git a/m4/guestfs-rust.m4 b/m4/guestfs-rust.m4 >new file mode 100644 >index 000000000..f46ce960b >--- /dev/null >+++ b/m4/guestfs-rust.m4 >@@ -0,0 +1,30 @@ >+# libguestfs >+# Copyright (C) 2009-2019 Red Hat Inc. >+# >+# This program is free software; you can redistribute it and/or modify >+# it under the terms of the GNU General Public License as published by >+# the Free Software Foundation; either version 2 of the License, or >+# (at your option) any later version. >+# >+# This program is distributed in the hope that it will be useful, >+# but WITHOUT ANY WARRANTY; without even the implied warranty of >+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >+# GNU General Public License for more details. >+# >+# You should have received a copy of the GNU General Public License >+# along with this program; if not, write to the Free Software >+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >+ >+dnl Rust >+AC_ARG_ENABLE([rust], >+ AS_HELP_STRING([--disable-rust], [disable Rust language bindings]), >+ [], >+ [enable_rust=yes]) >+AS_IF([test "x$enable_rust" != "xno"],[ >+ AC_CHECK_PROG([RUSTC],[rustc],[rustc],[no]) >+ AC_CHECK_PROG([CARGO],[cargo],[cargo],[no]) >+],[ >+ RUSTC=no >+ CARGO=no >+ ]) >+AM_CONDITIONAL([HAVE_RUST],[test "x$RUSTC" != "xno" && test "x$CARGO" != "xno"]) So even if someone explicitly specifies --enable-rust without having the tools, they will not get the bindings built, but they will also not get an error nor confirmation in the configure output, so it is really hard to figure out what is happening. I, personally, prefer the usual `--enable-rust=yes|check|no` semantics. Of course this can be fixed later, do not let this rust bindings-unrelated review stop others from reviewing the important parts ;) [...] >diff --git a/rust/Makefile.am b/rust/Makefile.am >new file mode 100644 >index 000000000..e8bf27894 >--- /dev/null >+++ b/rust/Makefile.am >@@ -0,0 +1,29 @@ >+# libguestfs golang bindings >+# Copyright (C) 2019 Red Hat Inc. >+# >+# This program is free software; you can redistribute it and/or modify >+# it under the terms of the GNU General Public License as published by >+# the Free Software Foundation; either version 2 of the License, or >+# (at your option) any later version. >+# >+# This program is distributed in the hope that it will be useful, >+# but WITHOUT ANY WARRANTY; without even the implied warranty of >+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >+# GNU General Public License for more details. >+# >+# You should have received a copy of the GNU General Public License >+# along with this program; if not, write to the Free Software >+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >+ >+include $(top_srcdir)/<a href="http://subdir-rules.mk" rel="noreferrer" target="_blank">subdir-rules.mk</a> >+ >+generator_built = \ >+ src/<a href="http://lib.rs" rel="noreferrer" target="_blank">lib.rs</a> >+ >+EXTRA_DIST = \ >+ $(generator_built) >+ >+if HAVE_RUST >+ >+endif Maybe something is missing here? Martin </blockquote></div>