all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
       [not found] ` <20170427213827.B6BDC22040@vcs0.savannah.gnu.org>
@ 2017-04-28  7:53   ` Katsumi Yamaoka
  2017-04-28  8:10     ` Katsumi Yamaoka
  0 siblings, 1 reply; 14+ messages in thread
From: Katsumi Yamaoka @ 2017-04-28  7:53 UTC (permalink / raw)
  To: emacs-devel; +Cc: Ted Zlatanov

On Thu, 27 Apr 2017 17:38:27 -0400, Teodor Zlatanov wrote:
> diff --git a/lisp/auth-source.el b/lisp/auth-source.el
> index 7402ab2..5ad4277 100644
> --- a/lisp/auth-source.el
> +++ b/lisp/auth-source.el
[...]
> +        (return)))

This is a cl runtime function.  Could you replace it with something?
Thanks.



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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28  7:53   ` master bd60ee2 1/5: auth-source: factor out parsers and add tests Katsumi Yamaoka
@ 2017-04-28  8:10     ` Katsumi Yamaoka
  2017-04-28 12:43       ` Ted Zlatanov
  2017-04-28 16:36       ` Glenn Morris
  0 siblings, 2 replies; 14+ messages in thread
From: Katsumi Yamaoka @ 2017-04-28  8:10 UTC (permalink / raw)
  To: emacs-devel; +Cc: Ted Zlatanov

On Fri, 28 Apr 2017 16:53:35 +0900, Katsumi Yamaoka wrote:
> On Thu, 27 Apr 2017 17:38:27 -0400, Teodor Zlatanov wrote:
>> diff --git a/lisp/auth-source.el b/lisp/auth-source.el
>> index 7402ab2..5ad4277 100644
>> --- a/lisp/auth-source.el
>> +++ b/lisp/auth-source.el
> [...]
>> +        (return)))

> This is a cl runtime function.  Could you replace it with something?
> Thanks.

Sorry for not explaining enough.   That is an alias to the macro
`cl-return' but not expanded in the elc file.



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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28  8:10     ` Katsumi Yamaoka
@ 2017-04-28 12:43       ` Ted Zlatanov
  2017-04-28 12:52         ` Stefan Monnier
  2017-04-28 16:36       ` Glenn Morris
  1 sibling, 1 reply; 14+ messages in thread
From: Ted Zlatanov @ 2017-04-28 12:43 UTC (permalink / raw)
  To: emacs-devel

On Fri, 28 Apr 2017 17:10:43 +0900 Katsumi Yamaoka <yamaoka@jpl.org> wrote: 

KY> On Fri, 28 Apr 2017 16:53:35 +0900, Katsumi Yamaoka wrote:
>> On Thu, 27 Apr 2017 17:38:27 -0400, Teodor Zlatanov wrote:
>>> diff --git a/lisp/auth-source.el b/lisp/auth-source.el
>>> index 7402ab2..5ad4277 100644
>>> --- a/lisp/auth-source.el
>>> +++ b/lisp/auth-source.el
>> [...]
>>> +        (return)))

>> This is a cl runtime function.  Could you replace it with something?

KY> Sorry for not explaining enough.   That is an alias to the macro
KY> `cl-return' but not expanded in the elc file.

Sorry, I don't know what's the proper fix. What breaks, and how should I
have caught this problem before it was comitted? Should I just replace
it with `cl-return' or is there more to do?

Thanks
Ted




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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28 12:43       ` Ted Zlatanov
@ 2017-04-28 12:52         ` Stefan Monnier
  0 siblings, 0 replies; 14+ messages in thread
From: Stefan Monnier @ 2017-04-28 12:52 UTC (permalink / raw)
  To: emacs-devel

> Sorry, I don't know what's the proper fix. What breaks, and how should I
> have caught this problem before it was comitted?

"make" says:

      ELC      auth-source.elc
    
    In toplevel form:
    auth-source.el:782:1:Warning: Unused lexical variable ‘sym’
    auth-source.el:818:1:Warning: Unused lexical variable ‘sym’
    auth-source.el:1196:1:Warning: Unused lexical variable ‘ret’
    
    In end of data:
    auth-source.el:2080:1:Warning: the function ‘return’ is not known to be
        defined.

