Skip to contentSkip to conversation

package/nginx: fix build failure

PATCH New Samuel Martin 3 participants 5 messages Oct 12, 2014 – Oct 25, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/nginx: fix build failure

Correctly set NGX_SYS_NERR, even when it is 0.

Fixes:
http://autobuild.buildroot.org/results/666/666d780bf61e10c9f89f765bb04ddc1afc11d0d1/
http://autobuild.buildroot.org/?reason=nginx-1.6.2&&arch=powerpc

---
 .../nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
package/nginx/nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch +2 -1
index 4c00755..d02de68 100644
@@ -22,7 +22,7 @@ new file mode 100644
 index 0000000..25e7c22
 --- /dev/null
 +++ b/auto/os/sys_nerr
-@@ -0,0 +1,77 @@
+@@ -0,0 +1,78 @@
 +
 +# Copyright (C) Samuel Martin <[email protected]>
 +
@@ -89,6 +89,7 @@ index 0000000..25e7c22
 +
 +if test -z $ngx_sys_nerr ; then
 +    ngx_size=0
++    ngx_sys_nerr=0
 +fi
 +
 +cat << END >> $NGX_AUTO_CONFIG_H
Signature · 1 line
2.1.2
#1 lore raw

PATCH 1/1 package/nginx: fix build failure

Dear Samuel Martin,

package/nginx/nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch

Quoted 15 lines · Samuel Martin

Correctly set NGX_SYS_NERR, even when it is 0.

Fixes:
http://autobuild.buildroot.org/results/666/666d780bf61e10c9f89f765bb04ddc1afc11d0d1/
http://autobuild.buildroot.org/?reason=nginx-1.6.2&&arch=powerpc

---
 .../nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
package/nginx/nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch +0 -0
index 4c00755..d02de68 100644

Description + SoB in the patch ?

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#2 lore raw

PATCH 1/1 package/nginx: fix build failure

Quoted 14 lines · Thomas Petazzoni

Dear Samuel Martin,

Quoted 11 lines · Samuel Martin

Correctly set NGX_SYS_NERR, even when it is 0.

Fixes:
http://autobuild.buildroot.org/results/666/666d780bf61e10c9f89f765bb04ddc1afc11d0d1/
http://autobuild.buildroot.org/?reason=nginx-1.6.2&&arch=powerpc

---
 .../nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Quoted 6 lines

package/nginx/nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch

Quoted 4 lines
package/nginx/nginx-0005-auto-unix-make-sys_nerr-guessing-cross-friendly.patch +0 -0
index 4c00755..d02de68 100644

Description + SoB in the patch ?

Well, this is a change on an existing patch which already contains a description and my SoB...
I don't think it is needed in this case.

BTW, I did not rename the patch(es) because I prefer doing this in another cset.

Regards,

Signature · 1 line
Samuel
#3 lore raw

PATCH 1/1 package/nginx: fix build failure

Dear Samuel Martin,

Quoted 5 lines · Samuel Martin
Quoted 1 line

Description + SoB in the patch ?

Well, this is a change on an existing patch which already contains a description and my SoB...
I don't think it is needed in this case.

Ah, yes, ok, fair enough. Would have been good to have a better commit message then that makes it explicit.

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#4 lore raw

PATCH 1/1 package/nginx: fix build failure

In reply to Samuel Martin
Quoted 8 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Samuel" == Samuel Martin <[email protected]> writes:

Correctly set NGX_SYS_NERR, even when it is 0.
Fixes:
http://autobuild.buildroot.org/results/666/666d780bf61e10c9f89f765bb04ddc1afc11d0d1/
http://autobuild.buildroot.org/?reason=nginx-1.6.2&&arch=powerpc

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard