Daniel P. Berrange wrote:
On Thu, Jul 10, 2008 at 10:10:25AM -0400, Mohammed Morsi wrote:
> Attached is a small fix to get libvirt working w/ autobuild. Autobuild
> attempts to create and test the RPM, which lists the PolicyKit as one of
> the files to be installed, and thus if it is not installed, autobuild
> reports an error and fails. The was to get this file installed is to
> specify --with-polkit to ./autogen which I added to the autobuild.sh
> build script. (also the syntax check errors broke autobuild, but these
> have been fixed according to the latest email on the list). Knock on
> wood, after this libvirt should work w/ autobuild and will be built
> nightly, and I can finish getting oVirt working with it.
>
Hmm, this shouldn't impact the RPM spec at all. The 'autogen.shj' call
in the autobuild.sh only applies to the non-RPM build. The RPM build
is done via 'configure' in the spec file itself. I think there's
something else going wrong you're hitting instead.
I'm not expert on autotools, but doesnt autogen.sh traditionally take
the configure.ac and Makefile.am files and generate the configure script
and make rules? While the build process isn't being held up at the
configure or make stages, rather the rpmbuild stage, rpmbuild depends on
the output of configure / make, which if --with-polkit is not enabled,
will result in the PolityKit file not being included in the build and
thus an error when rpm sees it listed under the files in the spec but
not the actual file itself. Feel free to correct me if I'm wrong.
-Mo
> Index: autobuild.sh
> ===================================================================
> RCS file: /data/cvs/libvirt/autobuild.sh,v
> retrieving revision 1.7
> diff -u -r1.7 autobuild.sh
> --- autobuild.sh 29 May 2008 20:43:08 -0000 1.7
> +++ autobuild.sh 9 Jul 2008 18:09:11 -0000
> @@ -16,7 +16,8 @@
>
> ./autogen.sh --prefix=$AUTOBUILD_INSTALL_ROOT \
> --enable-test-coverage \
> - --enable-compile-warnings=error
> + --enable-compile-warnings=error \
> + --with-polkit
>
> make
> make install
>
Daniel