[Spice-devel] [spice v1] syntax-check: Fix missing AUTHORS

2016-12-05 Thread Victor Toso
From: Victor Toso 

Signed-off-by: Victor Toso 
---
 .mailmap | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index c05ba00..d66229a 100644
--- a/.mailmap
+++ b/.mailmap
@@ -8,3 +8,4 @@
  
  
  
+ 
-- 
2.9.3

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


[Spice-devel] [spice v1] syntax-check: Fix missing AUTHORS

2017-02-28 Thread Victor Toso
From: Victor Toso 

Signed-off-by: Victor Toso 
---
 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
 소병철 
 Cédric Bosdonnat 
 Christian Ruppert 
+Christophe de Dinechin 
 Cole Robinson 
 Daniel P. Berrange 
 Dan Kenigsberg 
-- 
2.9.3

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


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

2016-12-05 Thread Frediano Ziglio
> 
> From: Victor Toso 
> 
> Signed-off-by: Victor Toso 
> ---
>  .mailmap | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/.mailmap b/.mailmap
> index c05ba00..d66229a 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -8,3 +8,4 @@
>   
>   
>   
> + 

Acked-by: Frediano Ziglio 

Still thanks to Pavel for the Gitlab setup, very useful!

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


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

2016-12-05 Thread Frediano Ziglio
> > 
> > From: Victor Toso 
> > 
> > Signed-off-by: Victor Toso 
> > ---
> >  .mailmap | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/.mailmap b/.mailmap
> > index c05ba00..d66229a 100644
> > --- a/.mailmap
> > +++ b/.mailmap
> > @@ -8,3 +8,4 @@
> >   
> >   
> >   
> > + 
> 
> Acked-by: Frediano Ziglio 
> 
> Still thanks to Pavel for the Gitlab setup, very useful!
> 

Green light! https://gitlab.com/spice/spice/builds.

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


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

2016-12-05 Thread Victor Toso
Hi,

On Mon, Dec 05, 2016 at 09:39:51AM -0500, Frediano Ziglio wrote:
> > > 
> > > From: Victor Toso 
> > > 
> > > Signed-off-by: Victor Toso 
> > > ---
> > >  .mailmap | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/.mailmap b/.mailmap
> > > index c05ba00..d66229a 100644
> > > --- a/.mailmap
> > > +++ b/.mailmap
> > > @@ -8,3 +8,4 @@
> > >   
> > >   
> > >   
> > > + 
> > 
> > Acked-by: Frediano Ziglio 
> > 
> > Still thanks to Pavel for the Gitlab setup, very useful!
> > 
> 
> Green light! https://gitlab.com/spice/spice/builds.
> 
> Frediano

Yes, it's super useful :)
Kudos Pavel!


signature.asc
Description: PGP signature
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel


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

2017-02-28 Thread Daniel P. Berrange
On Tue, Feb 28, 2017 at 02:37:25PM +0100, Victor Toso wrote:
> From: Victor Toso 
> 
> Signed-off-by: Victor Toso 
> ---
>  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
>  소병철 
>  Cédric Bosdonnat 
>  Christian Ruppert 
> +Christophe de Dinechin 
>  Cole Robinson 
>  Daniel P. Berrange 
>  Dan Kenigsberg 

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
-- 
|: 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/ :|
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel


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

2017-03-06 Thread Frediano Ziglio
> 
> From: Victor Toso 
> 
> Signed-off-by: Victor Toso 
> ---
>  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
>  소병철 
>  Cédric Bosdonnat 
>  Christian Ruppert 
> +Christophe de Dinechin 
>  Cole Robinson 
>  Daniel P. Berrange 
>  Dan Kenigsberg 

Acked-by: Frediano Ziglio 

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


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

2017-03-06 Thread Victor Toso
Hi,

On Tue, Feb 28, 2017 at 01:44:42PM +, Daniel P. Berrange wrote:
> On Tue, Feb 28, 2017 at 02:37:25PM +0100, Victor Toso wrote:
> > From: Victor Toso 
> > 
> > Signed-off-by: Victor Toso 
> > ---
> >  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
> >  소병철 
> >  Cédric Bosdonnat 
> >  Christian Ruppert 
> > +Christophe de Dinechin 
> >  Cole Robinson 
> >  Daniel P. Berrange 
> >  Dan Kenigsberg 
>
> 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/ :|


signature.asc
Description: PGP signature
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel