Re: php libvirt show active snapshot

2020-04-21 Thread Andrea Bolognani
On Tue, 2020-04-21 at 10:10 +0200, Michal Privoznik wrote:
> On 4/21/20 1:36 AM, Todd Lewis wrote:
> > Works perfectly thank you.
> > I did get an issue when trying to compile an rpm
> > 
> > The autobuild did run into an error when trying to create the rpm.
> > Not a big deal. I can make my own spec file.
> > 
> > EXTRA_RELEASE=""
> 
> 1: ^^^
> 
> > if [ -x /usr/bin/rpmbuild ]
> > then
> >rpmbuild --nodeps \
> >   --define "extra_release $EXTRA_RELEASE" \
> >   --define "_sourcedir `pwd`" \
> >   -ba --clean libvirt-php.spec
> > fi
> > pwd
> > error: Macro %extra_release has empty body
> > error: Macro %extra_release has empty body
> > error: Macro %extra_release has empty body
> 
> D'oh. The line [1] shouldn't be there. But actually, whole autobuild 
> script is a bit misleading. The way I build RPMs is autogen.sh + make 
> rpm. Would you mind if I remove the autobuild script?

FWIW there were autobuild.sh scripts in most repositories and I have
made an effort to get rid of them something like a year ago, so if
there are any remaining I would certainly welcome removing them.

-- 
Andrea Bolognani / Red Hat / Virtualization



Re: php libvirt show active snapshot

2020-04-21 Thread Michal Privoznik

On 4/21/20 1:36 AM, Todd Lewis wrote:

Works perfectly thank you.
I did get an issue when trying to compile an rpm

The autobuild did run into an error when trying to create the rpm.
Not a big deal. I can make my own spec file.

EXTRA_RELEASE=""


1: ^^^


if [ -x /usr/bin/rpmbuild ]
then
   rpmbuild --nodeps \
  --define "extra_release $EXTRA_RELEASE" \
  --define "_sourcedir `pwd`" \
  -ba --clean libvirt-php.spec
fi
pwd
error: Macro %extra_release has empty body
error: Macro %extra_release has empty body
error: Macro %extra_release has empty body


D'oh. The line [1] shouldn't be there. But actually, whole autobuild 
script is a bit misleading. The way I build RPMs is autogen.sh + make 
rpm. Would you mind if I remove the autobuild script?


Michal