<div dir="ltr"><div>While setting up a development environment on a clean Fedora 28 host,</div><div>I got some errors.</div><div><br></div><div><div>I followed the instructions in</div><div><a href="http://libguestfs.org/guestfs-building.1.html">http://libguestfs.org/guestfs-building.1.html</a><br></div><div><br></div><div>    dnf builddep libguestfs<br></div><div>    ./autogen.sh</div></div><div><br></div><div>autogen.sh failed:</div><div><br></div><div>./configure: line 57694: syntax error near unexpected token `external'<br></div><div><div>./configure: line 57694: `AM_GNU_GETTEXT(external)'</div></div><div><br></div><div>Checking the entire log show that autoreconf failed, but the script</div><div>continued:</div><div><br></div><div>...</div><div>m4/guestfs-libraries.m4:151: warning: macro 'AM_GNU_GETTEXT' not found in library<br></div><div>autoreconf: <a href="http://configure.ac">configure.ac</a>: tracing</div><div>autoreconf: running: true --copy</div><div>m4/guestfs-libraries.m4:151: warning: macro 'AM_GNU_GETTEXT' not found in library</div><div>autoreconf: running: /usr/bin/autoconf</div><div>configure:57694: error: possibly undefined macro: AM_GNU_GETTEXT</div><div>      If this token and others are legitimate, please use m4_pattern_allow.</div><div>      See the Autoconf documentation.</div><div>autoreconf: /usr/bin/autoconf failed with exit status: 1</div><div><br></div><div>We can fix this by adding "set -e" in bootstrap. Any reason not to do this?</div><div><br></div><div>I found old thread here about this:</div><div><a href="https://gb.redhat.com/archives/libguestfs/2018-February/msg00017.html">https://gb.redhat.com/archives/libguestfs/2018-February/msg00017.html</a><br></div><div><br></div><div>Richard suggested to install gettext-devel, and it solve this issue after</div><div>git clean -dxf and running ./autogen.sh again.</div><div><br></div><div>So it seems that gettest-devel should be in build requirements.</div><div><br></div><div>After fixing the first issue I found that jansson-devel is also missing.</div><div><br></div><div>Should'nt we add these packages to the build requirements?</div><div><br></div><div>Next issues - trying to test upload to ovirt, I found that these</div><div>packages should be installed:</div><div><br></div><div>nbdkit</div><div>nbdkit-plugin-python3<br></div><div>python3-ovirt-engine-sdk4<br></div><div><br></div><div>Should we document these requirements in the rhv-upload-plugin?</div><div><br></div><div>Nir</div><div><br></div><div><br></div></div>