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

Alexander Kolbasov commented on SENTRY-2090:
--------------------------------------------

[~spena] Do you think that this is a blocker for 2.0 release or not?

> 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)

Reply via email to