#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 (Ryan Carsten Schmidt) |
Port: | pureftpd |
Description
Here's a diff of the pureftpd Portfile with support for MySQL and an update to version 1.0.22
Attachments (1)
Change History (4)
Changed 15 years ago by lwilbrandt@…
Attachment: | Portfile-pureftpd.diff added |
---|
comment:1 follow-up: 3 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | scott@… added |
---|---|
Resolution: | → invalid |
Status: | new → closed |
comment:2 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|---|
Component: | contrib → ports |
comment:3 Changed 15 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.
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.