-
Notifications
You must be signed in to change notification settings - Fork 166
Rework TA makefiles #6203
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
Rework TA makefiles #6203
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6203 +/- ##
==========================================
- Coverage 44.41% 44.40% -0.02%
==========================================
Files 390 390
Lines 26986 26986
==========================================
- Hits 11985 11982 -3
- Misses 14158 14160 +2
- Partials 843 844 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
37a1019
to
f156b1c
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.
Mostly questions as I'm not very familiar with TA logic and what's being changed here 👍
packaging/technical-addon/Makefile
Outdated
PLATFORM="all" \ | ||
BUILD_DIR="$(BUILD_DIR)" \ | ||
SOURCE_DIR="$(ADDONS_SOURCE_DIR)" \ | ||
AUTOINSTRUMENTATION_DIR="$(AUTOINSTRUMENTATION_DIR)" \ |
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.
I'm seeing a common pattern in this Makefile is to have commands setting variables at the end. What is the value and purpose of this? Is AUTOINSTRUMENTATION_DIR
going to be used after this for something else?
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.
Removed as not needed until next review, but yes
I don't believe the one failure is related to this test, given it's for ARM
|
This is some clean up, also adds target for new makefiles