all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
@ 2018-09-12 15:44 Martin Kletzander
  2019-06-23 21:53 ` Lars Ingebrigtsen
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Martin Kletzander @ 2018-09-12 15:44 UTC (permalink / raw)
  To: 32723

When auto-joining channel with the same name on
multiple channels the auto-join will go over the
list of channels and for each one it checks if
there is a buffer and alive server process.
However it does not check if the server process
matches the server being currently processed for
auto-joining.

This happens since commit "Don't join erc channels
doubly"

Fix it by checking if the currently processed
server matches the erc process name of the already
joined channel.

* lisp/erc/erc-join.el (erc-autojoin-channels):
Join channels with the same name on different
servers.

Copyright-paperwork-exempt: yes
---
 lisp/erc/erc-join.el | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lisp/erc/erc-join.el b/lisp/erc/erc-join.el
index a6bf6518ea86..567a4cc27e19 100644
--- a/lisp/erc/erc-join.el
+++ b/lisp/erc/erc-join.el
@@ -161,6 +161,9 @@ This function is run from `erc-nickserv-identified-hook'."
 	      ;; Only auto-join the channels that we aren't already in
 	      ;; using a different nick.
 	      (when (or (not buffer)
+			;; If the same channel is joined on another
+			;; server the best-effort is to just join
+			(not (string-match (car l) (process-name erc-server-process)))
 			(not (with-current-buffer buffer
 			       (erc-server-process-alive))))
 		(erc-server-join-channel server chan))))))))
-- 
2.18.0






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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2018-09-12 15:44 bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers Martin Kletzander
@ 2019-06-23 21:53 ` Lars Ingebrigtsen
  2020-05-10 19:45 ` Kevin Brubeck Unhammer
       [not found] ` <87ftc7h9gp.fsf@fsfe.org>
  2 siblings, 0 replies; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-06-23 21:53 UTC (permalink / raw)
  To: Martin Kletzander; +Cc: 32723

Martin Kletzander <nert.pinx@gmail.com> writes:

> When auto-joining channel with the same name on
> multiple channels the auto-join will go over the
> list of channels and for each one it checks if
> there is a buffer and alive server process.
> However it does not check if the server process
> matches the server being currently processed for
> auto-joining.
>
> This happens since commit "Don't join erc channels
> doubly"
>
> Fix it by checking if the currently processed
> server matches the erc process name of the already
> joined channel.

Looks good to me; I've now applied it to the Emacs trunk.

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





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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2018-09-12 15:44 bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers Martin Kletzander
  2019-06-23 21:53 ` Lars Ingebrigtsen
@ 2020-05-10 19:45 ` Kevin Brubeck Unhammer
       [not found] ` <87ftc7h9gp.fsf@fsfe.org>
  2 siblings, 0 replies; 8+ messages in thread
From: Kevin Brubeck Unhammer @ 2020-05-10 19:45 UTC (permalink / raw)
  Cc: 32723

