Opened 14 years ago

Closed 13 years ago

#23423 closed enhancement (fixed)

imap-uw needs some cleanup

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: nerdling (Jeremy Lavergne)
Priority: Normal Milestone:
Component: ports Version: 1.8.2
Keywords: Cc:
Port: imap-uw

Description

I have a lot of problems with the imap-uw port.

  • The comment in the portfile says the default is to enable SSL support and plaintext authentication. Why then is there an ssl_plain variant whose purpose is to enable SSL support and plaintext authentication?
  • The ssl_pam variant does everything the ssl_plain variant does, and more. These variants need to therefore either conflict or the ssl_pam variant needs to depend on the ssl_plain variant and remove the duplicate directives from itself.
  • The openssl dependency in the ssl_pam variant should be a port:-style dependency.
  • The default build.target is oxp; there's no need for the ssl_pam variant to set the build.target to oxp again.
  • The post-destroot message in the ssl_pam variant references Mac OS X 10.3, an unsupported OS.
  • The nopwd variant deletes the nonexistent build.arg "SSLTYPE=unix". Perhaps this variant was meant to conflict with -- or require? -- the ssl_plain and/or ssl_pam variants?
  • To what mailbox format will new mailboxes default if neither the mbx nor the mix variant is selected? I would have expected all possible mailbox formats to be expressed as conflicting variants with one of them selected via default_variants.

Change History (1)

comment:1 in reply to:  description Changed 13 years ago by nerdling (Jeremy Lavergne)

Resolution: fixed
Status: newclosed

Replying to ryandesign@…:

  • To what mailbox format will new mailboxes default if neither the mbx nor the mix variant is selected? There aren't

"unixproto" is the default for the mailbox format in the Makefile, picking +mix or +mbx will swap it out.

r73666.

Note: See TracTickets for help on using tickets.