all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Disable emacs-diffs mails for scratch branches
@ 2017-01-28  9:56 David Engster
  2017-01-28 15:35 ` Stefan Monnier
  2017-01-29  2:31 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: David Engster @ 2017-01-28  9:56 UTC (permalink / raw)
  To: emacs-devel

Subject says it all. Could we please do that? I think it's pretty normal
for scratch-Branches being deleted and pushed anew, and for large
branches like cedet-merge or cedet-tests this needlessly creates
hundreds of mails each time.

-David



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Disable emacs-diffs mails for scratch branches
  2017-01-28  9:56 Disable emacs-diffs mails for scratch branches David Engster
@ 2017-01-28 15:35 ` Stefan Monnier
  2017-01-28 21:00   ` David Engster
  2017-01-29  2:31 ` Lars Ingebrigtsen
  1 sibling, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2017-01-28 15:35 UTC (permalink / raw)
  To: emacs-devel

> Subject says it all. Could we please do that? I think it's pretty normal
> for scratch-Branches being deleted and pushed anew, and for large
> branches like cedet-merge or cedet-tests this needlessly creates
> hundreds of mails each time.

I agree that those 300 emails dumps are more annoying than useful.
But FWIW, I see two problems with your suggestion:
1- I don't know how to do it.
2- I do find it very useful to see commits on scratch branches when
   those branches are used for gradual development (e.g. the byte-switch
   branch).

Maybe the problem can be solved by sending one email per push instead of
one email per commit.


        Stefan




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Disable emacs-diffs mails for scratch branches
  2017-01-28 15:35 ` Stefan Monnier
@ 2017-01-28 21:00   ` David Engster
  2017-01-29  2:09     ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: David Engster @ 2017-01-28 21:00 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier writes:
>> Subject says it all. Could we please do that? I think it's pretty normal
>> for scratch-Branches being deleted and pushed anew, and for large
>> branches like cedet-merge or cedet-tests this needlessly creates
>> hundreds of mails each time.
>
> I agree that those 300 emails dumps are more annoying than useful.
> But FWIW, I see two problems with your suggestion:
> 1- I don't know how to do it.

We'd have to adapt the post-receive hook, or whatever else is triggering
the mails.

> 2- I do find it very useful to see commits on scratch branches when
>    those branches are used for gradual development (e.g. the byte-switch
>    branch).

How about simply not naming those branches with the 'scratch' prefix,
then?  I was under the impression that 'scratch' means "can be
force-pushed/deleted any time, don't base your work onto it".

-David



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Disable emacs-diffs mails for scratch branches
  2017-01-28 21:00   ` David Engster
@ 2017-01-29  2:09     ` Stefan Monnier
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Monnier @ 2017-01-29  2:09 UTC (permalink / raw)
  To: David Engster; +Cc: emacs-devel

>>> Subject says it all. Could we please do that? I think it's pretty normal
>>> for scratch-Branches being deleted and pushed anew, and for large
>>> branches like cedet-merge or cedet-tests this needlessly creates
>>> hundreds of mails each time.
>> I agree that those 300 emails dumps are more annoying than useful.
>> But FWIW, I see two problems with your suggestion:
>> 1- I don't know how to do it.
> We'd have to adapt the post-receive hook, or whatever else is triggering
> the mails.

Yes, that much I know, but I also know that we don't have direct access
to it.  So it depends on Savannah admins, and if I was them I wouldn't
just accept random patches, since AFAIK they use an existing mail-hook
and so they probably want to stay close to the upstream version to be
able to incorporate future changes.

So, we should figure out exactly what choices we have available.

>> 2- I do find it very useful to see commits on scratch branches when
>> those branches are used for gradual development (e.g. the byte-switch
>> branch).
> How about simply not naming those branches with the 'scratch' prefix,
> then?  I was under the impression that 'scratch' means "can be
> force-pushed/deleted any time, don't base your work onto it".

AFAICT the byte-switch does belong to scratch.  Being able/allowed to
force-push/delete doesn't mean that you have to do it all the time.


        Stefan



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Disable emacs-diffs mails for scratch branches
  2017-01-28  9:56 Disable emacs-diffs mails for scratch branches David Engster
  2017-01-28 15:35 ` Stefan Monnier
@ 2017-01-29  2:31 ` Lars Ingebrigtsen
  2017-01-29  9:59   ` David Engster
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2017-01-29  2:31 UTC (permalink / raw)
  To: David Engster; +Cc: emacs-devel

David Engster <deng@randomsample.de> writes:

> Subject says it all. Could we please do that? 

Can't you just kill-file the messages you're not interested in?  I mean
score down.  That's what I do with the scratch/ branches...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Disable emacs-diffs mails for scratch branches
  2017-01-29  2:31 ` Lars Ingebrigtsen
@ 2017-01-29  9:59   ` David Engster
  0 siblings, 0 replies; 6+ messages in thread
From: David Engster @ 2017-01-29  9:59 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

Lars Ingebrigtsen writes:
> David Engster <deng@randomsample.de> writes:
>
>> Subject says it all. Could we please do that? 
>
> Can't you just kill-file the messages you're not interested in?  I mean
> score down.  That's what I do with the scratch/ branches...

Yeah sure, I do that as well. It's just that I'm kinda old-school and
think that force-pushing an amended commit shouldn't trigger 300
mails...

But since this doesn't seem to be easily changed, I guess we just live
with it.

-David



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-01-29  9:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-28  9:56 Disable emacs-diffs mails for scratch branches David Engster
2017-01-28 15:35 ` Stefan Monnier
2017-01-28 21:00   ` David Engster
2017-01-29  2:09     ` Stefan Monnier
2017-01-29  2:31 ` Lars Ingebrigtsen
2017-01-29  9:59   ` David Engster

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.