-
Notifications
You must be signed in to change notification settings - Fork 668
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
build(trtexec_vendor): fix cmake to find trtexec bin #5464
Conversation
bf7f263
to
d485c7e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Head branch was pushed to by a user without write access
73bb7db
to
0e09e59
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #5464 +/- ##
==========================================
- Coverage 15.32% 15.32% -0.01%
==========================================
Files 1721 1721
Lines 118559 118561 +2
Branches 37995 37999 +4
==========================================
- Hits 18169 18168 -1
Misses 79657 79657
- Partials 20733 20736 +3
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
0e09e59
to
b6eee13
Compare
Signed-off-by: CW01\uig08771 <[email protected]>
Signed-off-by: CW01\uig08771 <[email protected]>
Co-authored-by: M. Fatih Cırıt <[email protected]>
fffcc61
to
9ae9d03
Compare
Internet access is disable during build process in github actions due to this PR. |
The package was removed by this PR. |
Description
This pr fixes #5453
Tests performed
cmake build of trtexec_vendor
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.