public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Jeroen Roovers" <jer@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: net-libs/nodejs/files/
Date: Tue, 21 Nov 2017 12:01:59 +0000 (UTC)	[thread overview]
Message-ID: <1511265716.d904f25f8d73ab12e9a0a6e33da052d9f5f8a1ed.jer@gentoo> (raw)

commit:     d904f25f8d73ab12e9a0a6e33da052d9f5f8a1ed
Author:     Jeroen Roovers <jer <AT> gentoo <DOT> org>
AuthorDate: Tue Nov 21 12:00:09 2017 +0000
Commit:     Jeroen Roovers <jer <AT> gentoo <DOT> org>
CommitDate: Tue Nov 21 12:01:56 2017 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d904f25f

net-libs/nodejs: Use upstream patches for shared nghttp2 support.

Package-Manager: Portage-2.3.14, Repoman-2.3.6

 .../nodejs/files/nodejs-8.9.0-shared-nghttp2.patch | 95 ++++++++++------------
 .../nodejs/files/nodejs-9.2.0-shared-nghttp2.patch | 75 +++++++++--------
 2 files changed, 88 insertions(+), 82 deletions(-)

diff --git a/net-libs/nodejs/files/nodejs-8.9.0-shared-nghttp2.patch b/net-libs/nodejs/files/nodejs-8.9.0-shared-nghttp2.patch
index dfccb1b0aea..c5e0d837f30 100644
--- a/net-libs/nodejs/files/nodejs-8.9.0-shared-nghttp2.patch
+++ b/net-libs/nodejs/files/nodejs-8.9.0-shared-nghttp2.patch
@@ -1,8 +1,10 @@
+diff --git a/configure b/configure
+index 5703a70cb2d..3b6fc7347db 100755
 --- a/configure
 +++ b/configure
-@@ -182,6 +182,27 @@
-     dest='shared_http_parser_libpath',
-     help='a directory to search for the shared http_parser DLL')
+@@ -220,6 +220,27 @@ shared_optgroup.add_option('--shared-libuv-libpath',
+     dest='shared_libuv_libpath',
+     help='a directory to search for the shared libuv DLL')
  
 +shared_optgroup.add_option('--shared-nghttp2',
 +    action='store_true',
@@ -23,48 +25,40 @@
 +shared_optgroup.add_option('--shared-nghttp2-libpath',
 +    action='store',
 +    dest='shared_nghttp2_libpath',
-+    help='a directory to search for the shared nghttp2 DLL')
++    help='a directory to search for the shared nghttp2 DLLs')
 +
