Only add julia if julia is found on the system
AbandonedPublic

Authored by filipesaraiva on Aug 25 2016, 12:02 PM.

Details

Reviewers
tcanabrava
Summary

This code is based on code found on https://gist.github.com/JayKickliter
for the FindJulia.cmake. the rest was just adding the necessary
bits for correctnly finding and linking against the libjulia if julia
is found on the system, and do nothing otherwise.

Test Plan

compiles fine now.

Diff Detail

Repository
R55 Cantor
Branch
fixJulia
Lint
No Linters Available
Unit
No Unit Test Coverage
tcanabrava updated this revision to Diff 6268.Aug 25 2016, 12:02 PM
tcanabrava retitled this revision from to Only add julia if julia is found on the system.
tcanabrava updated this object.
tcanabrava edited the test plan for this revision. (Show Details)
tcanabrava added a reviewer: filipesaraiva.
filipesaraiva edited edge metadata.Aug 25 2016, 2:45 PM
filipesaraiva added a subscriber: ivanlakhtanov.

Thanks @tcanabrava but @ivanlakhtanov is working in this bug with this same approach in D2575.

Please, can you close this revision? I can not to do it. :\

ivanlakhtanov added inline comments.Aug 25 2016, 2:52 PM
cmake/FindJulia.cmake
36

This is not always correct as output contains newlines. You need to strip returned value.

40

You aren't checking that /usr/include/julia even exists. Julia executable returns it always, even when no folder present. Installation of Julia without devel package may be accepted by this script.

45

This is now outdated. Latest julia has this function: Libdl.dlpath and there is no dlpath in Sys

src/backends/julia/juliaserver/CMakeLists.txt
10

Thanks, I've missed it creating my patch.

Thanks @tcanabrava, the revision D2575 was merged, now this revision is no more necessary.

filipesaraiva commandeered this revision.Sep 20 2016, 1:59 PM
filipesaraiva abandoned this revision.
filipesaraiva edited reviewers, added: tcanabrava; removed: filipesaraiva.