[net][PATCH] "MVS is the operating system of this server. FTP Server is running on z/OS."

• Key: NET-71
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Major
• Assignee: Unassigned
• Reporter: William Noto
• Affects Version/s: Nightly Builds

Environment

Operating System: All
Platform: All

Description

Jeff Nadler created the original fix to this issue and I wrote the test case. I
will post the patch directly.

Comments

[William Noto]

Created an attachment (id=14649)
This fixes the MVS FTP issue.

[William Noto]

The attached patch fixes this.

[Steve Cohen]

This bug cannot be marked as fixed - your code has not yet been accepted into
the commons-net project yet. However, I see no problem in its being accepted,
and when it is, this bug can close.

[Steve Cohen]

Now it can be called "fixed". I have committed your new parser to SVN and it is
part of the project. Thank you both for your excellent contributions. This is
how Open Source is supposed to work.

I made a couple of minor changes to your code so that it might fit better into
the hierarchy. I made your parser class a subclass of
ConfigurableFTPFileEntryParserImpl (which may not have been around at the time
you originally wrote this) and I made your test class a subclass of
FTPParseTestFramework which is more appropriate than the original
CompositeFTPParseTestFramework (the latter is for parsers that are "polymorphic"

• for example, NT's parser which can be configured either in the native NT
  format or the UNIX format).

You guys might want to submit more patches if and when you get a chance, if you
learn any more about how timestamps and directories are handled.

Again, thanks.

[Steve Cohen]

• • • COM-1438 has been marked as a duplicate of this bug. ***