You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sentry.apache.org by "Sergio Peña (JIRA)" <ji...@apache.org> on 2017/12/05 17:11:00 UTC

[jira] [Commented] (SENTRY-2090) Move license information downloads to mvn package phase

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

Sergio Peña commented on SENTRY-2090:
-------------------------------------

[~kkalyan] [~btowles] FYI

> Move license information downloads to mvn package phase
> -------------------------------------------------------
>
>                 Key: SENTRY-2090
>                 URL: https://issues.apache.org/jira/browse/SENTRY-2090
>             Project: Sentry
>          Issue Type: Bug
>          Components: Sentry
>    Affects Versions: 2.0.0
>            Reporter: Sergio Peña
>
> I noticed that license information is downloaded on newer developer environments everytime I run {{mvn install}}:
> {noformat}
> [INFO] ------------------------------------------------------------------------                                                                                                                                    
> [INFO] Building Sentry Distribution 2.1.0-SNAPSHOT                                                                                                                                                                 
> [INFO] ------------------------------------------------------------------------                                                                                                                                    
> [INFO]                                                                                                                                                                                                             
> [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ sentry-dist ---                                                                                                                                          
> [INFO] Deleting /home/sergio/Development/Apache/sentry/sentry-dist/target                                                                                                                                          
> [INFO]                                                                                                                                                                                                             
> [INFO] >>> maven-pmd-plugin:3.7:check (validate) > :pmd @ sentry-dist >>>                                                                                                                                          
> [INFO]                                                                                                                                                                                                             
> [INFO] --- maven-pmd-plugin:3.7:pmd (pmd) @ sentry-dist ---                                                                                                                                                        
> [INFO]                                                                                                                                                                                                             
> [INFO] <<< maven-pmd-plugin:3.7:check (validate) < :pmd @ sentry-dist <<<                                                                                                                                          
> [INFO]                                                                                                                                                                                                             
> [INFO]                                                                                                                                                                                                             
> [INFO] --- maven-pmd-plugin:3.7:check (validate) @ sentry-dist ---                                                                                                                                                 
> [INFO]                                                                                                                                                                                                             
> [INFO]                                                                                                                                                                                                             
> [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ sentry-dist ---                                                                                                                                   
> [INFO]                                                                                                                                                                                                             
> [INFO] --- license-maven-plugin:1.14:add-third-party (add-third-party) @ sentry-dist ---                                                                                                                           
> [WARNING] The license for ant-contrib--ant-contrib--1.0b3 has no name (but exist)                                                                                                                                  
> [INFO] Load missing file /home/sergio/Development/Apache/sentry/sentry-dist/src/license/THIRD-PARTY.properties                                                                                                     
> Downloading: https://repo.maven.apache.org/maven2/ant/ant/1.5/ant-1.5-third-party.properties                                                                                                                       
> Downloading: https://repo.maven.apache.org/maven2/ant-contrib/ant-contrib/1.0b3/ant-contrib-1.0b3-third-party.properties                                                                                           
> Downloading: https://repo.maven.apache.org/maven2/aopalliance/aopalliance/1.0/aopalliance-1.0-third-party.properties                                                                                               
> Downloading: https://repo.maven.apache.org/maven2/asm/asm/3.1/asm-3.1-third-party.properties                                                                                                                       
> Downloading: https://repo.maven.apache.org/maven2/co/cask/tephra/tephra-api/0.6.0/tephra-api-0.6.0-third-party.properties                                                                                          
> Downloading: https://repo.maven.apache.org/maven2/co/cask/tephra/tephra-core/0.6.0/tephra-core-0.6.0-third-party.properties                                                                                        
> Downloading: https://repo.maven.apache.org/maven2/co/cask/tephra/tephra-hbase-compat-1.0/0.6.0/tephra-hbase-compat-1.0-0.6.0-third-party.properties                                                                
> Downloading: https://repo.maven.apache.org/maven2/com/beust/jcommander/1.27/jcommander-1.27-third-party.properties                                                                                                 
> Downloading: https://repo.maven.apache.org/maven2/com/budhash/cliche/cliche-shell/0.9.3/cliche-shell-0.9.3-third-party.properties                                                                                  
> Downloading: https://repo.maven.apache.org/maven2/com/carrotsearch/hppc/0.7.1/hppc-0.7.1-third-party.properties                                                                                                    
> Downloading: https://repo.maven.apache.org/maven2/com/codahale/metrics/metrics-core/3.0.2/metrics-core-3.0.2-third-party.properties                                                                                
> .....
> {noformat}
> It downloads several files and it adds extra time on the build even if they're not necessary. I think licensing information should be fetched on the mvn package phase and not the install.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)