On Tue, Feb 15, 2011 at 6:42 PM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com">berrange@redhat.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Tue, Feb 15, 2011 at 03:03:52PM +0800, Lyre wrote:<br>
> * libvirt-php.obs.spec: this file should works for Fedora 14,<br>
>   openSuSE 11.3, and SLES 11 SP1 on openSuSE Build Service or<br>
>   localhost.<br>
> * Makefile.am: added EXTRA_DIST = libvirt-php.obs.spec<br>
> ---<br>
>  Makefile.am          |    2 +<br>
>  aclocal.m4           |    4 +-<br>
>  libvirt-php.obs.spec |   80 ++++++++++++++++++++++++++++++++++++++++++++++++++<br>
>  3 files changed, 84 insertions(+), 2 deletions(-)<br>
>  create mode 100644 libvirt-php.obs.spec<br>
<br>
</div>Why do we need another RPM specfile ? Can't you just make your<br>
changes to the existing php-libvirt.spec ?<br>
<br>
Regards,<br>
Daniel<br>
<font color="#888888">--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/%7Edanberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</font></blockquote></div><br>Hi Daniel:<br><br>The problem was, I'm not quite sure what to do with this spec file.<br><br>Currently, It may be better to name the spec file libvirt-php.spec.<br><br>This spec file contains many if...else for compatibility, some macros such as %{php_zend_api} are not acceptable in SLES. The original php-libvirt.spec may be preferred for Fedora.<br>
<br>At last, I intend to use the openSuSE Build Service for easier deployment. It will be easier for me to maintain this file if it is rejected.<br>