ServoKvd commented on issue #2979: Removal of openwhisk cli and cli related 
test cases
URL: 
https://github.com/apache/incubator-openwhisk/pull/2979#issuecomment-346580524
 
 
   Hi houshengbo, csantanapr
   Today I notice the following files in openwhisk still has references to cli, 
though in some cases the build does nto complain:
   1.)reference to cli settings.gradle in the root folder -
    ./gradlew distDocker executes fine, even though cli folder is not there, 
since it recursively checks for build.gradle and doesn seem to use 
settings.gradle to do the build itself. 
   2.) Reference to cli in  tools/vagrant/Vagrantfile
   Chetanm reported a build failure on teh slack channel today - so i guess 
this needs to be fixed.
   3.) references to  "tools:cli" in bin/wskdev,  tools/admin/wskadmin
   I guess wskdev & wdkadmin will also move to the cli folder soon?
   4.) reference to tools:cli component in tools/build/redo 
   
   I was wondering about a fix for (1,2,4) above - can it simply be omiitting 
to make the "cli" or should it include sncyign to the cli repo and copying the 
cli from there.  
   
   Any comments?  

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to