and while the 3 "unused ..." lines are due to my setting
byte-compile-force-lexical-warnings, the last warning should be present
for you as well.


        Stefan




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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28  8:10     ` Katsumi Yamaoka
  2017-04-28 12:43       ` Ted Zlatanov
@ 2017-04-28 16:36       ` Glenn Morris
  2017-04-28 17:12         ` Ted Zlatanov
                           ` (2 more replies)
  1 sibling, 3 replies; 14+ messages in thread
From: Glenn Morris @ 2017-04-28 16:36 UTC (permalink / raw)
  To: emacs-devel

Ted Zlatanov wrote:

> What breaks, and how should I have caught this problem before it was
> comitted?

You could have run "make check", to see a bunch of test failures due to
"return" being void.

I can't resist pointing out that this was flagged by
http://hydra.nixos.org/build/52055147

(BTW, your Mail-Followup-To header is broken: it only points to the list,
so dropping anyone you replied to who isn't subscribed.)



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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28 16:36       ` Glenn Morris
@ 2017-04-28 17:12         ` Ted Zlatanov
  2017-04-28 18:32           ` Stefan Monnier
  2017-04-28 19:28         ` Mail-Followup-To and Gmane and emacs-devel (was: master bd60ee2 1/5: auth-source: factor out parsers and add tests) Ted Zlatanov
  2017-04-28 22:53         ` master bd60ee2 1/5: auth-source: factor out parsers and add tests Stefan Monnier
  2 siblings, 1 reply; 14+ messages in thread
From: Ted Zlatanov @ 2017-04-28 17:12 UTC (permalink / raw)
  To: emacs-devel

On Fri, 28 Apr 2017 12:36:40 -0400 Glenn Morris <rgm@gnu.org> wrote: 

GM> Ted Zlatanov wrote:
>> What breaks, and how should I have caught this problem before it was
>> comitted?

GM> You could have run "make check", to see a bunch of test failures due to
GM> "return" being void.

GM> I can't resist pointing out that this was flagged by
GM> http://hydra.nixos.org/build/52055147

Huh. Thanks for the pointer. I'm glad you didn't resist.

But I didn't see that failure on my system:

SUMMARY OF TEST RESULTS
-----------------------
Files examined: 154
Ran 2250 tests, 2233 results as expected, 17 skipped

So my "make check" doesn't flag that `return' as a void-function, and
the warning during compilation that Stefan pointed out is easy to miss.
Hydra may be doing something differently to catch this?

Stefan fixed it this morning by using `cl-return' and `cl-dolist'
instead, which is great, but I'd like to make sure I don't break the
build again. What am I missing in my test process?

Thanks
Ted




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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28 17:12         ` Ted Zlatanov
@ 2017-04-28 18:32           ` Stefan Monnier
  2017-04-28 19:08             ` Ted Zlatanov
  0 siblings, 1 reply; 14+ messages in thread
From: Stefan Monnier @ 2017-04-28 18:32 UTC (permalink / raw)
  To: emacs-devel

> Stefan fixed it this morning by using `cl-return' and `cl-dolist'
> instead, which is great, but I'd like to make sure I don't break the
> build again. What am I missing in my test process?

My guess is that you byte-compiled the file from within your running
Emacs (which had `cl` loaded, so it was byte-compiled correctly) rather
than from `make`.


        Stefan




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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28 18:32           ` Stefan Monnier
@ 2017-04-28 19:08             ` Ted Zlatanov
  0 siblings, 0 replies; 14+ messages in thread
From: Ted Zlatanov @ 2017-04-28 19:08 UTC (permalink / raw)
  To: emacs-devel

On Fri, 28 Apr 2017 14:32:29 -0400 Stefan Monnier <monnier@iro.umontreal.ca> wrote: 

>> Stefan fixed it this morning by using `cl-return' and `cl-dolist'
>> instead, which is great, but I'd like to make sure I don't break the
>> build again. What am I missing in my test process?

SM> My guess is that you byte-compiled the file from within your running
SM> Emacs (which had `cl` loaded, so it was byte-compiled correctly) rather
SM> than from `make`.

OK, I'll make sure to use a clean build next time. Thanks for the guidance.

Ted




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

* Mail-Followup-To and Gmane and emacs-devel (was: master bd60ee2 1/5: auth-source: factor out parsers and add tests)
  2017-04-28 16:36       ` Glenn Morris
  2017-04-28 17:12         ` Ted Zlatanov
