SSE4 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from 
conan
URL: https://github.com/apache/incubator-mxnet/pull/13400#issuecomment-461372230
 
 
   @edisongustavo 
   1. for sure I know about targets vs variables approaches in CMake. I suppose 
it's just matter of minimum CMake version being targeted, as there are still 
many projects targetting CMake 2.X, article mentions variable approach being 
more compatible, in general. as MXNet is targeting 3.0 and higher, it's safe to 
use CMake targets instead.
   2. yes, it's possible to use system libraries with route specified, and via 
[conditional 
requirements](https://docs.conan.io/en/latest/mastering/conditional.html). I 
believe, `conanfile.py` of MXNet will look like:
   ```
   def requirements(self):
       if not self.options.use_system_openblas:
           self.requires.add("openblas/0.2.20@conan/stable")
       # similar code for other libraries
   ```
   and then use something like:
   ```
   conan install -o MXNet:use_system_openblas=True ...
   ```
   you know, this PR is more like proof-of-concept, and `conanfile.py` is very 
flexible in terms of adding custom logic in order to satisfy requirements of 
specific project (MXNet)
   3. yes, mirroring is definitely possible, and some conan users have already 
set up mirroring for their repositories. I think it's just matter of simple 
script combining [conan 
download](https://docs.conan.io/en/latest/reference/commands/misc/download.html)
 and [conan 
upload](https://docs.conan.io/en/latest/reference/commands/creator/upload.html) 
commands.

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