Sergio Peña created SENTRY-2090:
-----------------------------------

             Summary: 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