Re: [PATCH spice-protocol 1/3] Add a .gitpublish

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

 



On Fri, Mar 22, 2019 at 02:56:46PM +0100, marcandre.lureau@xxxxxxxxxx wrote:
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> 
> Set a default profile for git-publish.
> (https://github.com/stefanha/git-publish)
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  .gitpublish | 4 ++++
>  1 file changed, 4 insertions(+)
>  create mode 100644 .gitpublish
> 
> diff --git a/.gitpublish b/.gitpublish
> new file mode 100644
> index 0000000..80d46b2
> --- /dev/null
> +++ b/.gitpublish
> @@ -0,0 +1,4 @@
> +[gitpublishprofile "default"]
> +to = spice-devel@xxxxxxxxxxxxxxxxxxxxx
> +prefix = PATCH spice-protocol
> +signoff = true

Setting signoff=true is defeating the purpose of signing off commits IMHO.

The act of adding the S-o-B is usually expected to be a concious decision
by the developer, to assert that their contribution is in compliance with
the DCO (or equivalent contribution rules the project defines [1]).

If tools add the S-o-B automatically with no developer decision involved,
then it looses its meaning and adds no value over the Author: metadata
git already provides by default.

Regards,
Daniel

[1] I'm actually not seeing anything in spice-protocol git that says
    what adding the S-o-B actually means for contributors. Usually
    there should be a copy of the DCO or something in CONTIRBUTING or
    README file to point to it online.
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




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