Added check for return code when checking for bzr.

 






 

 


  Review Request #11196
  —
  Created Sept. 21, 2020 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  ruonanjia
 









 
 

  RBTools
 









 
 

  master
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  rbtools, students
 









 
 

  
 






 








 





  


 
 

  
When pyenv is in use and the needed executable is only available in a 

different Python version, pyenv intercepts the call and will print 

an error/return 127.


When running bzr help under these conditions, we see this:

pyenv: bzr: command not found


The `bzr' command exists in these Python versions:

  2.7.14

 








  


 
 

  
Tested 'rbt post' manually in my local integrations repo with the changes.

 







 









 





  








 




 







   

    
    
Diff Revision 4


This is not the most recent revision of the diff. The latest diff is revision 8. See what's changed.

    

 

 

 

 
orig
1
2
3
4
5
6
7
8

 



    


    

      
Commits

      

 

 

  
   First
   Last
  
  Summary
  ID
  Author
 



 
  
   
  
  
   
  
 
 
  
   
    
added check for return code and stderr when running bzr help

   
  		
  931fe641ac5fc74dd397c36fb3869a2940402a4f
  ruonan
 


 
  
   
  
  
   
  
 
 
  
   
    
added check for return code and stderr when running bzr help

   
  		
  bf05dc72ccb71a2f2cabcd24b801b23077db2567
  ruonan
 


 
  
   
  
  
   
  
 
 
  
   
    
added check for return code and stderr when running bzr help

   
  		
  1be08c9d4b8681bc48bb0743941ea9535928ee64
  ruonan
 




    


    
    

   

  

 









 

  

   
    

     
      
      rbtools/utils/checks.py
     
    

   
  

 










     

    

   

  

  

   
   Loading...