[
https://issues.apache.org/jira/browse/OPENNLP-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17881969#comment-17881969
]
ASF GitHub Bot commented on OPENNLP-1607:
-----------------------------------------
rzo1 commented on PR #652:
URL: https://github.com/apache/opennlp/pull/652#issuecomment-2352301857
There is already a unit test but the branch isn't hit because of the
surefire configuration.
@codyfearer Maybe just add an additional execution phase for surefire
without the `--add-opens`, so the branch actually gets triggered. This will
show, that the current impl is broken and your patch is a fix for it.
> SimpleClassPathModelFinder is not parsing classpath properly
> ------------------------------------------------------------
>
> Key: OPENNLP-1607
> URL: https://issues.apache.org/jira/browse/OPENNLP-1607
> Project: OpenNLP
> Issue Type: Bug
> Reporter: Cody Fearer
> Priority: Major
> Attachments: Before.png
>
>
> The getClassPathElements method in SimpleClassPathModelFinder is not parsing
> the classpath properly. The Matcher returns a list of delimiters instead of
> the paths.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
