Make some CLI classes serializable

• Key: CLI-140
• Type: Improvement
• Status: Closed
• Resolution: Fixed
• Priority: Minor
• Assignee: Unassigned
• Reporter: Henning Schmiedehausen
• Component/s: CLI-1.x
• Affects Version/s: 1.0, 1.1
• Fix Version/s: 1.2

Description

A number of CLI classes are serializable and could be used e.g. with RMI if only they would implement Serializable

• Option.java
• OptionGroup.java
• CommandLine.java
• Options.java

Comments

[Henri Yandell]

Consider for 1.2, especially if a patch shows up

[Bjorn Townsend]

Implemented serializable in the classes mentioned above. Note that Henning's patch to CLI-141 also implements Serializable in the Option class. Since I don't know which of these patches might be applied first, I've created two:

If you have already applied the patch for CLI-141, apply CLI-140_yes_141.patch.

If you have not already applied the patch for CLI-141, apply CLI-140_no_141.patch.

[Brian Egge]

I took a look at the CLI-140_no_141.patch, and I don't see any reason why it shouldn't be applied. I ran a clirr check, and the changes are backwards compatible.

INFO: 4000: org.apache.commons.cli.CommandLine: Added java.io.Serializable to the set of implemented interfaces
INFO: 4000: org.apache.commons.cli.Option: Added java.io.Serializable to the set of implemented interfaces
INFO: 4000: org.apache.commons.cli.OptionGroup: Added java.io.Serializable to the set of implemented interfaces
INFO: 4000: org.apache.commons.cli.Options: Added java.io.Serializable to the set of implemented interfaces

[Henri Yandell]

Applied 'yes' patch: 596142.