You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/10/26 19:17:10 UTC
[GitHub] [incubator-tvm] gussmith23 opened a new pull request #6765: Add include path to Rust build script
gussmith23 opened a new pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765
@jroesch
I'm not sure what we want to do here. I needed to add this path to make the build script work on my system -- perhaps there's something I should do on my side, though?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tkonolige edited a comment on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tkonolige edited a comment on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718200850
Have you tried specifying `LLVM_CONFIG_PATH` to point to your llvm install?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] gussmith23 commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
gussmith23 commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718262036
Oops, this problem isn't resolved! Sorry, didn't mean to imply that. I had already set `LLVM_CONFIG_PATH` before getting this error.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] gussmith23 commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
gussmith23 commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718217093
Yes, it's set! I think I was getting a different error before I realized I needed to set that.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] gussmith23 commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
gussmith23 commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718198239
```
error: failed to run custom build command for `tvm-sys v0.1.0 (https://github.com/apache/incubator-tvm?branch=main#4c4888bc)`
Caused by:
process didn't exit successfully: `/home/gus/glenside-evaluation/target/debug/build/tvm-sys-6e86f41bfc277513/build-script-build` (exit code: 1)
--- stdout
cargo:rerun-if-env-changed=TVM_HOME
cargo:rustc-link-lib=dylib=tvm
cargo:rustc-link-search=/home/gus/tvm/build
--- stderr
/home/gus/tvm/3rdparty/dlpack/include/dlpack/dlpack.h:30:10: fatal error: 'stddef.h' file not found
/home/gus/tvm/3rdparty/dlpack/include/dlpack/dlpack.h:30:10: fatal error: 'stddef.h' file not found, err: true
Error: failed to generate bindings
```
Yeah, I agree that it doesn't seem to make sense. I'm having a hell of a time resolving this on one of our machines, and I have to assume it's because of how the machine is configured. We have stddef.h (multiple, in fact) but I have no idea which toolchain rust is trying to use here, so I don't know which one it's failing to find.
It seems to be happening in bindgen, and there's a running thread about issues like this: https://github.com/rust-lang/rust-bindgen/issues/1226. It seems like it's not actually an issue on their side, but it's common when using bindgen.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tkonolige commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tkonolige commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-716823445
Could you post the error message you get when building?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tkonolige commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tkonolige commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718200850
Have you tried specifying `LLVM_CONFIG_PATH`?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tkonolige edited a comment on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tkonolige edited a comment on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-716823445
Could you post the error message you get when building?
I don't think we should hardcode a path like this.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] gussmith23 commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
gussmith23 commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-721327488
I have, to seemingly no effect. I'll bug our sysadmin about this. It must
be on my side.
On Wed, Oct 28, 2020 at 4:23 PM Tianqi Chen <no...@github.com>
wrote:
> In this case it is indeed versy strange that stddef is not part of your
> include path, how about you add it to your CPLUS_INCLUDE_PATH and
> C_INCLUDE_PATH env variable in your system?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718264432>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAJFZAKQIN4YXDUNFVHLUGTSNCRXNANCNFSM4S7ZXP4Q>
> .
>
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tqchen commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718218179
Closing, glad that the problem is resolved
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] gussmith23 edited a comment on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
gussmith23 edited a comment on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718198239
```
error: failed to run custom build command for `tvm-sys v0.1.0 (https://github.com/apache/incubator-tvm?branch=main#4c4888bc)`
Caused by:
process didn't exit successfully: `/home/gus/glenside-evaluation/target/debug/build/tvm-sys-6e86f41bfc277513/build-script-build` (exit code: 1)
--- stdout
cargo:rerun-if-env-changed=TVM_HOME
cargo:rustc-link-lib=dylib=tvm
cargo:rustc-link-search=/home/gus/tvm/build
--- stderr
/home/gus/tvm/3rdparty/dlpack/include/dlpack/dlpack.h:30:10: fatal error: 'stddef.h' file not found
/home/gus/tvm/3rdparty/dlpack/include/dlpack/dlpack.h:30:10: fatal error: 'stddef.h' file not found, err: true
Error: failed to generate bindings
```
Yeah, I agree that it doesn't make sense to hardcode it here. I'm having a hell of a time resolving this on one of our machines, and I have to assume it's because of how the machine is configured. We have stddef.h (multiple, in fact) but I have no idea which toolchain rust is trying to use here, so I don't know which one it's failing to find.
It seems to be happening in bindgen, and there's a running thread about issues like this: https://github.com/rust-lang/rust-bindgen/issues/1226. It seems like it's not actually an issue on their side, but it's common when using bindgen.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tqchen commented on pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765#issuecomment-718264432
In this case it is indeed versy strange that stddef is not part of your include path, how about you add it to your CPLUS_INCLUDE_PATH and C_INCLUDE_PATH env variable in your system?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tqchen closed pull request #6765: Add include path to Rust build script
Posted by GitBox <gi...@apache.org>.
tqchen closed pull request #6765:
URL: https://github.com/apache/incubator-tvm/pull/6765
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org