Message ID | 1448958904-12009-1-git-send-email-root@renaud.io |
---|---|
State | Superseded |
Headers | show |
Hi Renaud AUBIN, On Tue, Dec 01, 2015 at 09:35:04AM +0100, Renaud AUBIN wrote: > The http-parser repository has been transfered from joyent to nodejs. > Bump version to 2.6.0. > > Signed-off-by: Renaud AUBIN <root@renaud.io> [...] > -From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001 > -From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > -Date: Fri, 28 Aug 2015 19:02:11 +0200 > +From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001 > +From: Renaud AUBIN <root@renaud.io> > +Date: Tue, 1 Dec 2015 09:24:41 +0100 Please do not change the patch authorship. > 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 <thomas.petazzoni@free-electrons.com> Also, please don't remove existing sign-off. You may add your sign-off here when you update the patch. baruch
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 <baruch@tkos.co.il> a écrit : > > Hi Renaud AUBIN, > > On Tue, Dec 01, 2015 at 09:35:04AM +0100, Renaud AUBIN wrote: > > The http-parser repository has been transfered from joyent to nodejs. > > Bump version to 2.6.0. > > > > Signed-off-by: Renaud AUBIN <root@renaud.io> > > [...] > > > -From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001 > > -From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > > -Date: Fri, 28 Aug 2015 19:02:11 +0200 > > +From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001 > > +From: Renaud AUBIN <root@renaud.io> > > +Date: Tue, 1 Dec 2015 09:24:41 +0100 > > Please do not change the patch authorship. > > > 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 <thomas.petazzoni@free-electrons.com> > > Also, please don't remove existing sign-off. You may add your sign-off here > when you update the patch. > > baruch > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open Systems > =}------------------------------------------------ooO--U--Ooo------------{= > - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
On Di, 2015-12-01 at 13:21 +0100, Renaud AUBIN wrote: > 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
On 01-12-15 21:20, Jörg Krause wrote: > On Di, 2015-12-01 at 13:21 +0100, Renaud AUBIN wrote: >> 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 > > Best regards > Jörg Krause > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot >
diff --git a/package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch b/package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch index 29fe04d..336a9e0 100644 --- a/package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch +++ b/package/libhttpparser/0001-Use-f-option-when-calling-ln-at-install-time.patch @@ -1,34 +1,32 @@ -From d9dfeca8736318eaad5cd4554cbae059469decde Mon Sep 17 00:00:00 2001 -From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> -Date: Fri, 28 Aug 2015 19:02:11 +0200 +From fb2d2803bdad4847b3116a325a9ae0a56f906a08 Mon Sep 17 00:00:00 2001 +From: Renaud AUBIN <root@renaud.io> +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 <thomas.petazzoni@free-electrons.com> --- 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 diff --git a/package/libhttpparser/Config.in b/package/libhttpparser/Config.in index 31b8b9b..4356304 100644 --- a/package/libhttpparser/Config.in +++ b/package/libhttpparser/Config.in @@ -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 diff --git a/package/libhttpparser/libhttpparser.mk b/package/libhttpparser/libhttpparser.mk index d6ec236..015efc0 100644 --- a/package/libhttpparser/libhttpparser.mk +++ b/package/libhttpparser/libhttpparser.mk @@ -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
The http-parser repository has been transfered from joyent to nodejs. Bump version to 2.6.0. Signed-off-by: Renaud AUBIN <root@renaud.io> --- ...-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(-)