[-- Attachment #1: Type: text/plain, Size: 1653 bytes --]

Martin Kletzander <nert.pinx@gmail.com> čálii:

> When auto-joining channel with the same name on
> multiple channels the auto-join will go over the
> list of channels and for each one it checks if
> there is a buffer and alive server process.
> However it does not check if the server process
> matches the server being currently processed for
> auto-joining.
>
> This happens since commit "Don't join erc channels
> doubly"
>
> Fix it by checking if the currently processed
> server matches the erc process name of the already
> joined channel.

That's not always going to match though. The car's of
erc-autojoin-channels contain the values of erc-server-announced-name or
erc-session-server, not the process-name of
erc-server-process. E.g. when I join my weechat relay,
erc-session-server is "example.com" and erc-server-announced-name is
"weechat.relay.irc" while (process-name erc-server-process) is
"erc-example.com-9001<1>".

Also, the check is against the current erc-server-process (for the
server being joined). Isn't the intention of the patch to check against
the candidate buffer's server-process?

I would expect something like

	      (when (or (not buffer)
			;; If the same channel is joined on another
			;; server the best-effort is to just join
                        (not (with-current-buffer buffer
                               (or (string-match-p (car l) erc-session-server)
                                   (string-match-p (car l) erc-session-announced-name))))
			(not (with-current-buffer buffer
			       (erc-server-process-alive))))
		(erc-server-join-channel server chan))


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
       [not found] ` <87ftc7h9gp.fsf@fsfe.org>
@ 2020-05-11 15:06   ` Martin Kletzander
  2020-05-19 18:25     ` Martin Kletzander
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Kletzander @ 2020-05-11 15:06 UTC (permalink / raw)
  To: Kevin Brubeck Unhammer; +Cc: Lars Ingebrigtsen, 32723

On Sun, May 10, 2020 at 09:32:38PM +0200, Kevin Brubeck Unhammer wrote:
>Martin Kletzander <nert.pinx@gmail.com> čálii:
>
>> When auto-joining channel with the same name on
>> multiple channels the auto-join will go over the
>> list of channels and for each one it checks if
>> there is a buffer and alive server process.
>> However it does not check if the server process
>> matches the server being currently processed for
>> auto-joining.
>>
>> This happens since commit "Don't join erc channels
>> doubly"
>>
>> Fix it by checking if the currently processed
>> server matches the erc process name of the already
>> joined channel.
>
>That's not always going to match though. The car's of
>erc-autojoin-channels contain the values of erc-server-announced-name or
>erc-session-server, not the process-name of
>erc-server-process. E.g. when I join my weechat relay,
>erc-session-server is "example.com" and erc-server-announced-name is
>"weechat.relay.irc" while (process-name erc-server-process) is
>"erc-example.com-9001<1>".
>

Oh, I completely forgot what I was doing there, I just know that with this patch
it is fixed for me, even though it might be dependent on various quirks that
just happen on my client (be it joining servers with very different latency, not
using autojoin twice or something completely different).

>Also, the check is against the current erc-server-process (for the
>server being joined). Isn't the intention of the patch to check against
>the candidate buffer's server-process?
>

As I said, I do not remember what exactly was the reason for this particular way
of dealing with it.  I definitely made some mistakes.

>I would expect something like
>
>	      (when (or (not buffer)
>			;; If the same channel is joined on another
>			;; server the best-effort is to just join
>                        (not (with-current-buffer buffer
>                               (or (string-match-p (car l) erc-session-server)
>                                   (string-match-p (car l) erc-session-announced-name))))
>			(not (with-current-buffer buffer
>			       (erc-server-process-alive))))
>		(erc-server-join-channel server chan))
>

If that works, then I'm fine with it. I can test it later on and let you know if
it works for me.

Thanks,
Martin





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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2020-05-11 15:06   ` Martin Kletzander
@ 2020-05-19 18:25     ` Martin Kletzander
  2020-05-19 18:40       ` Kevin Brubeck Unhammer
  0 siblings, 1 reply; 8+ messages in thread
From: Martin Kletzander @ 2020-05-19 18:25 UTC (permalink / raw)
  To: Kevin Brubeck Unhammer; +Cc: Lars Ingebrigtsen, 32723

On Mon, May 11, 2020 at 05:06:54PM +0200, Martin Kletzander wrote:
>On Sun, May 10, 2020 at 09:32:38PM +0200, Kevin Brubeck Unhammer wrote:
>>Martin Kletzander <nert.pinx@gmail.com> čálii:
>>
>>> When auto-joining channel with the same name on
>>> multiple channels the auto-join will go over the
>>> list of channels and for each one it checks if
>>> there is a buffer and alive server process.
>>> However it does not check if the server process
>>> matches the server being currently processed for
>>> auto-joining.
>>>
>>> This happens since commit "Don't join erc channels
>>> doubly"
>>>
>>> Fix it by checking if the currently processed
>>> server matches the erc process name of the already
>>> joined channel.
>>
>>That's not always going to match though. The car's of
>>erc-autojoin-channels contain the values of erc-server-announced-name or
>>erc-session-server, not the process-name of
>>erc-server-process. E.g. when I join my weechat relay,
>>erc-session-server is "example.com" and erc-server-announced-name is
>>"weechat.relay.irc" while (process-name erc-server-process) is
>>"erc-example.com-9001<1>".
>>
>
>Oh, I completely forgot what I was doing there, I just know that with this patch
>it is fixed for me, even though it might be dependent on various quirks that
>just happen on my client (be it joining servers with very different latency, not
>using autojoin twice or something completely different).
>
>>Also, the check is against the current erc-server-process (for the
>>server being joined). Isn't the intention of the patch to check against
>>the candidate buffer's server-process?
>>

So I just tried emacs-27.0.91 and with your approach it does not join any
channel on any server.  With my patch (which I just learned is in, I found out
that it could not be applied locally) it joins all channels, but there is only
one of those two that have the same name.  However if I try joining the other
one manually it joins, but does not create a buffer.  It only gets created once
someone posts to the channel.  I actually started writing that it does not get
joined while a message appeared out of nowhere, I thought it was a private query
because there was no header, nothing.  Only after that I realised it has a name
of the channel, but on the second server, so I deleted that part of the e-mail!

I think there is something wrong with some other part of erc-join (or erc), so
for a while I'm sticking to 26.* with my patch on top of it, but sooner or later
I'll have to figure that out (or someone else, maybe).  Unfortunately I'm
swamped with more important things lately.

>
>As I said, I do not remember what exactly was the reason for this particular way
>of dealing with it.  I definitely made some mistakes.
>
>>I would expect something like
>>
>>	      (when (or (not buffer)
>>			;; If the same channel is joined on another
>>			;; server the best-effort is to just join
>>                        (not (with-current-buffer buffer
>>                               (or (string-match-p (car l) erc-session-server)
>>                                   (string-match-p (car l) erc-session-announced-name))))
>>			(not (with-current-buffer buffer
>>			       (erc-server-process-alive))))
>>		(erc-server-join-channel server chan))
>>
>
>If that works, then I'm fine with it. I can test it later on and let you know if
>it works for me.
>
>Thanks,
>Martin





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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2020-05-19 18:25     ` Martin Kletzander
@ 2020-05-19 18:40       ` Kevin Brubeck Unhammer
  2020-05-19 20:32         ` Corwin Brust
  0 siblings, 1 reply; 8+ messages in thread
From: Kevin Brubeck Unhammer @ 2020-05-19 18:40 UTC (permalink / raw)
  To: Martin Kletzander; +Cc: Lars Ingebrigtsen, 32723

[-- Attachment #1: Type: text/plain, Size: 602 bytes --]

Martin Kletzander <nert.pinx@gmail.com> čálii:

> Only after that I realised it has a name
> of the channel, but on the second server, so I deleted that part of the e-mail!
>
> I think there is something wrong with some other part of erc-join (or erc), so
> for a while I'm sticking to 26.*

I also noticed this with Emacs 27 after writing you. ERC shows things
like #emacs@BitlBee (should be @freenode). I haven't been able to find
out what causes it (doesn't happen every time I connect to two servers,
only when I don't have time to investigate). Will report back if I
figure it out.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2020-05-19 18:40       ` Kevin Brubeck Unhammer
@ 2020-05-19 20:32         ` Corwin Brust
  2020-06-09 14:52           ` Martin Kletzander
  0 siblings, 1 reply; 8+ messages in thread
From: Corwin Brust @ 2020-05-19 20:32 UTC (permalink / raw)
  To: Kevin Brubeck Unhammer; +Cc: Lars Ingebrigtsen, 32723, Martin Kletzander

FWIW, and please forgive the top-posted reply:

This seems related to this bug I opened fairly recently:

  https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40121#5

I'm not sure but I think this started happening for me after I pulled
in a new ERC from master to my ~/site-lisp, which was not too long
before I made the switch to Emacs 27 PR 1.  (I wanted Amin's patch to
make nicks highlight even when followed by a punctuation character.)

In any case this behavior has been consistent across all versions of
Emacs 27 I've used and started before I'd actually upgraded from Emacs
26.3, if I'm not mistaken.

Regards,
Corwin

On Tue, May 19, 2020 at 1:41 PM Kevin Brubeck Unhammer
<unhammer@fsfe.org> wrote:
>
> Martin Kletzander <nert.pinx@gmail.com> čálii:
>
> > Only after that I realised it has a name
> > of the channel, but on the second server, so I deleted that part of the e-mail!
> >
> > I think there is something wrong with some other part of erc-join (or erc), so
> > for a while I'm sticking to 26.*
>
> I also noticed this with Emacs 27 after writing you. ERC shows things
> like #emacs@BitlBee (should be @freenode). I haven't been able to find
> out what causes it (doesn't happen every time I connect to two servers,
> only when I don't have time to investigate). Will report back if I
> figure it out.



-- 
Corwin
corwin@bru.st





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

* bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers
  2020-05-19 20:32         ` Corwin Brust
@ 2020-06-09 14:52           ` Martin Kletzander
  0 siblings, 0 replies; 8+ messages in thread
From: Martin Kletzander @ 2020-06-09 14:52 UTC (permalink / raw)
  To: Corwin Brust; +Cc: Kevin Brubeck Unhammer, 32723, Lars Ingebrigtsen

On Tue, May 19, 2020 at 03:32:19PM -0500, Corwin Brust wrote:
>FWIW, and please forgive the top-posted reply:
>
>This seems related to this bug I opened fairly recently:
>
>  https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40121#5
>
>I'm not sure but I think this started happening for me after I pulled
>in a new ERC from master to my ~/site-lisp, which was not too long
>before I made the switch to Emacs 27 PR 1.  (I wanted Amin's patch to
>make nicks highlight even when followed by a punctuation character.)
>
>In any case this behavior has been consistent across all versions of
>Emacs 27 I've used and started before I'd actually upgraded from Emacs
>26.3, if I'm not mistaken.
>

Sorry for such a late reply, just wanted to share that I experience the same
thing as you described in the bug, it happens on 27.0.91, but *not* on 26.3.

>Regards,
>Corwin
>
>On Tue, May 19, 2020 at 1:41 PM Kevin Brubeck Unhammer
><unhammer@fsfe.org> wrote:
>>
>> Martin Kletzander <nert.pinx@gmail.com> čálii:
>>
>> > Only after that I realised it has a name
>> > of the channel, but on the second server, so I deleted that part of the e-mail!
>> >
>> > I think there is something wrong with some other part of erc-join (or erc), so
>> > for a while I'm sticking to 26.*
>>
>> I also noticed this with Emacs 27 after writing you. ERC shows things
>> like #emacs@BitlBee (should be @freenode). I haven't been able to find
>> out what causes it (doesn't happen every time I connect to two servers,
>> only when I don't have time to investigate). Will report back if I
>> figure it out.
>
>
>
>-- 
>Corwin
>corwin@bru.st





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

end of thread, other threads:[~2020-06-09 14:52 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-09-12 15:44 bug#32723: [PATCH] Fix erc-autojoin for same channel names on multiple servers Martin Kletzander
2019-06-23 21:53 ` Lars Ingebrigtsen
2020-05-10 19:45 ` Kevin Brubeck Unhammer
     [not found] ` <87ftc7h9gp.fsf@fsfe.org>
2020-05-11 15:06   ` Martin Kletzander
2020-05-19 18:25     ` Martin Kletzander
2020-05-19 18:40       ` Kevin Brubeck Unhammer
2020-05-19 20:32         ` Corwin Brust
2020-06-09 14:52           ` Martin Kletzander

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.