@ 2017-04-28 19:28         ` Ted Zlatanov
  2017-05-08 21:03           ` Mail-Followup-To and Gmane and emacs-devel Lars Ingebrigtsen
  2017-04-28 22:53         ` master bd60ee2 1/5: auth-source: factor out parsers and add tests Stefan Monnier
  2 siblings, 1 reply; 14+ messages in thread
From: Ted Zlatanov @ 2017-04-28 19:28 UTC (permalink / raw)
  To: emacs-devel

On Fri, 28 Apr 2017 12:36:40 -0400 Glenn Morris <rgm@gnu.org> wrote: 

GM> (BTW, your Mail-Followup-To header is broken: it only points to the list,
GM> so dropping anyone you replied to who isn't subscribed.)

I'm following up to the list since there are probably others using
gmane.emacs.devel instead of subscribing to the mailing list directly.
So whatever solution comes out of this discussion may help them too.

It's really annoying to get e-mail for discussions I see on
gmane.emacs.devel anyhow. This is the only way I've found to minimize
that e-mail.

So for a while now I've had it set this way. Actually, since I post
through Gmane, my headers are (the Gmane header is translated on the
Gmane side, Lars implemented that):

Mail-Copies-To: never
Gmane-Reply-To-List: yes

I'm not sure if there's a better approach, so I'll change my headers if
anyone has suggestions. But I really don't want the drive-by CC's back
in my mailbox.

Thanks
Ted




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

* Re: master bd60ee2 1/5: auth-source: factor out parsers and add tests
  2017-04-28 16:36       ` Glenn Morris
  2017-04-28 17:12         ` Ted Zlatanov
  2017-04-28 19:28         ` Mail-Followup-To and Gmane and emacs-devel (was: master bd60ee2 1/5: auth-source: factor out parsers and add tests) Ted Zlatanov
@ 2017-04-28 22:53         ` Stefan Monnier
  2 siblings, 0 replies; 14+ messages in thread
From: Stefan Monnier @ 2017-04-28 22:53 UTC (permalink / raw)
  To: emacs-devel

> so dropping anyone you replied to who isn't subscribed.)

Such as yours truly ;-)


        Stefan




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

* Re: Mail-Followup-To and Gmane and emacs-devel
  2017-04-28 19:28         ` Mail-Followup-To and Gmane and emacs-devel (was: master bd60ee2 1/5: auth-source: factor out parsers and add tests) Ted Zlatanov
@ 2017-05-08 21:03           ` Lars Ingebrigtsen
  2017-05-15 19:58             ` Ted Zlatanov
  0 siblings, 1 reply; 14+ messages in thread
From: Lars Ingebrigtsen @ 2017-05-08 21:03 UTC (permalink / raw)
  To: emacs-devel

Ted Zlatanov <tzz@lifelogs.com> writes:

> So for a while now I've had it set this way. Actually, since I post
> through Gmane, my headers are (the Gmane header is translated on the
> Gmane side, Lars implemented that):
>
> Mail-Copies-To: never

