Skip to content
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

fix(trtexec_vendor, tensorrt_yolo): fix build error #8959

Merged

Conversation

tkimura4
Copy link
Contributor

@tkimura4 tkimura4 commented Sep 26, 2024

Description

Fix following build error

--- stderr: trtexec_vendor                                                                                                                                                                       
[ 11%] Creating directories for 'tensorrt-populate'
[ 22%] Performing download step (git clone) for 'tensorrt-populate'
Cloning into 'tensorrt-src'...
fatal: invalid reference: 8.6.1
CMake Error at tensorrt-subbuild/tensorrt-populate-prefix/tmp/tensorrt-populate-gitclone.cmake:40 (message):
  Failed to checkout tag: '8.6.1'


gmake[2]: *** [CMakeFiles/tensorrt-populate.dir/build.make:102: tensorrt-populate-prefix/src/tensorrt-populate-stamp/tensorrt-populate-download] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/tensorrt-populate.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2

CMake Error at /usr/share/cmake-3.22/Modules/FetchContent.cmake:1087 (message):
  Build step for tensorrt failed: 2
Call Stack (most recent call first):
  /usr/share/cmake-3.22/Modules/FetchContent.cmake:1216:EVAL:2 (__FetchContent_directPopulate)
  /usr/share/cmake-3.22/Modules/FetchContent.cmake:1216 (cmake_language)
  CMakeLists.txt:45 (fetchcontent_populate)


---
Failed   <<< trtexec_vendor [36.4s, exited with code 1]

Related links

[TIERIV Internal Link] https://star4.slack.com/archives/CTEJP8L4T/p1727267287905739

How was this PR tested?

We confirmed that the build of autoware successed

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@tkimura4 tkimura4 requested review from wep21, a team and knzo25 as code owners September 26, 2024 01:58
@tkimura4 tkimura4 marked this pull request as draft September 26, 2024 01:58
@tkimura4 tkimura4 requested review from shmpwk and taikitanaka3 and removed request for a team, knzo25 and wep21 September 26, 2024 01:59
@tkimura4 tkimura4 marked this pull request as ready for review September 26, 2024 03:40
Copy link
Contributor

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shmpwk shmpwk merged commit f7a1c8d into autowarefoundation:awsim-stable Sep 26, 2024
16 of 20 checks passed
@shmpwk shmpwk deleted the awsim-stable-patch-20240926 branch September 26, 2024 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants