build: Add upstream pcre2 patch
This commit is contained in:
parent
b3a56a3076
commit
0b3cb5c712
|
@ -0,0 +1,285 @@
|
|||
From e2b8c3fd1e5eb06c137daf03dcb6c11f93aa2cd4 Mon Sep 17 00:00:00 2001
|
||||
From: Yuxuan Shui <yshuiv7@gmail.com>
|
||||
Date: Tue, 22 Nov 2022 16:12:36 +0000
|
||||
Subject: [PATCH] c2: replace pcre with pcre2
|
||||
|
||||
Because pcre has been deprecated.[1]
|
||||
|
||||
There are subtle changes from pcre to pcre2, so this could be a breaking
|
||||
change.
|
||||
|
||||
Closes #895
|
||||
|
||||
[1]: https://www.pcre.org/original/changelog.txt
|
||||
|
||||
Signed-off-by: Yuxuan Shui <yshuiv7@gmail.com>
|
||||
---
|
||||
src/c2.c | 155 ++++++++++++++++++++++++------------------------
|
||||
src/meson.build | 5 +-
|
||||
2 files changed, 79 insertions(+), 81 deletions(-)
|
||||
|
||||
diff --git a/src/c2.c b/src/c2.c
|
||||
index 80ecb24c..f8af6992 100644
|
||||
--- a/src/c2.c
|
||||
+++ b/src/c2.c
|
||||
@@ -17,15 +17,8 @@
|
||||
|
||||
// libpcre
|
||||
#ifdef CONFIG_REGEX_PCRE
|
||||
-#include <pcre.h>
|
||||
-
|
||||
-// For compatibility with <libpcre-8.20
|
||||
-#ifndef PCRE_STUDY_JIT_COMPILE
|
||||
-#define PCRE_STUDY_JIT_COMPILE 0
|
||||
-#define LPCRE_FREE_STUDY(extra) pcre_free(extra)
|
||||
-#else
|
||||
-#define LPCRE_FREE_STUDY(extra) pcre_free_study(extra)
|
||||
-#endif
|
||||
+#define PCRE2_CODE_UNIT_WIDTH 8
|
||||
+#include <pcre2.h>
|
||||
|
||||
#endif
|
||||
|
||||
@@ -89,49 +82,52 @@ struct _c2_b {
|
||||
/// Structure for leaf element in a window condition
|
||||
struct _c2_l {
|
||||
bool neg : 1;
|
||||
- enum { C2_L_OEXISTS,
|
||||
- C2_L_OEQ,
|
||||
- C2_L_OGT,
|
||||
- C2_L_OGTEQ,
|
||||
- C2_L_OLT,
|
||||
- C2_L_OLTEQ,
|
||||
+ enum {
|
||||
+ C2_L_OEXISTS,
|
||||
+ C2_L_OEQ,
|
||||
+ C2_L_OGT,
|
||||
+ C2_L_OGTEQ,
|
||||
+ C2_L_OLT,
|
||||
+ C2_L_OLTEQ,
|
||||
} op : 3;
|
||||
- enum { C2_L_MEXACT,
|
||||
- C2_L_MSTART,
|
||||
- C2_L_MCONTAINS,
|
||||
- C2_L_MWILDCARD,
|
||||
- C2_L_MPCRE,
|
||||
+ enum {
|
||||
+ C2_L_MEXACT,
|
||||
+ C2_L_MSTART,
|
||||
+ C2_L_MCONTAINS,
|
||||
+ C2_L_MWILDCARD,
|
||||
+ C2_L_MPCRE,
|
||||
} match : 3;
|
||||
bool match_ignorecase : 1;
|
||||
char *tgt;
|
||||
xcb_atom_t tgtatom;
|
||||
bool tgt_onframe;
|
||||
int index;
|
||||
- enum { C2_L_PUNDEFINED = -1,
|
||||
- C2_L_PID = 0,
|
||||
- C2_L_PX,
|
||||
- C2_L_PY,
|
||||
- C2_L_PX2,
|
||||
- C2_L_PY2,
|
||||
- C2_L_PWIDTH,
|
||||
- C2_L_PHEIGHT,
|
||||
- C2_L_PWIDTHB,
|
||||
- C2_L_PHEIGHTB,
|
||||
- C2_L_PBDW,
|
||||
- C2_L_PFULLSCREEN,
|
||||
- C2_L_POVREDIR,
|
||||
- C2_L_PARGB,
|
||||
- C2_L_PFOCUSED,
|
||||
- C2_L_PWMWIN,
|
||||
- C2_L_PBSHAPED,
|
||||
- C2_L_PROUNDED,
|
||||
- C2_L_PCLIENT,
|
||||
- C2_L_PWINDOWTYPE,
|
||||
- C2_L_PLEADER,
|
||||
- C2_L_PNAME,
|
||||
- C2_L_PCLASSG,
|
||||
- C2_L_PCLASSI,
|
||||
- C2_L_PROLE,
|
||||
+ enum {
|
||||
+ C2_L_PUNDEFINED = -1,
|
||||
+ C2_L_PID = 0,
|
||||
+ C2_L_PX,
|
||||
+ C2_L_PY,
|
||||
+ C2_L_PX2,
|
||||
+ C2_L_PY2,
|
||||
+ C2_L_PWIDTH,
|
||||
+ C2_L_PHEIGHT,
|
||||
+ C2_L_PWIDTHB,
|
||||
+ C2_L_PHEIGHTB,
|
||||
+ C2_L_PBDW,
|
||||
+ C2_L_PFULLSCREEN,
|
||||
+ C2_L_POVREDIR,
|
||||
+ C2_L_PARGB,
|
||||
+ C2_L_PFOCUSED,
|
||||
+ C2_L_PWMWIN,
|
||||
+ C2_L_PBSHAPED,
|
||||
+ C2_L_PROUNDED,
|
||||
+ C2_L_PCLIENT,
|
||||
+ C2_L_PWINDOWTYPE,
|
||||
+ C2_L_PLEADER,
|
||||
+ C2_L_PNAME,
|
||||
+ C2_L_PCLASSG,
|
||||
+ C2_L_PCLASSI,
|
||||
+ C2_L_PROLE,
|
||||
} predef;
|
||||
enum c2_l_type {
|
||||
C2_L_TUNDEFINED,
|
||||
@@ -142,15 +138,16 @@ struct _c2_l {
|
||||
C2_L_TDRAWABLE,
|
||||
} type;
|
||||
int format;
|
||||
- enum { C2_L_PTUNDEFINED,
|
||||
- C2_L_PTSTRING,
|
||||
- C2_L_PTINT,
|
||||
+ enum {
|
||||
+ C2_L_PTUNDEFINED,
|
||||
+ C2_L_PTSTRING,
|
||||
+ C2_L_PTINT,
|
||||
} ptntype;
|
||||
char *ptnstr;
|
||||
long ptnint;
|
||||
#ifdef CONFIG_REGEX_PCRE
|
||||
- pcre *regex_pcre;
|
||||
- pcre_extra *regex_pcre_extra;
|
||||
+ pcre2_code *regex_pcre;
|
||||
+ pcre2_match_data *regex_pcre_match;
|
||||
#endif
|
||||
};
|
||||
|
||||
@@ -1059,32 +1056,31 @@ static bool c2_l_postprocess(session_t *ps, c2_l_t *pleaf) {
|
||||
// PCRE patterns
|
||||
if (C2_L_PTSTRING == pleaf->ptntype && C2_L_MPCRE == pleaf->match) {
|
||||
#ifdef CONFIG_REGEX_PCRE
|
||||
- const char *error = NULL;
|
||||
- int erroffset = 0;
|
||||
- int options = 0;
|
||||
+ int errorcode = 0;
|
||||
+ PCRE2_SIZE erroffset = 0;
|
||||
+ unsigned int options = 0;
|
||||
|
||||
// Ignore case flag
|
||||
- if (pleaf->match_ignorecase)
|
||||
- options |= PCRE_CASELESS;
|
||||
+ if (pleaf->match_ignorecase) {
|
||||
+ options |= PCRE2_CASELESS;
|
||||
+ }
|
||||
|
||||
// Compile PCRE expression
|
||||
pleaf->regex_pcre =
|
||||
- pcre_compile(pleaf->ptnstr, options, &error, &erroffset, NULL);
|
||||
- if (!pleaf->regex_pcre) {
|
||||
- log_error("Pattern \"%s\": PCRE regular expression parsing "
|
||||
+ pcre2_compile((PCRE2_SPTR)pleaf->ptnstr, PCRE2_ZERO_TERMINATED,
|
||||
+ options, &errorcode, &erroffset, NULL);
|
||||
+ if (pleaf->regex_pcre == NULL) {
|
||||
+ PCRE2_UCHAR buffer[256];
|
||||
+ pcre2_get_error_message(errorcode, buffer, sizeof(buffer));
|
||||
+ log_error("Pattern \"%s\": PCRE regular expression "
|
||||
+ "parsing "
|
||||
"failed on "
|
||||
- "offset %d: %s",
|
||||
- pleaf->ptnstr, erroffset, error);
|
||||
+ "offset %zu: %s",
|
||||
+ pleaf->ptnstr, erroffset, buffer);
|
||||
return false;
|
||||
}
|
||||
-#ifdef CONFIG_REGEX_PCRE_JIT
|
||||
- pleaf->regex_pcre_extra =
|
||||
- pcre_study(pleaf->regex_pcre, PCRE_STUDY_JIT_COMPILE, &error);
|
||||
- if (!pleaf->regex_pcre_extra) {
|
||||
- printf("Pattern \"%s\": PCRE regular expression study failed: %s",
|
||||
- pleaf->ptnstr, error);
|
||||
- }
|
||||
-#endif
|
||||
+ pleaf->regex_pcre_match =
|
||||
+ pcre2_match_data_create_from_pattern(pleaf->regex_pcre, NULL);
|
||||
|
||||
// Free the target string
|
||||
// free(pleaf->tgt);
|
||||
@@ -1102,16 +1098,18 @@ static bool c2_tree_postprocess(session_t *ps, c2_ptr_t node) {
|
||||
if (!node.isbranch) {
|
||||
return c2_l_postprocess(ps, node.l);
|
||||
}
|
||||
- if (!c2_tree_postprocess(ps, node.b->opr1))
|
||||
+ if (!c2_tree_postprocess(ps, node.b->opr1)) {
|
||||
return false;
|
||||
+ }
|
||||
return c2_tree_postprocess(ps, node.b->opr2);
|
||||
}
|
||||
|
||||
bool c2_list_postprocess(session_t *ps, c2_lptr_t *list) {
|
||||
c2_lptr_t *head = list;
|
||||
while (head) {
|
||||
- if (!c2_tree_postprocess(ps, head->ptr))
|
||||
+ if (!c2_tree_postprocess(ps, head->ptr)) {
|
||||
return false;
|
||||
+ }
|
||||
head = head->next;
|
||||
}
|
||||
return true;
|
||||
@@ -1124,8 +1122,9 @@ static void c2_free(c2_ptr_t p) {
|
||||
if (p.isbranch) {
|
||||
c2_b_t *const pbranch = p.b;
|
||||
|
||||
- if (!pbranch)
|
||||
+ if (!pbranch) {
|
||||
return;
|
||||
+ }
|
||||
|
||||
c2_free(pbranch->opr1);
|
||||
c2_free(pbranch->opr2);
|
||||
@@ -1135,14 +1134,15 @@ static void c2_free(c2_ptr_t p) {
|
||||
else {
|
||||
c2_l_t *const pleaf = p.l;
|
||||
|
||||
- if (!pleaf)
|
||||
+ if (!pleaf) {
|
||||
return;
|
||||
+ }
|
||||
|
||||
free(pleaf->tgt);
|
||||
free(pleaf->ptnstr);
|
||||
#ifdef CONFIG_REGEX_PCRE
|
||||
- pcre_free(pleaf->regex_pcre);
|
||||
- LPCRE_FREE_STUDY(pleaf->regex_pcre_extra);
|
||||
+ pcre2_code_free(pleaf->regex_pcre);
|
||||
+ pcre2_match_data_free(pleaf->regex_pcre_match);
|
||||
#endif
|
||||
free(pleaf);
|
||||
}
|
||||
@@ -1550,9 +1550,10 @@ static inline void c2_match_once_leaf(session_t *ps, const struct managed_win *w
|
||||
case C2_L_MPCRE:
|
||||
#ifdef CONFIG_REGEX_PCRE
|
||||
assert(strlen(tgt) <= INT_MAX);
|
||||
- res = (pcre_exec(pleaf->regex_pcre,
|
||||
- pleaf->regex_pcre_extra, tgt,
|
||||
- (int)strlen(tgt), 0, 0, NULL, 0) >= 0);
|
||||
+ assert(pleaf->regex_pcre);
|
||||
+ res = (pcre2_match(pleaf->regex_pcre, (PCRE2_SPTR)tgt,
|
||||
+ strlen(tgt), 0, 0,
|
||||
+ pleaf->regex_pcre_match, NULL) > 0);
|
||||
#else
|
||||
assert(0);
|
||||
#endif
|
||||
diff --git a/src/meson.build b/src/meson.build
|
||||
index 60d83a89..09eb07b6 100644
|
||||
--- a/src/meson.build
|
||||
+++ b/src/meson.build
|
||||
@@ -44,11 +44,8 @@ if get_option('config_file')
|
||||
srcs += [ 'config_libconfig.c' ]
|
||||
endif
|
||||
if get_option('regex')
|
||||
- pcre = dependency('libpcre', required: true)
|
||||
+ pcre = dependency('libpcre2-8', required: true)
|
||||
cflags += ['-DCONFIG_REGEX_PCRE']
|
||||
- if pcre.version().version_compare('>=8.20')
|
||||
- cflags += ['-DCONFIG_REGEX_PCRE_JIT']
|
||||
- endif
|
||||
deps += [pcre]
|
||||
endif
|
||||
|
|
@ -12,6 +12,11 @@ License: MPLv2.0 and MIT
|
|||
URL: https://github.com/yshui/picom
|
||||
Source0: %{url}/archive/v%{version}/%{name}-%{version}.tar.gz
|
||||
|
||||
# FR: Please port pcre dependency to pcre2. Pcre has been deprecated.
|
||||
# https://fedoraproject.org/wiki/PcreDeprecation
|
||||
# https://github.com/yshui/picom/issues/895
|
||||
Patch0: https://github.com/yshui/picom/pull/937.patch#/c2:-replace-pcre-with-pcre2.patch
|
||||
|
||||
BuildRequires: asciidoc
|
||||
BuildRequires: desktop-file-utils
|
||||
BuildRequires: gcc
|
||||
|
@ -23,7 +28,7 @@ BuildRequires: pkgconfig(dbus-1)
|
|||
BuildRequires: pkgconfig(egl)
|
||||
BuildRequires: pkgconfig(gl)
|
||||
BuildRequires: pkgconfig(libconfig)
|
||||
BuildRequires: pkgconfig(libpcre)
|
||||
BuildRequires: pkgconfig(libpcre2-8)
|
||||
BuildRequires: pkgconfig(libxdg-basedir)
|
||||
BuildRequires: pkgconfig(pixman-1)
|
||||
BuildRequires: pkgconfig(x11)
|
||||
|
|
Loading…
Reference in New Issue