mirror of
https://github.com/RGBCube/serenity
synced 2025-07-28 10:07:35 +00:00
Ports/cmake: Update cmake to version 3.23.0 and remove upstreamed patch
The check for HAVE_SIZEOF_SSIZE_T and not HAVE_SSIZE_T has been upstreamed in 99acfe41f5
.
This commit is contained in:
parent
7be57c322e
commit
e6d28a9dd4
10 changed files with 10 additions and 72 deletions
|
@ -1,43 +0,0 @@
|
|||
From b0ae5bb3a030450d9102a67b3695738a65531606 Mon Sep 17 00:00:00 2001
|
||||
From: Ali Mohammad Pur <ali.mpfard@gmail.com>
|
||||
Date: Fri, 24 Dec 2021 05:13:08 +0330
|
||||
Subject: [PATCH 1/7] cmnghttp2: check for HAVE_SIZEOF_SSIZE_T and not
|
||||
HAVE_SSIZE_T
|
||||
|
||||
The `check_size_type(ssize_t SIZEOF_SSIZE_T` call in cmcurl (referenced
|
||||
by the comment above, which also references some other variables that
|
||||
no longer seem to be used) defines HAVE_SIZEOF_SSIZE_T and not
|
||||
HAVE_SSIZE_T.
|
||||
The HAVE_SSIZE_T variable *does* get defined, but via the
|
||||
`CHECK_SIZE_TYPE(ssize_t SSIZE_T)` call in cmlibarchive, which gets
|
||||
configured *after* cmnghttp2, and so the first configure leads to an
|
||||
invalid cmnghttp2/config.h file.
|
||||
|
||||
- [ ] Local?
|
||||
- [X] Should be merged to upstream?
|
||||
- [ ] Resolves issues(s) with our side of things
|
||||
- [ ] Hack
|
||||
---
|
||||
Utilities/cmnghttp2/CMakeLists.txt | 6 ++----
|
||||
1 file changed, 2 insertions(+), 4 deletions(-)
|
||||
|
||||
diff --git a/Utilities/cmnghttp2/CMakeLists.txt b/Utilities/cmnghttp2/CMakeLists.txt
|
||||
index 3bc2778e..7e8bf99b 100644
|
||||
--- a/Utilities/cmnghttp2/CMakeLists.txt
|
||||
+++ b/Utilities/cmnghttp2/CMakeLists.txt
|
||||
@@ -7,10 +7,8 @@ elseif(CMAKE_C_COMPILER_ID STREQUAL "PathScale")
|
||||
endif()
|
||||
|
||||
# Re-use some check result cache entries from cmcurl:
|
||||
-# * HAVE_ARPA_INET_H
|
||||
-# * HAVE_NETINET_IN_H
|
||||
-# * HAVE_SSIZE_T
|
||||
-if(NOT HAVE_SSIZE_T)
|
||||
+# * HAVE_SIZEOF_SSIZE_T
|
||||
+if(NOT HAVE_SIZEOF_SSIZE_T)
|
||||
set(ssize_t KWIML_INT_intptr_t)
|
||||
endif()
|
||||
configure_file(cmakeconfig.h.in config.h)
|
||||
--
|
||||
2.34.1
|
||||
|
|
@ -1,24 +1,6 @@
|
|||
# Patches for cmake on SerenityOS
|
||||
|
||||
## `0001-cmnghttp2-check-for-HAVE_SIZEOF_SSIZE_T-and-not-HAVE.patch`
|
||||
|
||||
cmnghttp2: check for HAVE_SIZEOF_SSIZE_T and not HAVE_SSIZE_T
|
||||
|
||||
The `check_size_type(ssize_t SIZEOF_SSIZE_T` call in cmcurl (referenced
|
||||
by the comment above, which also references some other variables that
|
||||
no longer seem to be used) defines HAVE_SIZEOF_SSIZE_T and not
|
||||
HAVE_SSIZE_T.
|
||||
The HAVE_SSIZE_T variable *does* get defined, but via the
|
||||
`CHECK_SIZE_TYPE(ssize_t SSIZE_T)` call in cmlibarchive, which gets
|
||||
configured *after* cmnghttp2, and so the first configure leads to an
|
||||
invalid cmnghttp2/config.h file.
|
||||
|
||||
- [ ] Local?
|
||||
- [X] Should be merged to upstream?
|
||||
- [ ] Resolves issues(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
## `0002-kwsys-Don-t-use-siginfo.patch`
|
||||
## `0001-kwsys-Don-t-use-siginfo.patch`
|
||||
|
||||
kwsys: Don't use siginfo
|
||||
|
||||
|
@ -29,7 +11,7 @@ We don't support SIGINFO. This patch removes uses of SIGINFO.
|
|||
- [X] Resolves issue(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
## `0003-bin-bash.patch`
|
||||
## `0002-bin-bash.patch`
|
||||
|
||||
/bin/bash
|
||||
|
||||
|
@ -40,7 +22,7 @@ This patch swaps out /bin/sh for /bin/bash in two scripts that need it.
|
|||
- [ ] Resolves issue(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
## `0004-Platform-SerenityOS.patch`
|
||||
## `0003-Platform-SerenityOS.patch`
|
||||
|
||||
Platform/SerenityOS
|
||||
|
||||
|
@ -51,7 +33,7 @@ This patch adds the SerenityOS platform config file to CMake.
|
|||
- [ ] Resolves issue(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
## `0005-cmcurl-Include-unistd.patch`
|
||||
## `0004-cmcurl-Include-unistd.patch`
|
||||
|
||||
cmcurl: Include unistd
|
||||
|
||||
|
@ -62,18 +44,18 @@ Everyone gets this wrong. most platforms are very lax with these includes, but w
|
|||
- [ ] Resolves issue(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
## `0006-cmcurl-Use-struct-stat-and-include-sys-stat.h.patch`
|
||||
## `0005-cmcurl-Use-struct-stat-and-include-sys-stat.h.patch`
|
||||
|
||||
cmcurl: Use struct stat and include sys/stat.h
|
||||
|
||||
For unknown reasons, curl_setup_once.h does not include sys/stat.h. this patch includes sys/stat.h.
|
||||
For unknown reasons, curl_setup_once.h does not include sys/stat.h. This patch includes sys/stat.h.
|
||||
|
||||
- [ ] Local?
|
||||
- [ ] Should be merged to upstream?
|
||||
- [X] Resolves issue(s) with our side of things
|
||||
- [X] Hack
|
||||
|
||||
## `0007-CMake-Disable-tests.patch`
|
||||
## `0006-CMake-Disable-tests.patch`
|
||||
|
||||
CMake: Disable tests
|
||||
|
||||
|
@ -83,4 +65,3 @@ We don't care about building tests for now, and it makes the compilation much fa
|
|||
- [ ] Should be merged to upstream?
|
||||
- [ ] Resolves issue(s) with our side of things
|
||||
- [ ] Hack
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue