FYI: patch applied and the site has been updated. Thanks Steve. http://jakarta.apache.org/commons/sandbox/net
================================================================= Jeffrey D. Brekke Quad/Graphics [EMAIL PROTECTED] http://www.qg.com > -----Original Message----- > From: Brekke, Jeff > Sent: Sunday, April 28, 2002 10:22 PM > To: '[EMAIL PROTECTED]' > Subject: RE: [PATCH] - New FTP File Listing Mechanism > > > > I'll just apply what I have an we can clean up after. I did > change the > format of the code in the patch to match more closely the coding style > posted on the site. No functional changes, just spacing, > author/version > fixes, and import statement cleansing. > > Is the idea to move forward and replace FTPClient with > FTPClient2 or add in > the fucntionality some other way? I suppose a nice way to > deprecate the > older listing methods and still have the new versions should > be the next > topic of discussion? Although we could just replace the > listing stuff for > the first release of this, I think maintaining the older > interface would be > the best for users. Then users of the orig netcomponents > code would really > just need to change package names. > > ================================================================= > Jeffrey D. Brekke Quad/Graphics > [EMAIL PROTECTED] http://www.qg.com > > > > -----Original Message----- > > From: Steve Cohen [mailto:[EMAIL PROTECTED]] > > Sent: Sunday, April 28, 2002 9:40 AM > > To: Brekke, Jeff > > Subject: Re: [PATCH] - New FTP File Listing Mechanism > > > > > > Say Jeff: > > I was looking back over my code and I see one thing that I > > meant to clean up > > and forgot about: > > > > FTPFileList.create() (FTPFileList.java, line 85) eats the > > IOException instead > > of throwing it, as it should (and as the analogous > > functionality in Daniel's > > code does). Worse, it System.out.println's the error. > > > > Would you rather I clean this now, or get it in as soon as > the commit > > happens? I'd rather not send a whole new patch that's 99% > > the same as the > > other, but I will leave up to you the best way to handle it. > > > > Steve > > > > On Sunday 28 April 2002 07:33 am, Brekke, Jeff wrote: > > > Thanks Steve. I've integrated your patch and and looking > at it now. > > > I've recently placed the projects generate documentation here: > > > > > > http://jakarta.apache.org/commons/sandbox/net > > > > > > It include javadocs, metrics report, checkstyle report, > > etc. that maven > > > generates. > > > > > > ================================================================= > > > Jeffrey D. Brekke Quad/Graphics > > > [EMAIL PROTECTED] http://www.qg.com > > > > > > > -----Original Message----- > > > > From: Steve Cohen [mailto:[EMAIL PROTECTED]] > > > > Sent: Saturday, April 27, 2002 1:16 PM > > > > To: [EMAIL PROTECTED] > > > > Subject: [PATCH] - New FTP File Listing Mechanism > > > > > > > > > > > > The attached patch implements an alternative file listing > > mechanism. > > > > While designed to replace and enhance the original mechanism, > > > > complete > > > > backward compatibility is preserved. > > > > > > > > This patch was originally submitted in a slightly different > > > > form before the > > > > NetComponents project was moved to the Jakarta Sandbox in > > > > April 2002. I am > > > > also attaching the original comments that were > submitted with that > > > > submission. They are all still valid. The only differences > > > > between the > > > > earlier patch and this one are these > > > > 1) my email address is different > > > > 2) the package names were changed to conform with the new > > > > commons location > > > > 3) licenses were changed to Apache > > > > 4) the File Parsers and their associated test classes were > > > > moved to a new > > > > subpackage: > > > > org.apache.commons.net.ftp.ftp2.parser > > > > for greater clarity. > > > > 5) project.properties has been changed so that new > > > > Maven-based build (a > > > > really nice system, by the way, which I hadn't known about) > > > > knows where to > > > > find the test classes. > > > > > > > > I would have liked to submit this patch in smaller pieces but > > > > it is really > > > > impossible to do so and still have a functioning package. > > > > Breaking the > > > > parsers into a subpackage perhaps makes clearer that the > > > > changes are really > > > > not massive. > > > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
