search

azynheira / distel

Automatically exported from code.google.com/p/distel
0 stars 0 forks source link

Distel does not work with epmd from R14B #24

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?

1. Start a node, named eg. "emacs"
2. C-c C-d n, set name to "emacs"
3. Try to ping it: C-c C-d g

What is the expected output? What do you see instead?

Expected to see "Successfully communicated with remote node emacs@seere", but 
only got "nodedown: emacs@seere". It fails with R14B, but works correctly with 
R13B04. One may even start only epmd from R13B04, and use R14B with distel. In 
this case, distel works to. 

As distel with R14B's epmd can not determine if the node is alive, none of the 
advanced distel features (process listing, completion, ...) work.

What version of the product are you using? On what operating system?

I checked this with self-compiled emacs from emacs-23 branch, distel 4.03, 
erlang R13B04 and R14B.

Please provide any additional information below.

The reason distel fails comes from erlang upstream commit 
http://github.com/erlang/otp/commit/6499bf2816c3834059ebdeff75d98f310cb305b4, 
which removed old PORT_REQ protoccol from epmd, but distel uses just this...

After fiddling around with empd.el, I came up with a patch to empd.el to use 
EPMD_PORT2_REQ/EPMD_PORT2_RESP instead. A patch is attached.

Original issue reported on code.google.com by ralf.doe...@gmail.com on 12 Oct 2010 at 8:24

Attachments:

GoogleCodeExporter commented 8 years ago 
Hi,

We also have just spotted this issue.  It's down to a change in the EPMD 
protocol - the last commit to epmd.h removes the protocol that distel is using, 
check out the commit comment for this file: 
http://github.com/erlang/otp/commit/6499bf2816c3834059ebdeff75d98f310cb305b4

Hope that helps!

Cheers,

Steve

Original comment by srstr...@gmail.com on 14 Oct 2010 at 8:10

GoogleCodeExporter commented 8 years ago 
Ah - just read the rest of the report and see that the original reported also 
spotted this and provided a patch to distel - nice job, thanks!

Original comment by srstr...@gmail.com on 14 Oct 2010 at 8:12

GoogleCodeExporter commented 8 years ago 
fixed in commit a1777fff on github

Original comment by mats.cro...@gmail.com on 28 Oct 2010 at 10:57