michael-o commented on PR #1089: URL: https://github.com/apache/maven/pull/1089#issuecomment-1510944227
> @michael-o on JIRA you mention "Commons CLI must not be touched because the upgrade requires also code changes. See master.". What code changes you mean here? > > I found this [4476026](https://github.com/apache/maven/commit/4476026c525a22900c6c5e4f0de6f7362cb201f2) but here parser is changed. Is it necessary? GnuParser is already deprecated (was in 1.3), but imho does not require code change, or I miss somethin? The migration introduces a slight incompat problem. That is why we did not touch it in 3.x. The previous impl was ugly, now I'd say that it is correct. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
