[PR #1285] [MERGED] fix: Failed to execute cmake at client_server directory. #942 #1889

Open
opened 2026-01-29 15:25:48 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/1285
Author: @rbevin777
Created: 7/6/2023
Status: Merged
Merged: 7/19/2023
Merged by: @Panquesito7

Base: masterHead: Bug-CMake-build-problem-in-client_server-dir-#942


📝 Commits (3)

  • 213a922 fix: missing include file for cmake function
  • 763cbb2 fix: using newer version of check include file
  • 10eeedf Merge branch 'master' into Bug-CMake-build-problem-in-client_server-dir-#942

📊 Changes

1 file changed (+4 additions, -2 deletions)

View changed files

📝 client_server/CMakeLists.txt (+4 -2)

📄 Description

Description of Change

The include file for this function was missing so I added it in and updated the function call too.

References

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/TheAlgorithms/C/pull/1285 **Author:** [@rbevin777](https://github.com/rbevin777) **Created:** 7/6/2023 **Status:** ✅ Merged **Merged:** 7/19/2023 **Merged by:** [@Panquesito7](https://github.com/Panquesito7) **Base:** `master` ← **Head:** `Bug-CMake-build-problem-in-client_server-dir-#942` --- ### 📝 Commits (3) - [`213a922`](https://github.com/TheAlgorithms/C/commit/213a922a82f9eb71ab06ce13e9d0e13cb249467b) fix: missing include file for cmake function - [`763cbb2`](https://github.com/TheAlgorithms/C/commit/763cbb28fea693e3e638184272897b835228c023) fix: using newer version of check include file - [`10eeedf`](https://github.com/TheAlgorithms/C/commit/10eeedfd0a64d1b1b9adfe05475269d226bd5d0f) Merge branch 'master' into Bug-CMake-build-problem-in-client_server-dir-#942 ### 📊 Changes **1 file changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `client_server/CMakeLists.txt` (+4 -2) </details> ### 📄 Description #### Description of Change The include file for this function was missing so I added it in and updated the function call too. <!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Contributors guide: https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md --> #### References <!-- Add any reference to previous pull-request or issue --> #### Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [x] Added description of change - [ ] ~Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#File-Name-guidelines)~ - [ ] ~Added tests and example, test must pass~ - [ ] ~Relevant documentation/comments is changed or added~ - [x] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes: <!-- Please add a one-line description for developers or pull request viewers --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:25:48 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1889