-
Notifications
You must be signed in to change notification settings - Fork 32
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
Name remux threads #221
Name remux threads #221
Conversation
This gives the threads custom names to easily identify which thread is doing work in tools like htop.
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.
looks good, I need to test it on kobo, too, then will merge. should be good enough for light debugging now, though
|
||
- name: "Build artifacts" | ||
run: | | ||
ARCH=rm make docker | ||
ARCH=kobo make docker | ||
CROSS_TC=arm-kobo-linux-gnueabihf DOCKERFILE=Dockerfile.kobo FBINK=1 TARGET=kobo make docker |
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.
not sure we need this change or if using target=kobo is enough
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 was copying what was in the stable-release action. Should they both be changed to just TARGET=kobo
?
rmkit/.github/workflows/stable-release.yml
Lines 29 to 30 in 381ef9d
CROSS_TC=arm-kobo-linux-gnueabihf DOCKERFILE=Dockerfile.kobo FBINK=1 TARGET=kobo make docker | |
TARGET=rm make docker |
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.
ah, thanks for fixing it here, then. I think i trust stable release, since that's been running and uploading artifacts
This gives the threads custom names to easily identify which thread is doing work in tools like htop.