-
-
Notifications
You must be signed in to change notification settings - Fork 782
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
custom toolchain indexing (arch info) fails #912
Comments
I have fixed this problem, you can update to dev branch and try it again. $ xmake update -s dev
$ xmake f --toolchain=my_llvm -c
$ xmake |
set_toolset("nm", "llvm-nm-10")
set_toolset("objcopy", "llvm-objcopy-10")
set_toolset("objdump", "llvm-objdump-10")
set_toolset("objsize", "llvm-size-10")
set_toolset("strip" , "llvm-strip-10")
set_toolset("readelf", "llvm-readelf-10")
set_toolset("hostcc", "clang-10")
set_toolset("hostcxx", "clang++-10")
set_toolset("hostar", "llvm-ar-10")
set_toolset("hostld", "ld.lld-10") we only support these the toolset key types cc, cxx, ld, ar, sh, strip, .. now. you can see https://xmake.io/#/manual/project_target?id=targetset_toolset we need not nm, objcopy, objsize, objdump, readelf, hostcc, hostar ... |
toolchain("my_llvm")
set_kind("standalone")
set_toolset("cc", "clang-10")
set_toolset("cxx", "clang++-10")
set_toolset("ar", "llvm-ar-10")
set_toolset("ld", "clang++-10") -- we need clang++ as ld to link libc++ library when compile c++ program
set_toolset("sh", "clang++-10") -- for linking shared library
set_toolset("strip" , "llvm-strip-10")
add_defines("my_llvm")
-- optional, If you are sure that you can run clang-10 successfully, you can return true directly, or remove on_check
on_check(function (toolchain)
return import("lib.detect.find_tool")("clang-10")
end)
toolchain_end() |
and neither does
executing |
Despite the issue with
that I do not understand - some source code specify those in their makefiles - could you explain why Xmake does not need those? |
Please try run
xmake does not rely on makefile/make, usually does not use readelf/nm to build the program. |
|
This is because -s will only update lua scripts without binary program and directly overwrite the |
being represented with
sort of leaves one (least it does me) with the impression that things went wrong, more so that the full update process by comparison does not produce such |
You can run After appending -s, it will only update the Lua script, which will be slightly faster, so I added the |
It is ok now that I know that What is the difference between the dev and the master branch, since https://github.com/xmake-io/xmake/compare/dev does not show any? |
Currently these two branches are the same, dev is the development branch, bug fixes and feature updates will be pushed to the dev branch first, and then I will merge to the master branch after stable. |
minor issue
this one https://github.com/xmake-io/xmake/blob/dev/xmake/core/tool/toolchain.lua#L423 raising it? |
I have fixed it, you can update to dev version and try it again. |
confirmed as fixed |
Describe the bug
proceeded according to https://xmake.io/#/manual/custom_toolchain and created custom toolchain
Expected behavior
indexing for custom toolchain to pass
Error output
Related Environment
Please provide compiling and running environment information:
Additional context
Also set
arch = "x86_64"
in xmake.conf (globaldir) which is met with the same errorThe text was updated successfully, but these errors were encountered: