New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #29438 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

nginx @1.0.0 add ipv6 variant

Reported by: rene@… Owned by: boeyms@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: Cc:
Port: nginx

Description

Add support for ipv6 as a variant

Attachments

Portfile-nginx.diff (463 bytes) - added by rene@… 4 years ago.
Portfile-nginx-v2.diff (711 bytes) - added by rene@… 4 years ago.
Updated with default_variants added for ipv6

Change History

Changed 4 years ago by rene@…

comment:1 Changed 4 years ago by rene@…

Sorry forgot to set the Port field to nginx

comment:2 Changed 4 years ago by cal@…

  • Owner changed from macports-tickets@… to boeyms@…
  • Port set to nginx

comment:3 follow-up: ↓ 4 Changed 4 years ago by jmr@…

Is there a reason why IPv6 shouldn't be enabled all the time?

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 4 years ago by cal@…

Replying to jmr@…:

Is there a reason why IPv6 shouldn't be enabled all the time?

Well, nginx should have an IPv6 variant for the same reasons the Wireshark port has it. I personally think we should add the variant and add it to default_variants.

Changed 4 years ago by rene@…

Updated with default_variants added for ipv6

comment:5 in reply to: ↑ 4 Changed 4 years ago by rene@…

Replying to cal@…:

Replying to jmr@…:

Is there a reason why IPv6 shouldn't be enabled all the time?

Well, nginx should have an IPv6 variant for the same reasons the Wireshark port has it. I personally think we should add the variant and add it to default_variants.

I agree that it should be a default variant. I've updated the patch with default_variants added.

comment:6 Changed 4 years ago by cal@…

  • Status changed from new to closed
  • Resolution set to fixed

Commited in r83269.

Note: See TracTickets for help on using tickets.