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

Fix bug in CMake function #7604

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Fix bug in CMake function #7604

merged 1 commit into from
Jun 27, 2023

Conversation

stsoe
Copy link
Collaborator

@stsoe stsoe commented Jun 27, 2023

Problem solved by the commit

Ignore missing directory in xrt_add_subdirectory.

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

xrt_add_subdirectory is used to conditionally exclude directories from CMake generation. If the directory was explicitly removed prior to CMake generation, the function would error.

How problem was solved, alternative solutions (if any) and why they were rejected

Add test that directory exists.

Ignore missing directory in xrt_add_subdirectory.

xrt_add_subdirectory is used to conditionally exclude directories from
CMake generation.  If the directory was explicitly removed prior to
CMake generation, the function would error.

Add test that directory exists.
@stsoe stsoe requested a review from rozumx as a code owner June 27, 2023 19:52
@gbuildx
Copy link
Collaborator

gbuildx commented Jun 27, 2023

Build Passed!

@stsoe stsoe merged commit ff95b8a into Xilinx:master Jun 27, 2023
2 checks passed
@stsoe stsoe deleted the xrt_add_subdiretory branch June 27, 2023 21:53
abalasa pushed a commit to abalasa/XRT-1 that referenced this pull request Jun 30, 2023
Ignore missing directory in xrt_add_subdirectory.

xrt_add_subdirectory is used to conditionally exclude directories from
CMake generation.  If the directory was explicitly removed prior to
CMake generation, the function would error.

Add test that directory exists.
abalasa added a commit that referenced this pull request Jun 30, 2023
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