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

Name remux threads #221

Merged
merged 6 commits into from
May 28, 2024
Merged

Name remux threads #221

merged 6 commits into from
May 28, 2024

Conversation

Eeems
Copy link
Contributor

@Eeems Eeems commented May 24, 2024

This gives the threads custom names to easily identify which thread is doing work in tools like htop.

@Eeems
Copy link
Contributor Author

Eeems commented May 25, 2024

image

@Eeems Eeems marked this pull request as ready for review May 25, 2024 00:01
Copy link
Member

@raisjn raisjn left a 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
Copy link
Member

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

Copy link
Contributor Author

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?

CROSS_TC=arm-kobo-linux-gnueabihf DOCKERFILE=Dockerfile.kobo FBINK=1 TARGET=kobo make docker
TARGET=rm make docker

Copy link
Member

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

@raisjn raisjn merged commit 3197799 into rmkit-dev:master May 28, 2024
1 check passed
@Eeems Eeems deleted the patch-1 branch May 28, 2024 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants