Ticket #20836 (closed enhancement: invalid)
Pureftpd with MySQL
| Reported by: | lwilbrandt@… | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.7.1 |
| Keywords: | enhancement, update | Cc: | scott@…, ryandesign@… |
| Port: | pureftpd |
Description
Here's a diff of the pureftpd Portfile with support for MySQL and an update to version 1.0.22
Attachments
Change History
comment:1 follow-up: ↓ 3 Changed 3 years ago by ryandesign@…
- Cc scott@… added
- Status changed from new to closed
- Resolution set to invalid
The pureftpd port already supports mysql5. Your patch is also not acceptable because it changes the formatting of virtually every line of the portfile. In the future please submit diffs that change only what you mean to change. If you want to suggest whitespace changes too, please do so in a separate diff that does nothing but whitespace changes.
The port has been renamed pure-ftpd and updated to 1.0.27; see #22920.
comment:2 Changed 3 years ago by ryandesign@…
- Cc ryandesign@… added
- Component changed from contrib to ports
comment:3 in reply to: ↑ 1 Changed 3 years ago by scott@…
Replying to ryandesign@…:
The pureftpd port already supports mysql5. Your patch is also not acceptable because it changes the formatting of virtually every line of the portfile. In the future please submit diffs that change only what you mean to change. If you want to suggest whitespace changes too, please do so in a separate diff that does nothing but whitespace changes.
The port has been renamed pure-ftpd and updated to 1.0.27; see #22920.
Yes, this portfile is entirely different, as I really did not start with the old portfile other than as a point of reference. After talking to the developer, and seeing how the old poerfile installed, what it installed, and where, I made a new portfile. I used bits of the old one, but started with a new file. This is why it was not sent in as a port update, but as a totally new port.
Most of my other ports are diffs, unless they are truly new ports, I just felt this one was enough of a change, it should be totally new. Even if I were to diff it, I changed so much, I strongly suspect the dff would have looked about as bad as a line ending change with tab stop change.
Sorry about that.

