Date: Wed, 23 Oct 2002 09:56:27 -0700 (PDT) From: John Polstra <jdp@polstra.com> To: stable@freebsd.org Subject: Re: Where to send pam_radius feature request Message-ID: <200210231656.g9NGuRb7038435@vashon.polstra.com> In-Reply-To: <3DB3C686.5000306@ntmk.ru> References: <3DB3C686.5000306@ntmk.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
In article <3DB3C686.5000306@ntmk.ru>, Boris Kovalenko <boris@ntmk.ru> wrote: > I have feature request for pam_radius module and have done dirty > patch for it. Please don't take this personally -- I didn't look at your patch and I don't mean to single you out. For all I know, your patch wasn't "dirty" at all. But I want to use this opportunity to point out that in general, sending a "dirty patch" to a committer is not a productive approach. It is easy to create a quick and dirty patch. What takes a lot of time is to turn it into a well-tested, clean, directly committable patch. If you send a committer a quick & dirty patch, it just creates a ton of work for the committer. I personally tend to let that kind of patch rot in my inbox until it falls off the bottom. I don't even feel guilty about it any more. If you send a committer a quick and dirty patch, you are taking all of the fun part for yourself and handing the drudge work to the committer. Sorry, but you have to pay me a whole lot of money to accept that arrangement. If you want me to do something for free, it had better be either fun or very easy. :-) If you really want something to get committed, do _all_ of the work to make it into a top-quality piece of work. Fix the style bugs. Update the documentation. Test it thoroughly. Explain how it works and why it works. Make the committer's job as easy as possible, if you want results. Just a friendly suggestion. John -- John Polstra John D. Polstra & Co., Inc. Seattle, Washington USA "Disappointment is a good sign of basic intelligence." -- Chögyam Trungpa To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-stable" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200210231656.g9NGuRb7038435>