Gentoo Archives: gentoo-commits

From: Jeroen Roovers <jer@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: net-libs/http-parser/, net-libs/http-parser/files/
Date: Thu, 06 Dec 2018 09:55:48
Message-Id: 1544090139.6f5ac8dee85a2732348fe84c790b3646741d0c24.jer@gentoo
1 commit: 6f5ac8dee85a2732348fe84c790b3646741d0c24
2 Author: Jeroen Roovers <jer <AT> gentoo <DOT> org>
3 AuthorDate: Thu Dec 6 09:55:14 2018 +0000
4 Commit: Jeroen Roovers <jer <AT> gentoo <DOT> org>
5 CommitDate: Thu Dec 6 09:55:39 2018 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6f5ac8de
7
8 net-libs/http-parser: Old
9
10 Package-Manager: Portage-2.3.52, Repoman-2.3.12
11 Signed-off-by: Jeroen Roovers <jer <AT> gentoo.org>
12
13 net-libs/http-parser/Manifest | 2 -
14 .../files/0001-makefile-fix-DESTDIR-usage.patch | 64 ----------------------
15 .../files/0002-makefile-quote-variables.patch | 50 -----------------
16 ...x-SONAME-symlink-it-should-not-be-a-full-.patch | 35 ------------
17 ...04-makefile-add-CFLAGS-to-linking-command.patch | 31 -----------
18 ...0005-makefile-fix-install-rule-dependency.patch | 33 -----------
19 .../files/http-parser-2.6.2-darwin.patch | 16 ------
20 net-libs/http-parser/http-parser-2.6.2.ebuild | 45 ---------------
21 net-libs/http-parser/http-parser-2.8.0.ebuild | 35 ------------
22 9 files changed, 311 deletions(-)
23
24 diff --git a/net-libs/http-parser/Manifest b/net-libs/http-parser/Manifest
25 index 1f9ae37e62f..8557aad9d1f 100644
26 --- a/net-libs/http-parser/Manifest
27 +++ b/net-libs/http-parser/Manifest
28 @@ -1,3 +1 @@
29 -DIST http-parser-2.6.2.tar.gz 48292 BLAKE2B 054ffa66960bad29a31e1a697a035447eb551fb85ed2af3786664f161d8d5a62aad8142ce859b7f1af85489b703b188d09980bf82b56cb190c21ecbe9c3a2f5c SHA512 e19e5377b3eb7f149c428196826fb878564fdfa3716ff6df5a3845c51586aee0582e252e09d1f8ebad1163b3e66632ff0c6e78f6acb2f0da20d7a06e734406c1
30 -DIST http-parser-2.8.0.tar.gz 50575 BLAKE2B 0ea1973b452b62915ad2a92be95e09281c1aa2f152ff4a67580e99ab113b44544f98b2e33fa1fadb259144975f15dc69c8b0ecc9de59e603dbd0e0a0785b7976 SHA512 57252a29b75f91452fd9bf2d3805bf1ddd95c32aee864cbe47dc676861512b9db95a7b3a4e1b999d5c046ab0d50809e7f184994c24ad48fe97cc2bcc27d9eb7b
31 DIST http-parser-2.8.1.tar.gz 50731 BLAKE2B 1c4f3f61550f7f8e86d7b47e3986754a108dc979c6232d87ac418f9f8295b88ee43c4a5af150460f707e6ade01f5d435801f980ccbff93369ee0a9dc2eb60656 SHA512 6f52f543d979f39688ccefae236527a8183929b3d30f5370570107b01cf89d0338b448249a81102b78d31615d2e8f6e7c708f8961f55ece08e7d3a40e5ad0883
32
33 diff --git a/net-libs/http-parser/files/0001-makefile-fix-DESTDIR-usage.patch b/net-libs/http-parser/files/0001-makefile-fix-DESTDIR-usage.patch
34 deleted file mode 100644
35 index a819cf441d3..00000000000
36 --- a/net-libs/http-parser/files/0001-makefile-fix-DESTDIR-usage.patch
37 +++ /dev/null
38 @@ -1,64 +0,0 @@
39 -From 7fbc87986baa09c342abb21e34613e8bbdc3c9c7 Mon Sep 17 00:00:00 2001
40 -From: hasufell <hasufell@××××××××.de>
41 -Date: Mon, 2 Nov 2015 16:24:43 +0100
42 -Subject: [PATCH 1/4] makefile: fix DESTDIR usage
43 -
44 -DESTDIR is not supposed to be set inside other variables. It is
45 -standard to have this variable in install/uninstall rules, so it
46 -can be reliably set separately no matter what other variables are set
47 -to.
48 -This also avoids potential bugs with setting SONAME or seds on
49 -installed files (like pkgconfig) which then might include the
50 -temporary DESTDIR directory.
51 -
52 -DESTDIR is really just for installing into a temporary directory or
53 -a chroot, mostly used by package managers.
54 ----
55 - Makefile | 20 ++++++++++----------
56 - 1 file changed, 10 insertions(+), 10 deletions(-)
57 -
58 -diff --git a/Makefile b/Makefile
59 -index 33c8ba0..76153a0 100644
60 ---- a/Makefile
61 -+++ b/Makefile
62 -@@ -55,7 +55,7 @@ CFLAGS_LIB = $(CFLAGS_FAST) -fPIC
63 - LDFLAGS_LIB = $(LDFLAGS) -shared
64 -
65 - INSTALL ?= install
66 --PREFIX ?= $(DESTDIR)/usr/local
67 -+PREFIX ?= /usr/local
68 - LIBDIR = $(PREFIX)/lib
69 - INCLUDEDIR = $(PREFIX)/include
70 -
71 -@@ -123,19 +123,19 @@ tags: http_parser.c http_parser.h test.c
72 - ctags $^
73 -
74 - install: library
75 -- $(INSTALL) -D http_parser.h $(INCLUDEDIR)/http_parser.h
76 -- $(INSTALL) -D $(SONAME) $(LIBDIR)/$(SONAME)
77 -- ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
78 -+ $(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
79 -+ $(INSTALL) -D $(SONAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
80 -+ ln -s $(LIBDIR)/$(SONAME) $(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)
81 -
82 - install-strip: library
83 -- $(INSTALL) -D http_parser.h $(INCLUDEDIR)/http_parser.h
84 -- $(INSTALL) -D -s $(SONAME) $(LIBDIR)/$(SONAME)
85 -- ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
86 -+ $(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
87 -+ $(INSTALL) -D -s $(SONAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
88 -+ ln -s $(LIBDIR)/$(SONAME) $(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)
89 -
90 - uninstall:
91 -- rm $(INCLUDEDIR)/http_parser.h
92 -- rm $(LIBDIR)/$(SONAME)
93 -- rm $(LIBDIR)/libhttp_parser.so
94 -+ rm $(DESTDIR)$(INCLUDEDIR)/http_parser.h
95 -+ rm $(DESTDIR)$(LIBDIR)/$(SONAME)
96 -+ rm $(DESTDIR)$(LIBDIR)/libhttp_parser.so
97 -
98 - clean:
99 - rm -f *.o *.a tags test test_fast test_g \
100 ---
101 -2.6.1
102 -
103
104 diff --git a/net-libs/http-parser/files/0002-makefile-quote-variables.patch b/net-libs/http-parser/files/0002-makefile-quote-variables.patch
105 deleted file mode 100644
106 index 096ac6b6bd6..00000000000
107 --- a/net-libs/http-parser/files/0002-makefile-quote-variables.patch
108 +++ /dev/null
109 @@ -1,50 +0,0 @@
110 -From 9bce473ba7417b45bfdb59d4151a8857dcfff4ad Mon Sep 17 00:00:00 2001
111 -From: hasufell <hasufell@××××××××.de>
112 -Date: Mon, 2 Nov 2015 16:27:06 +0100
113 -Subject: [PATCH 2/4] makefile: quote variables
114 -
115 -Make does not take care of this in make rules. If any of the variables
116 -DESTDIR, INCLUDEDIR or LIBDIR contain whitespaces, then the related
117 -install command will fail.
118 -
119 -This is even more important for the uninstall rule.
120 ----
121 - Makefile | 18 +++++++++---------
122 - 1 file changed, 9 insertions(+), 9 deletions(-)
123 -
124 -diff --git a/Makefile b/Makefile
125 -index 76153a0..8c4a9d7 100644
126 ---- a/Makefile
127 -+++ b/Makefile
128 -@@ -123,19 +123,19 @@ tags: http_parser.c http_parser.h test.c
129 - ctags $^
130 -
131 - install: library
132 -- $(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
133 -- $(INSTALL) -D $(SONAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
134 -- ln -s $(LIBDIR)/$(SONAME) $(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)
135 -+ $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
136 -+ $(INSTALL) -D $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
137 -+ ln -s $(LIBDIR)/$(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
138 -
139 - install-strip: library
140 -- $(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
141 -- $(INSTALL) -D -s $(SONAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
142 -- ln -s $(LIBDIR)/$(SONAME) $(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)
143 -+ $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
144 -+ $(INSTALL) -D -s $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
145 -+ ln -s $(LIBDIR)/$(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
146 -
147 - uninstall:
148 -- rm $(DESTDIR)$(INCLUDEDIR)/http_parser.h
149 -- rm $(DESTDIR)$(LIBDIR)/$(SONAME)
150 -- rm $(DESTDIR)$(LIBDIR)/libhttp_parser.so
151 -+ rm "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
152 -+ rm "$(DESTDIR)$(LIBDIR)/$(SONAME)"
153 -+ rm "$(DESTDIR)$(LIBDIR)/libhttp_parser.so"
154 -
155 - clean:
156 - rm -f *.o *.a tags test test_fast test_g \
157 ---
158 -2.6.1
159 -
160
161 diff --git a/net-libs/http-parser/files/0003-makefile-fix-SONAME-symlink-it-should-not-be-a-full-.patch b/net-libs/http-parser/files/0003-makefile-fix-SONAME-symlink-it-should-not-be-a-full-.patch
162 deleted file mode 100644
163 index cafe68d220c..00000000000
164 --- a/net-libs/http-parser/files/0003-makefile-fix-SONAME-symlink-it-should-not-be-a-full-.patch
165 +++ /dev/null
166 @@ -1,35 +0,0 @@
167 -From f45b38c42e7e92a5d0215c44dcf306616536011e Mon Sep 17 00:00:00 2001
168 -From: hasufell <hasufell@××××××××.de>
169 -Date: Mon, 2 Nov 2015 16:32:11 +0100
170 -Subject: [PATCH 3/4] makefile: fix SONAME symlink, it should not be a full
171 - path
172 -
173 -The symlink destination being a full path doesn't give any benefit and
174 -may break a few use cases of copying these files to a different
175 -destination, while preserving the symlink.
176 ----
177 - Makefile | 4 ++--
178 - 1 file changed, 2 insertions(+), 2 deletions(-)
179 -
180 -diff --git a/Makefile b/Makefile
181 -index 8c4a9d7..cbe93e8 100644
182 ---- a/Makefile
183 -+++ b/Makefile
184 -@@ -125,12 +125,12 @@ tags: http_parser.c http_parser.h test.c
185 - install: library
186 - $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
187 - $(INSTALL) -D $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
188 -- ln -s $(LIBDIR)/$(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
189 -+ ln -s $(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
190 -
191 - install-strip: library
192 - $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
193 - $(INSTALL) -D -s $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
194 -- ln -s $(LIBDIR)/$(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
195 -+ ln -s $(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
196 -
197 - uninstall:
198 - rm "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
199 ---
200 -2.6.1
201 -
202
203 diff --git a/net-libs/http-parser/files/0004-makefile-add-CFLAGS-to-linking-command.patch b/net-libs/http-parser/files/0004-makefile-add-CFLAGS-to-linking-command.patch
204 deleted file mode 100644
205 index e21cd715499..00000000000
206 --- a/net-libs/http-parser/files/0004-makefile-add-CFLAGS-to-linking-command.patch
207 +++ /dev/null
208 @@ -1,31 +0,0 @@
209 -From 62b1450cfe2e0df2d912279d38edf1b916020101 Mon Sep 17 00:00:00 2001
210 -From: hasufell <hasufell@××××××××.de>
211 -Date: Mon, 2 Nov 2015 16:39:31 +0100
212 -Subject: [PATCH 4/4] makefile: add CFLAGS to linking command
213 -
214 -Although we compile the objects explicitly there are some CFLAGS
215 -that may also affect linking, which is not always obvious.
216 -
217 -This can also be a problem for toolchains that support multiple ABIs
218 -and need to set CFLAGS=<abi selector>, which will cause linking
219 -to either fail or produce an unusable executable/library.
220 ----
221 - Makefile | 2 +-
222 - 1 file changed, 1 insertion(+), 1 deletion(-)
223 -
224 -diff --git a/Makefile b/Makefile
225 -index cbe93e8..ae16f08 100644
226 ---- a/Makefile
227 -+++ b/Makefile
228 -@@ -102,7 +102,7 @@ libhttp_parser.o: http_parser.c http_parser.h Makefile
229 - $(CC) $(CPPFLAGS_FAST) $(CFLAGS_LIB) -c http_parser.c -o libhttp_parser.o
230 -
231 - library: libhttp_parser.o
232 -- $(CC) $(LDFLAGS_LIB) -o $(SONAME) $<
233 -+ $(CC) $(CFLAGS_LIB) $(LDFLAGS_LIB) -o $(SONAME) $<
234 -
235 - package: http_parser.o
236 - $(AR) rcs libhttp_parser.a http_parser.o
237 ---
238 -2.6.1
239 -
240
241 diff --git a/net-libs/http-parser/files/0005-makefile-fix-install-rule-dependency.patch b/net-libs/http-parser/files/0005-makefile-fix-install-rule-dependency.patch
242 deleted file mode 100644
243 index da966373063..00000000000
244 --- a/net-libs/http-parser/files/0005-makefile-fix-install-rule-dependency.patch
245 +++ /dev/null
246 @@ -1,33 +0,0 @@
247 -From b67bfbe6a07529dd82e2ee83b6848d017e6e422f Mon Sep 17 00:00:00 2001
248 -From: hasufell <hasufell@××××××××.de>
249 -Date: Mon, 2 Nov 2015 16:51:28 +0100
250 -Subject: [PATCH 5/5] makefile: fix install rule dependency
251 -
252 -Otherwise the install rule will recompile the library, no matter
253 -if it has already been compiled.
254 ----
255 - Makefile | 4 ++--
256 - 1 file changed, 2 insertions(+), 2 deletions(-)
257 -
258 -diff --git a/Makefile b/Makefile
259 -index ae16f08..df0b59f 100644
260 ---- a/Makefile
261 -+++ b/Makefile
262 -@@ -122,12 +122,12 @@ parsertrace_g: http_parser_g.o contrib/parsertrace.c
263 - tags: http_parser.c http_parser.h test.c
264 - ctags $^
265 -
266 --install: library
267 -+install: $(SONAME)
268 - $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
269 - $(INSTALL) -D $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
270 - ln -s $(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
271 -
272 --install-strip: library
273 -+install-strip: $(SONAME)
274 - $(INSTALL) -D http_parser.h "$(DESTDIR)$(INCLUDEDIR)/http_parser.h"
275 - $(INSTALL) -D -s $(SONAME) "$(DESTDIR)$(LIBDIR)/$(SONAME)"
276 - ln -s $(SONAME) "$(DESTDIR)$(LIBDIR)/libhttp_parser.$(SOEXT)"
277 ---
278 -2.6.1
279 -
280
281 diff --git a/net-libs/http-parser/files/http-parser-2.6.2-darwin.patch b/net-libs/http-parser/files/http-parser-2.6.2-darwin.patch
282 deleted file mode 100644
283 index b04b4a32bb0..00000000000
284 --- a/net-libs/http-parser/files/http-parser-2.6.2-darwin.patch
285 +++ /dev/null
286 @@ -1,16 +0,0 @@
287 -Makefile: set install_name on Darwin
288 -
289 -https://bugs.gentoo.org/show_bug.cgi?id=546098
290 -https://github.com/nodejs/http-parser/issues/356
291 -
292 ---- http-parser-2.6.2/Makefile
293 -+++ http-parser-2.6.2/Makefile
294 -@@ -62,6 +62,8 @@
295 - ifneq (darwin,$(PLATFORM))
296 - # TODO(bnoordhuis) The native SunOS linker expects -h rather than -soname...
297 - LDFLAGS_LIB += -Wl,-soname=$(SONAME)
298 -+else
299 -+LDFLAGS_LIB += -Wl,-install_name,$(LIBDIR)/$(SONAME)
300 - endif
301 -
302 - test: test_g test_fast
303
304 diff --git a/net-libs/http-parser/http-parser-2.6.2.ebuild b/net-libs/http-parser/http-parser-2.6.2.ebuild
305 deleted file mode 100644
306 index 33986a95474..00000000000
307 --- a/net-libs/http-parser/http-parser-2.6.2.ebuild
308 +++ /dev/null
309 @@ -1,45 +0,0 @@
310 -# Copyright 1999-2017 Gentoo Foundation
311 -# Distributed under the terms of the GNU General Public License v2
312 -
313 -EAPI=5
314 -
315 -inherit eutils toolchain-funcs multilib multilib-minimal
316 -
317 -DESCRIPTION="Http request/response parser for C"
318 -HOMEPAGE="https://github.com/nodejs/http-parser"
319 -SRC_URI="https://github.com/nodejs/http-parser/archive/v${PV}.tar.gz -> ${P}.tar.gz"
320 -
321 -LICENSE="MIT"
322 -SLOT="0/${PV}"
323 -KEYWORDS="amd64 arm ~arm64 ppc ppc64 x86 ~amd64-linux ~x64-macos ~x64-solaris"
324 -IUSE="static-libs"
325 -
326 -# https://github.com/nodejs/http-parser/pull/272
327 -PATCHES=(
328 - "${FILESDIR}"/0001-makefile-fix-DESTDIR-usage.patch
329 - "${FILESDIR}"/0002-makefile-quote-variables.patch
330 - "${FILESDIR}"/0003-makefile-fix-SONAME-symlink-it-should-not-be-a-full-.patch
331 - "${FILESDIR}"/0004-makefile-add-CFLAGS-to-linking-command.patch
332 - "${FILESDIR}"/0005-makefile-fix-install-rule-dependency.patch
333 - "${FILESDIR}"/${PN}-2.6.2-darwin.patch
334 -)
335 -
336 -src_prepare() {
337 - tc-export CC AR
338 - epatch "${PATCHES[@]}"
339 - multilib_copy_sources
340 -}
341 -
342 -multilib_src_compile() {
343 - emake PREFIX="${EPREFIX}/usr" LIBDIR="${EPREFIX}/usr/$(get_libdir)" CFLAGS_FAST="${CFLAGS}" library
344 - use static-libs && emake CFLAGS_FAST="${CFLAGS}" package
345 -}
346 -
347 -multilib_src_test() {
348 - emake CFLAGS_DEBUG="${CFLAGS}" test
349 -}
350 -
351 -multilib_src_install() {
352 - emake DESTDIR="${D}" PREFIX="${EPREFIX}/usr" LIBDIR="${EPREFIX}/usr/$(get_libdir)" install
353 - use static-libs && dolib.a libhttp_parser.a
354 -}
355
356 diff --git a/net-libs/http-parser/http-parser-2.8.0.ebuild b/net-libs/http-parser/http-parser-2.8.0.ebuild
357 deleted file mode 100644
358 index c4563e4bf41..00000000000
359 --- a/net-libs/http-parser/http-parser-2.8.0.ebuild
360 +++ /dev/null
361 @@ -1,35 +0,0 @@
362 -# Copyright 1999-2018 Gentoo Foundation
363 -# Distributed under the terms of the GNU General Public License v2
364 -
365 -EAPI=6
366 -
367 -inherit toolchain-funcs multilib-minimal
368 -
369 -DESCRIPTION="HTTP request/response parser for C"
370 -HOMEPAGE="https://github.com/nodejs/http-parser"
371 -SRC_URI="https://github.com/nodejs/http-parser/archive/v${PV}.tar.gz -> ${P}.tar.gz"
372 -
373 -LICENSE="MIT"
374 -SLOT="0/${PV}"
375 -KEYWORDS="~amd64 ~arm ~arm64 ~ppc ~ppc64 ~x86 ~amd64-linux ~x64-macos ~x64-solaris"
376 -IUSE="static-libs"
377 -
378 -src_prepare() {
379 - default
380 - tc-export CC AR
381 - multilib_copy_sources
382 -}
383 -
384 -multilib_src_compile() {
385 - emake PREFIX="${EPREFIX}/usr" LIBDIR="${EPREFIX}/usr/$(get_libdir)" CFLAGS_FAST="${CFLAGS}" library
386 - use static-libs && emake CFLAGS_FAST="${CFLAGS}" package
387 -}
388 -
389 -multilib_src_test() {
390 - emake CFLAGS_DEBUG="${CFLAGS}" CFLAGS_FAST="${CFLAGS}" test
391 -}
392 -
393 -multilib_src_install() {
394 - emake DESTDIR="${D}" PREFIX="${EPREFIX}/usr" LIBDIR="${EPREFIX}/usr/$(get_libdir)" install
395 - use static-libs && dolib.a libhttp_parser.a
396 -}