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(CMake): add a download target to only download all third parties #892

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

BotellaA
Copy link
Member

@BotellaA BotellaA commented Feb 6, 2024

No description provided.

Copy link
Contributor

github-actions bot commented Feb 6, 2024

clang-tidy review says "All clean, LGTM! 👍"

@@ -18,6 +18,8 @@
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
# SOFTWARE.

cmake_policy(SET CMP0114 NEW)
Copy link
Member

Choose a reason for hiding this comment

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

Only for personal culture, what does it stand for?

Copy link
Member Author

Choose a reason for hiding this comment

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

https://cmake.org/cmake/help/latest/policy/CMP0114.html
mainly a change in the behavior of the dependent steps between several external projects

@panquez panquez merged commit 5aa3414 into next Feb 6, 2024
39 checks passed
@panquez panquez deleted the fix/offline branch February 6, 2024 11:12
@BotellaA
Copy link
Member Author

BotellaA commented Feb 6, 2024

🎉 This PR is included in version 14.14.1-rc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@BotellaA
Copy link
Member Author

🎉 This PR is included in version 14.14.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants