all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Fran Litterio <flitterio@gmail.com>
To: 23658@debbugs.gnu.org
Subject: bug#23658: 25.1.50; make-network-connection fails to connect to IPv6 address on Windows
Date: Sun, 5 Jun 2016 15:04:56 +0000 (UTC)	[thread overview]
Message-ID: <loom.20160605T164523-19@post.gmane.org> (raw)
In-Reply-To: <87d1o3yty2.fsf@gmail.com>

Ken Brown <kbrown <at> cornell.edu> writes:

> While we're waiting for an expert to respond to this, it's
> easy to test if my guess is correct.  Please apply
> the following patch, rebuild, and retry your tests.

Hi Ken,

Unfortunately, your patch doesn't apply cleanly.  My source near
those lines no longer looks like the context in your patch.
In particular, I no longer have a call to connect_network_socket
at line 3142.  I see this commit by Paul Eggert in my source:

commit 918ef71538fd43049496e4370eba63e3096a4933
Author: Paul Eggert <eggert@cs.ucla.edu>
Date:   Mon May 23 08:56:42 2016 -0700

  Don't use only last protocol from getaddrinfo
    
  Problem reported by Ken Brown in:
  http://lists.gnu.org/archive/html/emacs-devel/2016-05/msg00483.html
  * src/process.c (conv_addrinfo_to_lisp): New function.
  (connect_network_socket): Arg is now a list of addrinfos, not
  merely IP addresses.  All uses changed.  Use protocol from
  each addrinfo.
  (Fmake_network_process): Accumulate protocols into addrinfos
  rather than just using the last one found.
  (check_for_dns): Accumulate protocols here, too.
  * src/process.h (struct Lisp_Process): Remove ai_protocol;
  no longer needed.

I built Emacs from source containing that commit, but it didn't fix
the problem.  Let me know if there's any more I can do to help.  I
will leave my local repo in the state where the above commit is
the most recent commit to src/process.c, in case you need me to
try another patch.
--
Fran Litterio






  parent reply	other threads:[~2016-06-05 15:04 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-30 23:44 bug#23658: 25.1.50; make-network-connection fails to connect to IPv6 address on Windows Francis Litterio
2016-06-01 14:54 ` Ken Brown
2016-06-01 15:30   ` Fran
2016-06-01 16:29     ` Ken Brown
2016-06-01 19:57       ` Fran
2016-06-01 21:55         ` Ken Brown
2016-06-02 14:50           ` Ken Brown
2016-06-04 16:01             ` Fran Litterio
2016-06-05 15:04             ` Fran Litterio [this message]
2016-06-05 15:31               ` Ken Brown
2016-06-05 15:52                 ` Fran
2016-06-05 16:56             ` Fran
2016-06-05 17:20               ` Ken Brown
2016-06-05 18:00                 ` Ken Brown
2016-06-12 20:04                   ` Ken Brown
2016-06-12 23:24                     ` Ken Brown
2016-06-02  8:16 ` Live System User
2016-06-02 15:01   ` Ken Brown
2020-08-04  9:34 ` Lars Ingebrigtsen
2020-09-14 15:00   ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=loom.20160605T164523-19@post.gmane.org \
    --to=flitterio@gmail.com \
    --cc=23658@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.