Re: [spice v1] syntax-check: Fix missing AUTHORS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Tue, Feb 28, 2017 at 01:44:42PM +0000, Daniel P. Berrange wrote:
> On Tue, Feb 28, 2017 at 02:37:25PM +0100, Victor Toso wrote:
> > From: Victor Toso <me@xxxxxxxxxxxxxx>
> > 
> > Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> > ---
> >  AUTHORS | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/AUTHORS b/AUTHORS
> > index 2b0fe24..99f6c18 100644
> > --- a/AUTHORS
> > +++ b/AUTHORS
> > @@ -28,6 +28,7 @@ Patches also contributed by
> >      소병철 <byungchul.so@xxxxxxxxxxx>
> >      Cédric Bosdonnat <cbosdonnat@xxxxxxxx>
> >      Christian Ruppert <idl0r@xxxxxxx>
> > +    Christophe de Dinechin <dinechin@xxxxxxxxxx>
> >      Cole Robinson <crobinso@xxxxxxxxxx>
> >      Daniel P. Berrange <berrange@xxxxxxxxxx>
> >      Dan Kenigsberg <danken@xxxxxxxxxx>
>
> FWIW, git can trivially auto-fill an AUTHORS file at make dist time.
> Libvirt does it as follows:
>
> Rename AUTHORS to AUTHORS.in, replace the names beneath the
> "Patches also contributed by" line, with "#authorslist#" then
> add to Makefile.am
>
> dist-hook: gen-AUTHORS
>
> .PHONY: gen-AUTHORS
> gen-AUTHORS:
>         $(AM_V_GEN)if test -d $(srcdir)/.git; then \
>             out="`cd $(srcdir) && git log --pretty=format:'%aN <%aE>' | sort -u`" && \
>             perl -p -e "s/#authorslist#// and print '$$out'" \
>               < $(srcdir)/AUTHORS.in > $(distdir)/AUTHORS-tmp && \
>             mv -f $(distdir)/AUTHORS-tmp $(distdir)/AUTHORS ; \
>         fi
>
> Regards,
> Daniel

Ah, sorry, forgot to reply to this.

Yes, I'm just being lazy to patch this in better way like your
suggestion. FWIW grilo-plugins has this release.mk file that works fine
too

https://git.gnome.org/browse/grilo-plugins/tree/release.mk

Cheers,
        toso

> --
> |: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
> |: http://libvirt.org              -o-             http://virt-manager.org :|
> |: http://entangle-photo.org       -o-    http://search.cpan.org/~danberr/ :|

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]