Skip to contentSkip to conversation

Update the libhttpparser package

PATCH New Renaud AUBIN 4 participants 5 messages Dec 1, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Update the libhttpparser package

The http-parser repository has been transfered from joyent to nodejs. Bump version to 2.6.0.

---
 ...-f-option-when-calling-ln-at-install-time.patch | 22 ++++++++++------------
 package/libhttpparser/Config.in                    |  2 +-
 package/libhttpparser/libhttpparser.mk             |  4 ++--
 3 files changed, 13 insertions(+), 15 deletions(-)
package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch +10 -12
index 29fe04d..336a9e0 100644
@@ -1,34 +1,32 @@
-From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001
-From: Thomas Petazzoni <[email protected]>
-Date: Fri, 28 Aug 2015 19:02:11 +0200
+From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001
+From: Renaud AUBIN <[email protected]>
+Date: Tue, 1 Dec 2015 09:24:41 +0100
 Subject: [PATCH] Use -f option when calling ln at install time
 
 This allows "make install; make install" to work properly.
-
-Signed-off-by: Thomas Petazzoni <[email protected]>
 ---
  Makefile | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/Makefile b/Makefile
-index 373709c..6206b09 100644
+index 33c8ba0..b36d95a 100644
 --- a/Makefile
 +++ b/Makefile
-@@ -113,12 +113,12 @@ tags: http_parser.c http_parser.h test.c
+@@ -125,12 +125,12 @@ tags: http_parser.c http_parser.h test.c
  install: library
  	$(INSTALL) -D  http_parser.h $(INCLUDEDIR)/http_parser.h
  	$(INSTALL) -D $(SONAME) $(LIBDIR)/$(SONAME)
--	ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.so
-+	ln -sf $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.so
+-	ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
++	ln -sf $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
  
  install-strip: library
  	$(INSTALL) -D  http_parser.h $(INCLUDEDIR)/http_parser.h
  	$(INSTALL) -D -s $(SONAME) $(LIBDIR)/$(SONAME)
--	ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.so
-+	ln -sf $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.so
+-	ln -s $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
++	ln -sf $(LIBDIR)/$(SONAME) $(LIBDIR)/libhttp_parser.$(SOEXT)
  
  uninstall:
  	rm $(INCLUDEDIR)/http_parser.h
 -- 
-2.5.0
+2.6.2
 
package/libhttpparser/Config.in +1 -1
index 31b8b9b..4356304 100644
@@ -11,7 +11,7 @@ config BR2_PACKAGE_LIBHTTPPARSER
 	  40 bytes of data per message stream (in a web server
 	  that is per connection).
 
-	  https://github.com/joyent/http-parser
+	  https://github.com/nodejs/http-parser
 
 comment "libhttpparser needs a toolchain w/ dynamic library"
 	depends on BR2_STATIC_LIBS
package/libhttpparser/libhttpparser.mk +2 -2
index d6ec236..015efc0 100644
@@ -4,8 +4,8 @@
 #
 ################################################################################
 
-LIBHTTPPARSER_VERSION = v2.5.0
-LIBHTTPPARSER_SITE = $(call github,joyent,http-parser,$(LIBHTTPPARSER_VERSION))
+LIBHTTPPARSER_VERSION = v2.6.0
+LIBHTTPPARSER_SITE = $(call github,nodejs,http-parser,$(LIBHTTPPARSER_VERSION))
 LIBHTTPPARSER_INSTALL_STAGING = YES
 LIBHTTPPARSER_LICENSE = MIT
 LIBHTTPPARSER_LICENSE_FILES = LICENSE-MIT
Signature · 1 line
2.6.2
#1 lore raw

PATCH 1/1 Update the libhttpparser package

Hi Renaud AUBIN,

Quoted 4 lines · Renaud AUBIN

The http-parser repository has been transfered from joyent to nodejs. Bump version to 2.6.0.

[...]

package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch line 3

Quoted 6 lines
package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch +3 -3
-From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001
-From: Thomas Petazzoni <[email protected]>
-Date: Fri, 28 Aug 2015 19:02:11 +0200
+From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001
+From: Renaud AUBIN <[email protected]>
+Date: Tue, 1 Dec 2015 09:24:41 +0100

Please do not change the patch authorship.

package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch line 8

Quoted 5 lines

Subject: [PATCH] Use -f option when calling ln at install time

This allows "make install; make install" to work properly.

package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch +0 -2
-
-Signed-off-by: Thomas Petazzoni <[email protected]>

Also, please don't remove existing sign-off. You may add your sign-off here when you update the patch.

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
#2 lore raw

PATCH 1/1 Update the libhttpparser package

I have resubmitted by figuring out by myself the ad hoc git am processing?
Maybe the Contributing chapter could be slightly improved by a section on packages' patching policies?

Thank you for your review.

Le?1 d?c. 2015 11:10 AM, Baruch Siach <[email protected]> a ?crit?:

Quoted 39 lines

Hi Renaud AUBIN,

Quoted 4 lines · Renaud AUBIN

The http-parser repository has been transfered from joyent to nodejs. Bump version to 2.6.0.

[...]

package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch line 3

Quoted 6 lines
package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch +3 -3
-From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001 
-From: Thomas Petazzoni <[email protected]> 
-Date: Fri, 28 Aug 2015 19:02:11 +0200 
+From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001 
+From: Renaud AUBIN <[email protected]> 
+Date: Tue, 1 Dec 2015 09:24:41 +0100 

Please do not change the patch authorship.

Quoted 5 lines

? Subject: [PATCH] Use -f option when calling ln at install time ?
? This allows "make install; make install" to work properly. -
-Signed-off-by: Thomas Petazzoni <[email protected]>

Also, please don't remove existing sign-off. You may add your sign-off here when you update the patch.

baruch

Signature · 7 lines
???? http://baruch.siach.name/blog/????????????????? ~. .~?? Tk Open Systems 
=}------------------------------------------------ooO--U--Ooo------------{= 
?? - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - 
_______________________________________________ 
buildroot mailing list 
buildroot at busybox.net 
http://lists.busybox.net/mailman/listinfo/buildroot 
#3 lore raw

PATCH 1/1 Update the libhttpparser package

Quoted 7 lines · Renaud AUBIN

I have resubmitted by figuring out by myself the ad hoc git am processing?
Maybe the Contributing chapter could be slightly improved by a section on packages' patching policies?

Thank you for your review.

We have a section about patch policy in the Buildroot manual:
http://nightly.buildroot.org/#patch-policy

Best regards
J?rg Krause

#4 lore raw

PATCH 1/1 Update the libhttpparser package

Quoted 11 lines · J?rg Krause
Quoted 7 lines · Renaud AUBIN

I have resubmitted by figuring out by myself the ad hoc git am processing?
Maybe the Contributing chapter could be slightly improved by a section on packages' patching policies?

Thank you for your review.

We have a section about patch policy in the Buildroot manual:
http://nightly.buildroot.org/#patch-policy

But it doesn't say anything about what you have to do when you update a patch.
I'll try to cook something up.

Regards,
Arnout

Quoted 8 lines

Best regards
J?rg Krause
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF