[jira] [Commented] (MNG-6203) Minor cleanup in MavenCli.java

[Karl Heinz Marbaise (JIRA)]

    [ 
https://issues.apache.org/jira/browse/MNG-6203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15952611#comment-15952611
 ] 

Karl Heinz Marbaise commented on MNG-6203:
------------------------------------------

I was just that fast cause I was at my computer at this moment.....furthermore 
if you take a look some lines down you will see that {{userToolchainsFile}} is 
used:
{code:java}
        File userToolchainsFile;

        if ( cliRequest.commandLine.hasOption( 
CLIManager.ALTERNATE_USER_TOOLCHAINS ) )
        {
            userToolchainsFile =
                new File( cliRequest.commandLine.getOptionValue( 
CLIManager.ALTERNATE_USER_TOOLCHAINS ) );
            userToolchainsFile = resolveFile( userToolchainsFile, 
cliRequest.workingDirectory );

            if ( !userToolchainsFile.isFile() )
            {
                throw new FileNotFoundException(
                    "The specified user toolchains file does not exist: " + 
userToolchainsFile );
            }
        }
        else
        {
            userToolchainsFile = DEFAULT_USER_TOOLCHAINS_FILE;
        }

        File globalToolchainsFile;

        if ( cliRequest.commandLine.hasOption( 
CLIManager.ALTERNATE_GLOBAL_TOOLCHAINS ) )
        {
            globalToolchainsFile =
                new File( cliRequest.commandLine.getOptionValue( 
CLIManager.ALTERNATE_GLOBAL_TOOLCHAINS ) );
            globalToolchainsFile = resolveFile( globalToolchainsFile, 
cliRequest.workingDirectory );

            if ( !globalToolchainsFile.isFile() )
            {
                throw new FileNotFoundException(
                    "The specified global toolchains file does not exist: " + 
globalToolchainsFile );
            }
        }
        else
        {
            globalToolchainsFile = DEFAULT_GLOBAL_TOOLCHAINS_FILE;
        }

        cliRequest.request.setGlobalToolchainsFile( globalToolchainsFile );
        cliRequest.request.setUserToolchainsFile( userToolchainsFile );
{code}
Kind regards
Karl Heinz

> Minor cleanup in MavenCli.java
> ------------------------------
>
>                 Key: MNG-6203
>                 URL: https://issues.apache.org/jira/browse/MNG-6203
>             Project: Maven
>          Issue Type: Improvement
>          Components: Embedding
>    Affects Versions: 3.5.0-beta-1
>            Reporter: Stefan Eicher
>            Priority: Trivial
>         Attachments: Bildschirmfoto 2017-04-02 um 11.26.40.png, 
> Bildschirmfoto 2017-04-02 um 11.37.41.png
>
>
> There seems to be some old unnecessary code in the MavenCli.java
>  that could be cleaned up.
> I will create a pull-request.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)