- shared_optgroup.add_option('--shared-libuv',
+ shared_optgroup.add_option('--shared-openssl',
      action='store_true',
-     dest='shared_libuv',
-@@ -1360,6 +1381,7 @@
- configure_node(output)
- configure_library('zlib', output)
+     dest='shared_openssl',
+@@ -1415,6 +1436,7 @@ configure_library('zlib', output)
  configure_library('http_parser', output)
-+configure_library('nghttp2', output)
  configure_library('libuv', output)
  configure_library('libcares', output)
++configure_library('nghttp2', output)
  # stay backwards compatible with shared cares builds
+ output['variables']['node_shared_cares'] = \
+     output['variables'].pop('node_shared_libcares')
+diff --git a/node.gyp b/node.gyp
+index 86a4331a4ed..38241302e26 100644
 --- a/node.gyp
 +++ b/node.gyp
-@@ -14,6 +14,7 @@
-     'node_module_version%': '',
-     'node_shared_zlib%': 'false',
+@@ -16,6 +16,7 @@
      'node_shared_http_parser%': 'false',
-+    'node_shared_nghttp2%': 'false',
      'node_shared_cares%': 'false',
      'node_shared_libuv%': 'false',
++    'node_shared_nghttp2%': 'false',
      'node_use_openssl%': 'true',
-@@ -151,8 +152,15 @@
-       'type': '<(node_target_type)',
+     'node_shared_openssl%': 'false',
+     'node_v8_options%': '',
+@@ -177,7 +178,6 @@
  
        'dependencies': [
--        'node_js2c#host',
+         'node_js2c#host',
 -        'deps/nghttp2/nghttp2.gyp:nghttp2'
-+        'node_js2c#host'
-+      ],
-+
-+      'conditions': [
-+        [ 'node_shared_nghttp2=="false"', {
-+          'dependencies': [
-+            'deps/nghttp2/nghttp2.gyp:nghttp2'
-+          ]
-+        }]
        ],
  
        'includes': [
-@@ -180,8 +180,7 @@
+@@ -187,8 +187,7 @@
        'include_dirs': [
          'src',
          'tools/msvs/genfiles',
@@ -74,32 +68,33 @@
        ],
  
        'sources': [
-@@ -691,6 +696,15 @@
-                 'deps/http_parser/http_parser.gyp:http_parser'
-               ]
-             }],
-+            [ 'node_shared_nghttp2=="false"', {
-+              'dependencies': [
-+                'deps/nghttp2/nghttp2.gyp:nghttp2'
-+              ],
-+              'defines': [
-+                # We're using the nghttp2 static lib
-+                'NGHTTP2_STATICLIB'
-+              ]
-+            }],
-             [ 'node_shared_libuv=="false"', {
-               'dependencies': [
-                 'deps/uv/uv.gyp:libuv'
+@@ -930,6 +929,14 @@
+             'deps/uv/uv.gyp:libuv'
+           ]
+         }],
++        [ 'node_shared_nghttp2=="false"', {
++          'dependencies': [
++            'deps/nghttp2/nghttp2.gyp:nghttp2'
++          ],
++          'include_dirs': [
++            'deps/nghttp2/lib/includes'
++          ]
++        }],
+         [ 'node_use_v8_platform=="true"', {
+           'dependencies': [
+             'deps/v8/src/v8.gyp:v8_libplatform',
+diff --git a/node.gypi b/node.gypi
+index ec78df2a339..3990c59ef98 100644
 --- a/node.gypi
 +++ b/node.gypi
-@@ -251,6 +251,10 @@
-       'dependencies': [ 'deps/http_parser/http_parser.gyp:http_parser' ],
+@@ -259,6 +259,10 @@
+     [ 'node_shared_libuv=="false"', {
+       'dependencies': [ 'deps/uv/uv.gyp:libuv' ],
      }],
- 
++
 +    [ 'node_shared_nghttp2=="false"', {
 +      'dependencies': [ 'deps/nghttp2/nghttp2.gyp:nghttp2' ],
 +    }],
-+
-     [ 'node_shared_cares=="false"', {
-       'dependencies': [ 'deps/cares/cares.gyp:cares' ],
-     }],
+ 
+     [ 'OS=="win"', {
+       'sources': [

diff --git a/net-libs/nodejs/files/nodejs-9.2.0-shared-nghttp2.patch b/net-libs/nodejs/files/nodejs-9.2.0-shared-nghttp2.patch
index a145c0c7116..e8483ab279f 100644
--- a/net-libs/nodejs/files/nodejs-9.2.0-shared-nghttp2.patch
+++ b/net-libs/nodejs/files/nodejs-9.2.0-shared-nghttp2.patch
@@ -1,8 +1,10 @@
+diff --git a/configure b/configure
+index 5703a70cb2d..3b6fc7347db 100755
 --- a/configure
 +++ b/configure
-@@ -182,6 +182,27 @@
-     dest='shared_http_parser_libpath',
-     help='a directory to search for the shared http_parser DLL')
+@@ -220,6 +220,27 @@ shared_optgroup.add_option('--shared-libuv-libpath',
+     dest='shared_libuv_libpath',
+     help='a directory to search for the shared libuv DLL')
  
 +shared_optgroup.add_option('--shared-nghttp2',
 +    action='store_true',
@@ -23,48 +25,40 @@
 +shared_optgroup.add_option('--shared-nghttp2-libpath',
 +    action='store',
 +    dest='shared_nghttp2_libpath',
-+    help='a directory to search for the shared nghttp2 DLL')
++    help='a directory to search for the shared nghttp2 DLLs')
 +
- shared_optgroup.add_option('--shared-libuv',
+ shared_optgroup.add_option('--shared-openssl',
      action='store_true',
-     dest='shared_libuv',
-@@ -1360,6 +1381,7 @@
- configure_node(output)
- configure_library('zlib', output)
+     dest='shared_openssl',
+@@ -1415,6 +1436,7 @@ configure_library('zlib', output)
  configure_library('http_parser', output)
-+configure_library('nghttp2', output)
  configure_library('libuv', output)
  configure_library('libcares', output)
++configure_library('nghttp2', output)
  # stay backwards compatible with shared cares builds
+ output['variables']['node_shared_cares'] = \
+     output['variables'].pop('node_shared_libcares')
+diff --git a/node.gyp b/node.gyp
+index 43a6576206d..c8ba6759246 100644
 --- a/node.gyp
 +++ b/node.gyp
-@@ -14,6 +14,7 @@
-     'node_module_version%': '',
-     'node_shared_zlib%': 'false',
+@@ -16,6 +16,7 @@
      'node_shared_http_parser%': 'false',
-+    'node_shared_nghttp2%': 'false',
      'node_shared_cares%': 'false',
      'node_shared_libuv%': 'false',
++    'node_shared_nghttp2%': 'false',
      'node_use_openssl%': 'true',
-@@ -151,8 +152,15 @@
-       'type': '<(node_target_type)',
+     'node_shared_openssl%': 'false',
+     'node_v8_options%': '',
+@@ -176,7 +177,6 @@
  
        'dependencies': [
--        'node_js2c#host',
+         'node_js2c#host',
 -        'deps/nghttp2/nghttp2.gyp:nghttp2'
-+        'node_js2c#host'
-+      ],
-+
-+      'conditions': [
-+        [ 'node_shared_nghttp2=="false"', {
-+          'dependencies': [
-+            'deps/nghttp2/nghttp2.gyp:nghttp2'
-+          ]
-+        }]
        ],
  
        'includes': [
-@@ -180,8 +180,7 @@
+@@ -186,8 +186,7 @@
        'include_dirs': [
          'src',
          'tools/msvs/genfiles',
@@ -74,16 +68,33 @@
        ],
  
        'sources': [
+@@ -929,6 +928,14 @@
+             'deps/uv/uv.gyp:libuv'
+           ]
+         }],
++        [ 'node_shared_nghttp2=="false"', {
++          'dependencies': [
++            'deps/nghttp2/nghttp2.gyp:nghttp2'
++          ],
++          'include_dirs': [
++            'deps/nghttp2/lib/includes'
++          ]
++        }],
+         [ 'node_use_v8_platform=="true"', {
+           'dependencies': [
+             'deps/v8/src/v8.gyp:v8_libplatform',
+diff --git a/node.gypi b/node.gypi
+index ec78df2a339..3990c59ef98 100644
 --- a/node.gypi
 +++ b/node.gypi
-@@ -251,6 +251,10 @@
-       'dependencies': [ 'deps/http_parser/http_parser.gyp:http_parser' ],
+@@ -133,6 +133,10 @@
+       'dependencies': [ 'deps/uv/uv.gyp:libuv' ],
      }],
  
 +    [ 'node_shared_nghttp2=="false"', {
 +      'dependencies': [ 'deps/nghttp2/nghttp2.gyp:nghttp2' ],
 +    }],
 +
-     [ 'node_shared_cares=="false"', {
-       'dependencies': [ 'deps/cares/cares.gyp:cares' ],
-     }],
+     [ 'OS=="mac"', {
+       # linking Corefoundation is needed since certain OSX debugging tools
+       # like Instruments require it for some features


             reply	other threads:[~2017-11-21 12:02 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-21 12:01 Jeroen Roovers [this message]
  -- strict thread matches above, loose matches on Subject: below --
2024-08-28  7:25 [gentoo-commits] repo/gentoo:master commit in: net-libs/nodejs/files/ Joonas Niilola
2024-05-30 14:55 Arthur Zamarin
2024-04-18 19:32 Conrad Kostecki
2022-07-19 17:10 Conrad Kostecki
2022-07-18 21:41 Conrad Kostecki
2022-06-05 23:03 Conrad Kostecki
2022-05-30  3:58 Sam James
2022-05-13 20:04 Sam James
2022-05-06  5:21 Joonas Niilola
2021-05-16 18:17 Marek Szuba
2021-05-16 18:17 Marek Szuba
2020-02-23 19:54 Magnus Granberg
2018-12-13 12:44 Jeroen Roovers
2018-01-04 12:31 Jeroen Roovers
2017-11-01 11:42 Jeroen Roovers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1511265716.d904f25f8d73ab12e9a0a6e33da052d9f5f8a1ed.jer@gentoo \
    --to=jer@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox