On Tue, Sep 09, 2014 at 01:10:18PM -0400, Brad Smith wrote:
> Any Chromium users out there able to take this for a spin? This
> re-enables the use of stack protector with Chromium. Lightly tested
> on amd64 and especially needs testing with i386.

An updated diff for Chromium 37.


Index: Makefile
===================================================================
RCS file: /home/cvs/ports/www/chromium/Makefile,v
retrieving revision 1.182
diff -u -p -u -p -r1.182 Makefile
--- Makefile    11 Sep 2014 19:04:52 -0000      1.182
+++ Makefile    11 Sep 2014 19:43:54 -0000
@@ -7,7 +7,7 @@ COMMENT=                Chromium browser
 
 V=                     37.0.2062.120
 DISTNAME=              chromium-${V}
-
+REVISION=              0
 CATEGORIES=            www
 
 HOMEPAGE=              https://code.google.com/chromium/
Index: patches/patch-build_common_gypi
===================================================================
RCS file: /home/cvs/ports/www/chromium/patches/patch-build_common_gypi,v
retrieving revision 1.25
diff -u -p -u -p -r1.25 patch-build_common_gypi
--- patches/patch-build_common_gypi     11 Sep 2014 19:04:52 -0000      1.25
+++ patches/patch-build_common_gypi     11 Sep 2014 19:55:23 -0000
@@ -1,6 +1,6 @@
 $OpenBSD: patch-build_common_gypi,v 1.25 2014/09/11 19:04:52 robert Exp $
---- build/common.gypi.orig.port        Wed Sep 10 01:47:11 2014
-+++ build/common.gypi  Wed Sep 10 22:28:34 2014
+--- build/common.gypi.orig.port        Tue Sep  9 19:47:11 2014
++++ build/common.gypi  Thu Sep 11 15:52:37 2014
 @@ -87,7 +87,7 @@
  
            'conditions': [
@@ -19,22 +19,7 @@ $OpenBSD: patch-build_common_gypi,v 1.25
            'test_isolation_mode%': 'check',
          }, {
            'test_isolation_mode%': 'noop',
-@@ -3311,6 +3311,14 @@
-         ],
-       },
-     }],
-+    ['OS=="openbsd"', {
-+      'target_defaults': {
-+        'cflags': [
-+          '-fno-stack-protector',
-+          '-Wno-unknown-warning-option',
-+        ],
-+      },
-+    }],
-     ['os_posix==1 and OS!="mac" and OS!="ios"', {
-       'target_defaults': {
-         # Enable -Werror by default, but put it in a variable so it can
-@@ -5385,8 +5393,8 @@
+@@ -5385,8 +5385,8 @@
      ['clang==1 and ((OS!="mac" and OS!="ios") or clang_xcode==0) '
          'and OS!="win"', {
        'make_global_settings': [

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

Reply via email to