Re: [go-nuts] Solaris gccgo http.Get error?

2013-09-17 Thread Ian Lance Taylor
On Tue, Sep 17, 2013 at 12:28 PM,  ernie.hers...@10gen.com wrote:
 If you don't mind explaining, can you tell me why you didn't apply the
 change to the 4.7 branch?

I'm not maintaining Go on the 4.7 branch.  I don't object to somebody
else doing it, I'm just not doing it myself.  My time is limited and I
have to draw the line somewhere.

Ian

 On Friday, August 9, 2013 4:53:30 PM UTC-4, Ian Lance Taylor wrote:

 On Thu, Aug 8, 2013 at 11:22 PM, Jakob Borg ja...@nym.se wrote:
 
  But, adding a
 
   hints.ai_socktype = SOCK_STREAM;
 
  gives me
 
  jb@zlogin2:~ $ ./test
  canonical name: www.google.com
  26 2 6
  26 2 6
  26 2 6
  26 2 6
  26 2 6
  26 2 6
 
  It seems we might need a tweak to support Solaris... :/

 Looks like it.  I committed a patch to the master repository.  This
 patch copies it over to gccgo.  Bootstrapped and ran Go testsuite on
 x86_64-unknown-linux-gnu.  Committed to mainline and 4.8 branch.

 Note that I have not made the change on the 4.7 branch which is what
 you are using.  The same patch should work for the 4.7 sources,
 though, if you want to copy it over.

 Ian

 --
 You received this message because you are subscribed to the Google Groups
 golang-nuts group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to golang-nuts+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.


Re: [go-nuts] Solaris gccgo http.Get error?

2013-08-09 Thread Ian Lance Taylor
On Thu, Aug 8, 2013 at 11:22 PM, Jakob Borg ja...@nym.se wrote:

 But, adding a

  hints.ai_socktype = SOCK_STREAM;

 gives me

 jb@zlogin2:~ $ ./test
 canonical name: www.google.com
 26 2 6
 26 2 6
 26 2 6
 26 2 6
 26 2 6
 26 2 6

 It seems we might need a tweak to support Solaris... :/

Looks like it.  I committed a patch to the master repository.  This
patch copies it over to gccgo.  Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu.  Committed to mainline and 4.8 branch.

Note that I have not made the change on the 4.7 branch which is what
you are using.  The same patch should work for the 4.7 sources,
though, if you want to copy it over.

Ian


foo.patch
Description: Binary data