[libvirt] [PATCH 3/4] build: .service files don't need to be executable
Guido Günther
agx at sigxcpu.org
Sun Jan 6 17:46:48 UTC 2013
On Fri, Jan 04, 2013 at 03:36:47PM -0700, Eric Blake wrote:
> See also commit 66ff2dd, where we avoided installing these files
> as executables.
>
> * daemon/Makefile.am (libvirtd.service): Drop chmod.
> * tools/Makefile.am (libvirt-guests.service): Likewise.
> * src/Makefile.am (virtlockd.service, virtlockd.socket):
> Likewise.
> ---
> daemon/Makefile.am | 1 -
> src/Makefile.am | 2 --
> tools/Makefile.am | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/daemon/Makefile.am b/daemon/Makefile.am
> index 60112ff..7bcc7e2 100644
> --- a/daemon/Makefile.am
> +++ b/daemon/Makefile.am
> @@ -354,7 +354,6 @@ libvirtd.service: libvirtd.service.in $(top_builddir)/config.status
> -e 's|[@]sbindir[@]|$(sbindir)|g' \
> -e 's|[@]sysconfdir[@]|$(sysconfdir)|g' \
> < $< > $@-t && \
> - chmod a+x $@-t && \
> mv $@-t $@
>
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index ba685bb..08611fb 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -1709,14 +1709,12 @@ virtlockd.service: locking/virtlockd.service.in $(top_builddir)/config.status
> $(AM_V_GEN)sed \
> -e 's|[@]sbindir[@]|$(sbindir)|g' \
> < $< > $@-t && \
> - chmod a+x $@-t && \
> mv $@-t $@
>
> virtlockd.socket: locking/virtlockd.socket.in $(top_builddir)/config.status
> $(AM_V_GEN)sed \
> -e 's|[@]localstatedir[@]|$(localstatedir)|g' \
> < $< > $@-t && \
> - chmod a+x $@-t && \
> mv $@-t $@
>
>
> diff --git a/tools/Makefile.am b/tools/Makefile.am
> index 58dbfe7..3775914 100644
> --- a/tools/Makefile.am
> +++ b/tools/Makefile.am
> @@ -260,7 +260,6 @@ libvirt-guests.service: libvirt-guests.service.in $(top_builddir)/config.status
> -e 's|[@]sysconfdir[@]|$(sysconfdir)|g' \
> -e 's|[@]libexecdir[@]|$(libexecdir)|g' \
> < $< > $@-t && \
> - chmod a+x $@-t && \
> mv $@-t $@
Now I see what you meant on your reply to my change. ACK.
-- Guido
>
>
> --
> 1.8.0.2
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
>
More information about the libvir-list
mailing list