This makes Message behave (i.e., don't include you in the Cc list)...

> Gmane-Reply-To-List: yes

This makes Gmane insert a Reply-To header in the mail, which is probably
what's making the problem here (i.e., dropping everybody from the Cc
list, even those who'd rather be on it).

> I'm not sure if there's a better approach, so I'll change my headers if
> anyone has suggestions. But I really don't want the drive-by CC's back
> in my mailbox.

I think the Gmane-Reply-To-List is the only realistic solution here for
this, but you could also just split to /dev/null all mail that has both
you and emacs-devel in the Cc headers.  

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



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

* Re: Mail-Followup-To and Gmane and emacs-devel
  2017-05-08 21:03           ` Mail-Followup-To and Gmane and emacs-devel Lars Ingebrigtsen
@ 2017-05-15 19:58             ` Ted Zlatanov
  2017-05-15 20:07               ` Lars Ingebrigtsen
  0 siblings, 1 reply; 14+ messages in thread
From: Ted Zlatanov @ 2017-05-15 19:58 UTC (permalink / raw)
  To: emacs-devel

On Mon, 08 May 2017 23:03:01 +0200 Lars Ingebrigtsen <larsi@gnus.org> wrote: 

LI> I think the Gmane-Reply-To-List is the only realistic solution here for
LI> this, but you could also just split to /dev/null all mail that has both
LI> you and emacs-devel in the Cc headers.  

Karl Fogel suggested letting Gnus deal with it, by suppressing duplicate
messages. Normally that only works in the same backend, but I could hook
into the gnus-registry and use it across backends. I don't know enough
of those internals, but the gnus-registry seems perfect for tracking by
message ID. Do you think that could work?

Ted




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

* Re: Mail-Followup-To and Gmane and emacs-devel
  2017-05-15 19:58             ` Ted Zlatanov
@ 2017-05-15 20:07               ` Lars Ingebrigtsen
  2017-05-15 20:20                 ` Ted Zlatanov
  0 siblings, 1 reply; 14+ messages in thread
From: Lars Ingebrigtsen @ 2017-05-15 20:07 UTC (permalink / raw)
  To: emacs-devel

Ted Zlatanov <tzz@lifelogs.com> writes:

> Karl Fogel suggested letting Gnus deal with it, by suppressing duplicate
> messages. Normally that only works in the same backend, but I could hook
> into the gnus-registry and use it across backends. I don't know enough
> of those internals, but the gnus-registry seems perfect for tracking by
> message ID. Do you think that could work?

Setting `gnus-suppress-duplicates' will mark articles with Message-IDs
you've seen already as read (by maintaining a cache of Message-IDs).  It
won't hide them completely, though, but that could be added...

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



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

* Re: Mail-Followup-To and Gmane and emacs-devel
  2017-05-15 20:07               ` Lars Ingebrigtsen
@ 2017-05-15 20:20                 ` Ted Zlatanov
  0 siblings, 0 replies; 14+ messages in thread
From: Ted Zlatanov @ 2017-05-15 20:20 UTC (permalink / raw)
  To: emacs-devel

On Mon, 15 May 2017 22:07:17 +0200 Lars Ingebrigtsen <larsi@gnus.org> wrote: 

LI> Ted Zlatanov <tzz@lifelogs.com> writes:
>> Karl Fogel suggested letting Gnus deal with it, by suppressing duplicate
>> messages. Normally that only works in the same backend, but I could hook
>> into the gnus-registry and use it across backends. I don't know enough
>> of those internals, but the gnus-registry seems perfect for tracking by
>> message ID. Do you think that could work?

LI> Setting `gnus-suppress-duplicates' will mark articles with Message-IDs
LI> you've seen already as read (by maintaining a cache of Message-IDs).  It
LI> won't hide them completely, though, but that could be added...

I can live with that, if it works across backends. I just looked and it
seems so; I didn't realize gnus-dup.el kept a global list. I read
gmane.emacs.devel most often, so I will have seen those message IDs.
I've now disabled `Gmane-Reply-To-List: yes` and let's see how it goes.

Thanks
Ted




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

end of thread, other threads:[~2017-05-15 20:20 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20170427213824.12196.63777@vcs0.savannah.gnu.org>
     [not found] ` <20170427213827.B6BDC22040@vcs0.savannah.gnu.org>
2017-04-28  7:53   ` master bd60ee2 1/5: auth-source: factor out parsers and add tests Katsumi Yamaoka
2017-04-28  8:10     ` Katsumi Yamaoka
2017-04-28 12:43       ` Ted Zlatanov
2017-04-28 12:52         ` Stefan Monnier
2017-04-28 16:36       ` Glenn Morris
2017-04-28 17:12         ` Ted Zlatanov
2017-04-28 18:32           ` Stefan Monnier
2017-04-28 19:08             ` Ted Zlatanov
2017-04-28 19:28         ` Mail-Followup-To and Gmane and emacs-devel (was: master bd60ee2 1/5: auth-source: factor out parsers and add tests) Ted Zlatanov
2017-05-08 21:03           ` Mail-Followup-To and Gmane and emacs-devel Lars Ingebrigtsen
2017-05-15 19:58             ` Ted Zlatanov
2017-05-15 20:07               ` Lars Ingebrigtsen
2017-05-15 20:20                 ` Ted Zlatanov
2017-04-28 22:53         ` master bd60ee2 1/5: auth-source: factor out parsers and add tests Stefan Monnier

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.