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

Format code for PR #6153 #42

Conversation

slangbot
Copy link

Automated code formatting for shader-slang#6153

@slangbot slangbot force-pushed the format-6153-aleino/formatting-fixes branch from 953feff to a60f6aa Compare January 23, 2025 13:35
@aleino-nv aleino-nv merged commit 510ac20 into aleino-nv:aleino/formatting-fixes Jan 23, 2025
1 check failed
aleino-nv pushed a commit that referenced this pull request Mar 3, 2025
)

* Fix build error when SLANG_ENABLE_TESTS is set to OFF

When SLANG_ENABLE_TESTS is disabled but SLANG_ENABLE_GFX is enabled,
cmake was failing. It turned out that gfx build target was trying to
copy some of files to the directory where slang-test uses. When
SLANG_ENABLE_TESTS is disabled, the directory information became
unavailable, because slang-test target became unavailable.

This commit makes the copying behavior conditional on SLANG_ENABLE_TESTS
in the gfx build target.

* format code (#42)

Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com>

---------

Co-authored-by: slangbot <ellieh+slangbot@nvidia.com>
Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com>
Co-authored-by: Ellie Hermaszewska <ellieh@nvidia.com>
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.

None yet

2 participants