diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-21 15:15:19 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-23 22:35:01 +0000 |
commit | 1229097db5cb7fa5790727d5af024dc0d0ccd342 (patch) | |
tree | d1f2c1e92efc157ecbddcfd7322e0afed6e15c0e | |
parent | c79cc7dcbec13e10a2a6750bbcd96142eb073829 (diff) | |
download | poky-1229097db5cb7fa5790727d5af024dc0d0ccd342.tar.gz |
libpcre: Drop old/stale patch
According to my tests this incorrect symbols resolution at runtime no
longer happens. Ubuntu is still carrying the patch but also probably
doesn't need to, they are also on a much older version. It sounds
like there was once a linkage bug somewhere which has likely been
resolved since.
Drop the patch as it doesn't seem needed anymore. If it were a real
issue it should be submitted upstream too, the status is incorrect.
(From OE-Core rev: 02f36ed515afed550dfcd986977ce2106dee556a)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/recipes-support/libpcre/libpcre/fix-pcre-name-collision.patch | 41 | ||||
-rw-r--r-- | meta/recipes-support/libpcre/libpcre_8.44.bb | 1 |
2 files changed, 0 insertions, 42 deletions
diff --git a/meta/recipes-support/libpcre/libpcre/fix-pcre-name-collision.patch b/meta/recipes-support/libpcre/libpcre/fix-pcre-name-collision.patch deleted file mode 100644 index 89b44f6aa6..0000000000 --- a/meta/recipes-support/libpcre/libpcre/fix-pcre-name-collision.patch +++ /dev/null | |||
@@ -1,41 +0,0 @@ | |||
1 | Upstream-Status: Inappropriate [debian patch] | ||
2 | |||
3 | This patch address a namespace collision with libc. | ||
4 | |||
5 | Although there is no "#include <regex.h>" in the source file, at | ||
6 | runtime, it's unintentionally linked to the libc version, the regcomp of | ||
7 | libc is called instead the pcre one using pcre's data structure... | ||
8 | that looks like a disaster. | ||
9 | |||
10 | Can patch is from Debian (and Ubuntu 11.04alpha has it also). | ||
11 | |||
12 | [sgw: added patch comment] | ||
13 | Signed-off-by: Qing He <qing.he@intel.com> | ||
14 | Signed-off-by: Saul Wold <sgw@linux.intel.com> | ||
15 | |||
16 | --- a/pcreposix.h 2010-05-17 00:17:23.000000000 +0800 | ||
17 | +++ b/pcreposix.h 2009-01-15 04:32:17.000000000 +0800 | ||
18 | @@ -133,14 +130,19 @@ | ||
19 | |||
20 | /* The functions */ | ||
21 | |||
22 | -PCREPOSIX_EXP_DECL int regcomp(regex_t *, const char *, int); | ||
23 | -PCREPOSIX_EXP_DECL int regexec(const regex_t *, const char *, size_t, | ||
24 | +PCREPOSIX_EXP_DECL int pcreposix_regcomp(regex_t *, const char *, int); | ||
25 | +PCREPOSIX_EXP_DECL int pcreposix_regexec(const regex_t *, const char *, size_t, | ||
26 | regmatch_t *, int); | ||
27 | -PCREPOSIX_EXP_DECL size_t regerror(int, const regex_t *, char *, size_t); | ||
28 | -PCREPOSIX_EXP_DECL void regfree(regex_t *); | ||
29 | +PCREPOSIX_EXP_DECL size_t pcreposix_regerror(int, const regex_t *, char *, size_t); | ||
30 | +PCREPOSIX_EXP_DECL void pcreposix_regfree(regex_t *); | ||
31 | |||
32 | #ifdef __cplusplus | ||
33 | } /* extern "C" */ | ||
34 | #endif | ||
35 | |||
36 | +#define regcomp pcreposix_regcomp | ||
37 | +#define regexec pcreposix_regexec | ||
38 | +#define regerror pcreposix_regerror | ||
39 | +#define regfree pcreposix_regfree | ||
40 | + | ||
41 | #endif /* End of pcreposix.h */ | ||
diff --git a/meta/recipes-support/libpcre/libpcre_8.44.bb b/meta/recipes-support/libpcre/libpcre_8.44.bb index e5471e81da..cd80dc7345 100644 --- a/meta/recipes-support/libpcre/libpcre_8.44.bb +++ b/meta/recipes-support/libpcre/libpcre_8.44.bb | |||
@@ -8,7 +8,6 @@ SECTION = "devel" | |||
8 | LICENSE = "BSD-3-Clause" | 8 | LICENSE = "BSD-3-Clause" |
9 | LIC_FILES_CHKSUM = "file://LICENCE;md5=3bb381a66a5385b246d4877922e7511e" | 9 | LIC_FILES_CHKSUM = "file://LICENCE;md5=3bb381a66a5385b246d4877922e7511e" |
10 | SRC_URI = "https://ftp.pcre.org/pub/pcre/pcre-${PV}.tar.bz2 \ | 10 | SRC_URI = "https://ftp.pcre.org/pub/pcre/pcre-${PV}.tar.bz2 \ |
11 | file://fix-pcre-name-collision.patch \ | ||
12 | file://run-ptest \ | 11 | file://run-ptest \ |
13 | file://Makefile \ | 12 | file://Makefile \ |
14 | " | 13 | " |