[
https://issues.apache.org/jira/browse/TWILL-125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14379315#comment-14379315
]
ASF GitHub Bot commented on TWILL-125:
--------------------------------------
Github user chtyim commented on the pull request:
https://github.com/apache/incubator-twill/pull/28#issuecomment-85843423
Thanks for the fix. Just one comment about creation of tmp file.
> BundledJarRunner doesn't work for classes with no zero-args constructor
> -----------------------------------------------------------------------
>
> Key: TWILL-125
> URL: https://issues.apache.org/jira/browse/TWILL-125
> Project: Apache Twill
> Issue Type: Bug
> Components: ext
> Affects Versions: 0.5.0-incubating
> Reporter: Rob Morgan
>
> BundledJarRunner attempts to load the main class using Class#newInstance. The
> call to newInstance will then attempt to find and invoke the class's
> zero-args constructor. This will throw an InstantiationException if there is
> no zero-args constructor to be found.
> Loading the main class is unnecessary. Since main is always static it can be
> invoked with a null target object.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
