Onnx-tensorrt: Missing one header file from installation target.

[lazykyama]

Sorry in advance if it's my misunderstanding. This is a trivial issue.

One header file ( NvOnnxParserTypedefs.h ) seems not to be installed after make install. This file is included from NvOnnxParser.h so needs to be copied to the same directory. The installation log is below.

Install the project...
-- Install configuration: "Release"
-- Installing: /usr/local/bin/onnx2trt
-- Set runtime path of "/usr/local/bin/onnx2trt" to ""
-- Installing: /usr/local/lib/libnvonnxparser.so.0.1.0
-- Installing: /usr/local/lib/libnvonnxparser.so.0
-- Set runtime path of "/usr/local/lib/libnvonnxparser.so.0.1.0" to ""
-- Installing: /usr/local/lib/libnvonnxparser.so
-- Installing: /usr/local/lib/libnvonnxparser_runtime.so.0.1.0
-- Installing: /usr/local/lib/libnvonnxparser_runtime.so.0
-- Installing: /usr/local/lib/libnvonnxparser_runtime.so
-- Installing: /usr/local/lib/libnvonnxparser_static.a
-- Installing: /usr/local/lib/libnvonnxparser_runtime_static.a
-- Installing: /usr/local/include/NvOnnxParser.h
-- Installing: /usr/local/include/NvOnnxParserRuntime.h

[LukeAI]

yep, had the same problem - my packages wouldn't build until I manually did this:
sudo cp NvOnnxParserTypedefs.h /usr/local/include/
and it fixed things - dunno how to edit the repo. to get this to happen by default though?

[bpinaya]

Indeed, this causes the following issue:

/usr/local/include/NvOnnxParser.h:27:10: fatal error: NvOnnxParserTypedefs.h: No such file or directory                                                                                                     
 #include "NvOnnxParserTypedefs.h"  

On Xavier and TX2 and I assume also tx1, file had to manually be copied to install location to fix this.
I think a fix would be to add NvOnnxParserTypedefs.h here in the CMakeLists.txt:

set(HEADERS
  NvOnnxParser.h
  NvOnnxParserRuntime.h
  NvOnnxParserTypedefs.h <-- like this
)

Will try latter.

[kevinch-nv]

This header has been merged into NvOnnxParser.h in later releases. I'll make the change on any older versions that still this.

[kevinch-nv]

Done in https://github.com/onnx/onnx-tensorrt/commit/0b5aeae128a351ef8b130d30d42b66268cb2278f, and cherry-picked across all affected branches. Closing issue