We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 2959

Summary: Enable IPV6 feature in SDL_net 1.2.8.
Product: SDL_net Reporter: Nitz <nitin.j4>
Component: miscAssignee: Sam Lantinga <slouken>
Status: NEW --- QA Contact:
Severity: enhancement    
Priority: P1 CC: ewmailing, philipp.wiesemann, raincomplex
Version: 1.2.8   
Hardware: x86   
OS: Linux   
Attachments: IPV6 Patch
updated patch for sdl2_net-2.0.1

Description Nitz 2015-04-24 08:59:22 UTC
Created attachment 2132 [details]
IPV6 Patch

Hi,
I have implemented IPV6 support in SDL_net-1.2.8 version, 

which means if Server is dual stack then Server can communicate with IPV6 and IPV4 clients both at same time.

 

I have done best possible changes with optimization in SDL_net as all of the API signatures are same except  SDLNet_UDP_Open

 

Kindly review and release the changes as per your policy.

 

If having any doubts please ask.


Attaching the Patch with IPV6 changes


Build steps:

export CPPFLAGS=-DDUAL_IP

./configure

make all

make install
Comment 1 Nitz 2015-05-20 13:08:07 UTC
(In reply to Nitz from comment #0)
> Created attachment 2132 [details]
> IPV6 Patch
> 
> Hi,
> I have implemented IPV6 support in SDL_net-1.2.8 version, 
> 
> which means if Server is dual stack then Server can communicate with IPV6
> and IPV4 clients both at same time.
> 
>  
> 
> I have done best possible changes with optimization in SDL_net as all of the
> API signatures are same except  SDLNet_UDP_Open
> 
>  
> 
> Kindly review and release the changes as per your policy.
> 
>  
> 
> If having any doubts please ask.
> 
> 
> Attaching the Patch with IPV6 changes
> 
> 
> Build steps:
> 
> export CPPFLAGS=-DDUAL_IP
> 
> ./configure
> 
> make all
> 
> make install
Comment 2 Nitz 2015-05-20 13:10:35 UTC
Hi Mr.Sam,
Any updates regarding IPV6 enhancement in SDL_net?
Please feel free if having any doubts.

Regards,
Nitin
Comment 3 Nitz 2015-07-22 09:06:08 UTC
A gentle reminder!!!

Thanks...
Comment 4 Philipp Wiesemann 2015-07-22 19:23:20 UTC
Bug 283 also has patches for IPv6.
Comment 5 Nitz 2015-07-23 03:50:42 UTC
Yes. I have seen that but the patch which I suggested has minimal changes as compared to Bug 283 and important thing is API signatures are also same except  SDLNet_UDP_Open.
You can compare and take the decision as currently I am using the same patch successfully.
Comment 6 Nitz 2015-09-29 05:54:08 UTC
A gentle reminder!!!
Thanks.
Comment 7 raincomplex 2016-08-03 00:20:39 UTC
Created attachment 2536 [details]
updated patch for sdl2_net-2.0.1

I merged Nitz's patch with sdl2_net-2.0.1 and removed the #defines so the code isn't dependent on DUAL_IP being defined. It's not a very nice diff to look at, but perhaps it will save someone else some work.

I don't know much about IPv6 but I did test this against both IPv4 and IPv6 networks, and it seems to work fine.
Comment 8 Eric wing 2018-05-25 21:53:53 UTC
Just wondering what the status of IPv6 in SDL_net is. Apple's App Stores require IPv6 operability as of 2 years ago. They seem to have an automated way to test compliance so they can automatically reject IPv4 only apps. 

So it would be really useful to have IPv6 support now.