Use nix #614
Use nix #614
Conversation
AndreyMlashkin
force-pushed
the
use_nix
branch
4 times, most recently
from
c0a8c53
to
a0d813c
Compare
AndreyMlashkin
force-pushed
the
use_nix
branch
4 times, most recently
from
5288fba
to
e919736
Compare
bin/transpiler/CMakeLists.txt
Outdated
| ${CMAKE_SOURCE_DIR}/libs/assigner/include | ||
| ${CMAKE_SOURCE_DIR}/libs/transpiler/include |
There was a problem hiding this comment.
libs/transpiler/include still remains
There was a problem hiding this comment.
I've tried removing it, it seems, it does not export the targets the right way.
bin/assigner/CMakeLists.txt
Outdated
| ${CMAKE_CURRENT_SOURCE_DIR}/../../libs/circifier/utils/bazel/llvm-project-overlay/llvm/include | ||
| ${CMAKE_SOURCE_DIR}/libs/circifier/llvm/include | ||
| ${CMAKE_SOURCE_DIR}/libs/circifier/utils/bazel/llvm-project-overlay/llvm/include | ||
| ${CMAKE_SOURCE_DIR}/libs/assigner/include |
There was a problem hiding this comment.
libs/assigner/include still remains
There was a problem hiding this comment.
I've tried removing it, it seems, it does not export the targets the right way.
There was a problem hiding this comment.
Yes, it is not enough just to remove it. But you can restore crypto3::assigner and crypto3::transpiler as it was at lines 55-56 of the previous version of the file. It is basically the same way as you have just fixed bin/recursive_gen/CMakeLists.txt, so I am almost sure that it would work.
There was a problem hiding this comment.
Let's try use interface libraries: crypto3::assigner and crypto3::transpiler.
Sound like quick change. If doesn't works - keep as is
AndreyMlashkin
force-pushed
the
use_nix
branch
2 times, most recently
from
4d8469b
to
a0cc252
Compare
AndreyMlashkin
force-pushed
the
use_nix
branch
3 times, most recently
from
07c8201
to
1bd3267
Compare
No description provided.