Opened 13 years ago
Closed 13 years ago
#29438 closed enhancement (fixed)
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 (2)
Change History (8)
Changed 13 years ago by rene@…
| Attachment: | Portfile-nginx.diff added |
|---|
comment:1 Changed 13 years ago by rene@…
comment:2 Changed 13 years ago by neverpanic (Clemens Lang)
| Owner: | changed from macports-tickets@… to boeyms@… |
|---|---|
| Port: | nginx added |
comment:3 follow-up: 4 Changed 13 years ago by jmroot (Joshua Root)
Is there a reason why IPv6 shouldn't be enabled all the time?
comment:4 follow-up: 5 Changed 13 years ago by neverpanic (Clemens Lang)
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 13 years ago by rene@…
| Attachment: | Portfile-nginx-v2.diff added |
|---|
Updated with default_variants added for ipv6
comment:5 Changed 13 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 13 years ago by neverpanic (Clemens Lang)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Commited in r83269.
Sorry forgot to set the Port field to nginx