Improvements to Debian rules file
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
chromium-browser (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Hi, I help maintain the Chromium package in Endless OS, which is based on Ubuntu's package. We have two minor improvements to the Debian rules file that you probably want. These diffs are based on your Xenial v56 package. First, a simple typo:
From b794c427fa8146e
From: Michael Catanzaro <email address hidden>
Date: Mon, 2 Jan 2017 18:39:20 -0600
Subject: [PATCH] Fix upstream typo in rules file
So now we know armhf has not been tested....
---
debian/rules | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/debian/rules b/debian/rules
index 881dd1a..3baa6d2 100755
--- a/debian/rules
+++ b/debian/rules
@@ -127,7 +127,7 @@ else ifeq (armhf,
common_defines += target_cpu="arm"
#common_defines += arm_optionally_
common_defines += arm_use_neon=false
-common_defined += arm_float_
+common_defines += arm_float_
else ifeq (arm64,
common_defines += target_cpu="arm64"
else ifeq (i386,$
Second we have a patch to remove some environment variables that are ignored by the new build system:
From 3d7d37f64a31f4d
From: Michael Catanzaro <email address hidden>
Date: Sun, 8 Jan 2017 14:37:03 -0600
Subject: [PATCH] Remove unused code from rules file
All of these environment variables are ignored by the GN build, so it is
misleading to keep them here.
Also, remove the incorrect comment immediately below, since it prefaces
a list of features that are both enabled and disabled.
---
debian/rules | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/debian/rules b/debian/rules
index 773b78e..33b7374 100755
--- a/debian/rules
+++ b/debian/rules
@@ -62,17 +62,6 @@ DEB_DH_SCOUR_ARGS += -Xsvg
DEB_DH_
DEB_DH_
-### linker flags to avoid memory allocation issues on 32-bit
-export LDFLAGS+
-
-# See https:/
-#export CFLAGS+
-
-export CC := $(shell which gcc-4 gcc-5 gcc cc |head -1)
-export CXX := $(shell which g++-4 g+++5 g++ c++ |head -1)
-export CPP := $(shell which g++-4 g+++5 g++ c++ |head -1)
-
-# disabled features
common_defines+=\
enable_
enable_
tags: | added: patch |
Thanks!
On Wed, Mar 15, 2017 at 6:39 PM, Michael Catanzaro <email address hidden>
wrote:
> Public bug reported: 8f3eb38f593c0d6 e0d42692c7 Mon Sep 17 00:00:00 2001 $(DEB_HOST_ ARCH)) use_neon= false # implies Android in abi="hard" arm_use_thumb=true abi="hard" arm_use_thumb=true $(DEB_HOST_ ARCH)) (DEB_HOST_ ARCH)) ad7e43bf78269a7 14bae1ed42 Mon Sep 17 00:00:00 2001 SHLIBDEPS_ ARGS_chromium- codecs- ffmpeg := -ldebian/ chromium- codecs- SHLIBDEPS_ ARGS_chromium- codecs- ffmpeg- extra := chromium- codecs- ffmpeg- extra/$ (LIB_DIR) =-Wl,-- no-keep- memory -Wl,--stats /bugs.chromium. org/p/v8/ issues/ detail? id=3782 =-fno-delete- null-pointer- checks google_ now=false \ hangout_ services_ extension= true \ /bugs.launchpad .net/bugs/ 1673276
>
> Hi, I help maintain the Chromium package in Endless OS, which is based
> on Ubuntu's package. We have two minor improvements to the Debian rules
> file that you probably want. These diffs are based on your Xenial v56
> package. First, a simple typo:
>
> >From b794c427fa8146e
> From: Michael Catanzaro <email address hidden>
> Date: Mon, 2 Jan 2017 18:39:20 -0600
> Subject: [PATCH] Fix upstream typo in rules file
>
> So now we know armhf has not been tested....
> ---
> debian/rules | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/debian/rules b/debian/rules
> index 881dd1a..3baa6d2 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -127,7 +127,7 @@ else ifeq (armhf,
> common_defines += target_cpu="arm"
> #common_defines += arm_optionally_
> 2016-10
> common_defines += arm_use_neon=false
> -common_defined += arm_float_
> +common_defines += arm_float_
> else ifeq (arm64,
> common_defines += target_cpu="arm64"
> else ifeq (i386,$
>
>
> Second we have a patch to remove some environment variables that are
> ignored by the new build system:
>
> >From 3d7d37f64a31f4d
> From: Michael Catanzaro <email address hidden>
> Date: Sun, 8 Jan 2017 14:37:03 -0600
> Subject: [PATCH] Remove unused code from rules file
>
> All of these environment variables are ignored by the GN build, so it is
> misleading to keep them here.
>
> Also, remove the incorrect comment immediately below, since it prefaces
> a list of features that are both enabled and disabled.
> ---
> debian/rules | 11 -----------
> 1 file changed, 11 deletions(-)
>
> diff --git a/debian/rules b/debian/rules
> index 773b78e..33b7374 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -62,17 +62,6 @@ DEB_DH_SCOUR_ARGS += -Xsvg
> DEB_DH_
> ffmpeg/$(LIB_DIR)
> DEB_DH_
> -ldebian/
>
> -### linker flags to avoid memory allocation issues on 32-bit
> -export LDFLAGS+
> -
> -# See https:/
> -#export CFLAGS+
> -
> -export CC := $(shell which gcc-4 gcc-5 gcc cc |head -1)
> -export CXX := $(shell which g++-4 g+++5 g++ c++ |head -1)
> -export CPP := $(shell which g++-4 g+++5 g++ c++ |head -1)
> -
> -# disabled features
> common_defines+=\
> enable_
> enable_
>
> ** Affects: chromium-browser (Ubuntu)
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are subscribed to
> chromium-browser in Ubuntu.
> Matching subscriptions: Cr
> https:/
>
> Title:
> Improvements to Debian rules file
>
> Status in chromiu...