[PATCH] gitpublish: Tweak prefix

Andrea Bolognani posted 1 patch 4 months, 3 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20231201161451.1098057-1-abologna@redhat.com
.gitpublish | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] gitpublish: Tweak prefix
Posted by Andrea Bolognani 4 months, 3 weeks ago
Having the name of the project as part of the prefix was useful
back when we used the mailing list for all subprojects, but
these days the only patches that are sent to the list are for
the main library so it no longer makes sense to include this.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
 .gitpublish | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitpublish b/.gitpublish
index 1c89c0039e..01c5c1af21 100644
--- a/.gitpublish
+++ b/.gitpublish
@@ -1,5 +1,5 @@
 [gitpublishprofile "default"]
 base = master
 to = devel@lists.libvirt.org
-prefix = libvirt PATCH
+prefix = PATCH
 suppresscc = misc-by
-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org
Re: [PATCH] gitpublish: Tweak prefix
Posted by Martin Kletzander 4 months, 3 weeks ago
On Fri, Dec 01, 2023 at 05:14:51PM +0100, Andrea Bolognani wrote:
>Having the name of the project as part of the prefix was useful
>back when we used the mailing list for all subprojects, but
>these days the only patches that are sent to the list are for
>the main library so it no longer makes sense to include this.
>
>Signed-off-by: Andrea Bolognani <abologna@redhat.com>
>---
> .gitpublish | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/.gitpublish b/.gitpublish
>index 1c89c0039e..01c5c1af21 100644
>--- a/.gitpublish
>+++ b/.gitpublish
>@@ -1,5 +1,5 @@
> [gitpublishprofile "default"]
> base = master
> to = devel@lists.libvirt.org
>-prefix = libvirt PATCH
>+prefix = PATCH

You can remove it as that's the default ;)

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

> suppresscc = misc-by
>-- 
>2.43.0
>_______________________________________________
>Devel mailing list -- devel@lists.libvirt.org
>To unsubscribe send an email to devel-leave@lists.libvirt.org
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org
Re: [PATCH] gitpublish: Tweak prefix
Posted by Andrea Bolognani 4 months, 3 weeks ago
On Tue, Dec 05, 2023 at 10:51:11AM +0100, Martin Kletzander wrote:
> On Fri, Dec 01, 2023 at 05:14:51PM +0100, Andrea Bolognani wrote:
> > +++ b/.gitpublish
> > @@ -1,5 +1,5 @@
> > [gitpublishprofile "default"]
> > base = master
> > to = devel@lists.libvirt.org
> > -prefix = libvirt PATCH
> > +prefix = PATCH
>
> You can remove it as that's the default ;)

I thought about that, but it wouldn't do the right thing if the user
has set format.subjectPrefix to something non-standard. This is more
robust.

> Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

Thanks!

-- 
Andrea Bolognani / Red Hat / Virtualization
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org