Skip to content

enhance get_cuda_object_dump_raw to not run cuobjdump on shell scripts to determine CUDA architecture#5161

Merged
boegel merged 2 commits intoeasybuilders:developfrom
Flamefire:cuobjdump-text
Apr 8, 2026
Merged

enhance get_cuda_object_dump_raw to not run cuobjdump on shell scripts to determine CUDA architecture#5161
boegel merged 2 commits intoeasybuilders:developfrom
Flamefire:cuobjdump-text

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

Before checking a file for CUDA code a file command is used and the
output search for "executable", which wrongly triggers for Python, Bash, ... scripts

Exclude those to avoid unnecessary commands and errors logged.

Before checking a file for CUDA code a `file` command is used and the
output search for "executable", which wrongly triggers for Python, Bash, ... scripts

Exclude those to avoid uneccessary commands and errors logged.
@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 8, 2026

@Flamefire Do you have a concrete example of what this avoids/fixes, just for context?

@boegel boegel changed the title Don't check scripts for CUDA architectures enhance get_cuda_object_dump_raw to not run cuobjdump on shell scripts to determine CUDA architecture Apr 8, 2026
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel added the bug fix label Apr 8, 2026
@boegel boegel added this to the next release (5.3.0) milestone Apr 8, 2026
@boegel boegel merged commit 097ccec into easybuilders:develop Apr 8, 2026
40 checks passed
@Flamefire Flamefire deleted the cuobjdump-text branch April 8, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants