~alpine/aports

1

[alpine-aports] [PATCH] testing/firefox: Update to version 55.0.1

Marian Buschsieweke <marian.buschsieweke@ovgu.de>
Details
Message ID
<20170816104447.14396-1-marian.buschsieweke@ovgu.de>
Sender timestamp
1502880287
DKIM signature
missing
Download raw message
Patch: +32 -42
fix-tools.patch: Can no longer be applied, source changed significantly
getcontext.patch: Solves the same problem fix-tools.patch solved previously
tools_profiler_missing_header.patch: Adds a missing #include (obviously this
                                     header was pulled in on glibc by an
                                     unrelated #include)
---
 testing/firefox/APKBUILD                           | 13 ++++----
 testing/firefox/fix-tools.patch                    | 36 ----------------------
 testing/firefox/getcontext.patch                   | 15 +++++++++
 .../firefox/tools_profiler_missing_header.patch    | 10 ++++++
 4 files changed, 32 insertions(+), 42 deletions(-)
 delete mode 100644 testing/firefox/fix-tools.patch
 create mode 100644 testing/firefox/getcontext.patch
 create mode 100644 testing/firefox/tools_profiler_missing_header.patch

diff --git a/testing/firefox/APKBUILD b/testing/firefox/APKBUILD
index 39ed093eeb..ebe0c03797 100644
--- a/testing/firefox/APKBUILD
+++ b/testing/firefox/APKBUILD
@@ -1,10 +1,10 @@
# Contributor: William Pitcock <nenolod@dereferenced.org>
# Maintainer: Natanael Copa <ncopa@alpinelinux.org>
pkgname=firefox
pkgver=54.0.1
pkgver=55.0.1
_pkgver=$pkgver
_xulver=$pkgver
pkgrel=2
pkgrel=0
pkgdesc="Firefox web browser"
url="http://www.firefox.com"
# limited by rust and cargo
@@ -58,7 +58,6 @@ source="https://ftp.mozilla.org/pub/firefox/releases/$pkgver/source/firefox-$pkg
	disable-hunspell_hooks.patch
	fix-seccomp-bpf.patch
	fix-toolkit.patch
	fix-tools.patch
	mallinfo.patch

	fix-fortify-system-wrappers.patch
@@ -68,6 +67,8 @@ source="https://ftp.mozilla.org/pub/firefox/releases/$pkgver/source/firefox-$pkg
	disable-moz-stackwalk.patch
	fix-rust-target.patch
	fix-bug-1261392.patch
	tools_profiler_missing_header.patch
	getcontext.patch

	firefox.desktop
	firefox-safe.desktop"
@@ -119,7 +120,6 @@ build() {
		\
		--enable-alsa \
		--enable-default-toolkit=cairo-gtk3 \
		--enable-gio \
		--enable-gold=no \
		--enable-official-branding \
		--enable-optimize="$CFLAGS" \
@@ -217,14 +217,13 @@ __EOF__
	rm -f "$pkgdir"/${_mozappdirdev}/sdk/lib/libxul.so
}

sha512sums="43607c2c0af995a21dc7f0f68b24b7e5bdb3faa5ee06025901c826bfe4d169256ea1c9eb5fcc604c4d6426ced53e80787c12fc07cda014eca09199ef3df783a2  firefox-54.0.1.source.tar.xz
sha512sums="2c15cb3e1a9f464f63ff7ac8ccf0625ed845a5cb9b186d7acf121c439cec38bcdeee93630e99dbfd336f1b7e60a7c09822a1eba59f308bba8866f155b2ed1c47  firefox-55.0.1.source.tar.xz
0b3f1e4b9fdc868e4738b5c81fd6c6128ce8885b260affcb9a65ff9d164d7232626ce1291aaea70132b3e3124f5e13fef4d39326b8e7173e362a823722a85127  stab.h
7e123144bc2b1efed149dfb41b255c447d43ea93a63ebe114d01945e6a6d69edc2f2a3c36980a93279106c1842355851b8b6c1d96679ee6be7b9b30513e0b1a8  0002-Use-C99-math-isfinite.patch
09bc32cf9ee81b9cc6bb58ddbc66e6cc5c344badff8de3435cde5848e5a451e0172153231db85c2385ff05b5d9c20760cb18e4138dfc99060a9e960de2befbd5  fix-fortify-inline.patch
0fcc647af53a3ce21c2bc36e5631eb0935e7243ebb3ab59b5719542cc54a6ac023a4a857b43b75756efb9ed80c0aecaa94dc5679a3b3792f82e87bf2c1af82e1  disable-hunspell_hooks.patch
5e7722b8899bf25f5a941980de9a4e9e9a1efcfa9ce12edb70e525e6022bc0e8ef284804be622e8d17eb3bbc1d99c1554b54cfd4d066ea639de810d30005267f  fix-seccomp-bpf.patch
a2925045154f4fd34e5fc056656f4f9da100341529e5d4104d249154db0c7863384083f421ce6e47e0f20566a8b20787fa35444c7933c03cd03f96f06dcd4532  fix-toolkit.patch
94884c11ff4b38ed33cd890c21956f956fadc9b53c78891074501388cb2c95674a143a0109cbd7cc2b7f4035d6514cd800686f56778fc3711603dc0187c1b703  fix-tools.patch
bdcd1b402d2ec94957ba5d08cbad7b1a7f59c251c311be9095208491a05abb05a956c79f27908e1f26b54a3679387b2f33a51e945b650671ad85c0a2d59a5a29  mallinfo.patch
2a967e4619fd89a046bd40be7fdcef646fcb1eba5e47afc96aa1d59c25bcc8747cb07baee6c11f3b803db0981aced181534b806778e4d21e74958f5f4d74c784  fix-fortify-system-wrappers.patch
ed0d344c66fc8e1cc83a11e9858b32c42e841cbeedd9eb9438811e9fcc3593dc824a8336d00058d55836cedc970aeadd6a82c6dcd7bc0fb746e564d8b478cc6c  fix-arm-atomics-grsec.patch
@@ -233,5 +232,7 @@ e61664bc93eadce5016a06a4d0684b34a05074f1815e88ef2613380d7b369c6fd305fb34f83b5eb1
4797d2d89ac63a57abb826b8ea9f751314ce66946194033deb9d78c2ff377b88106fd2c7bc5034dc13ad03dd5085b1893c3ccae1a9e63fde35655bb0921f7188  disable-moz-stackwalk.patch
09d73d9058fbdb0992758e3377d8d4ea0b01cd291bdfbb918ec8b7b1ffb118bba3ff71efaca6107a0510fad295dd8de3102d8933b80d3a3546d7423e707e37b2  fix-rust-target.patch
a50b412edf9573a0bd04a43578b1c927967a616b73a5995eefb15bfa78fd2bd14e36ec05315a0703f6370ecd524e6bcb012e7285beb1245e9add9b8553acb79e  fix-bug-1261392.patch
67b3587a9434e61b7b013b3dce266d516d3658f094c393b3e02278c4d6b670a7acb391b10e07dce55613338107c18958b209d769aa9cbe05e2dd9eef27f23eeb  tools_profiler_missing_header.patch
6bcd37a44e6ba89fe62108a7d96591cf2fe36ce9149ced00e9086f38ef4529c7c4feb1cf9c64693c7ddd9f0a9580a6dc67227ebfbdac925dfb46577a9241d2fa  getcontext.patch
f3b7c3e804ce04731012a46cb9e9a6b0769e3772aef9c0a4a8c7520b030fdf6cd703d5e9ff49275f14b7d738fe82a0a4fde3bc3219dff7225d5db0e274987454  firefox.desktop
5dcb6288d0444a8a471d669bbaf61cdb1433663eff38b72ee5e980843f5fc07d0d60c91627a2c1159215d0ad77ae3f115dcc5fdfe87e64ca704b641aceaa44ed  firefox-safe.desktop"
diff --git a/testing/firefox/fix-tools.patch b/testing/firefox/fix-tools.patch
deleted file mode 100644
index c7357b6f7e..0000000000
--- a/testing/firefox/fix-tools.patch
@@ -1,36 +0,0 @@
--- a/tools/profiler/core/platform-linux-android.cpp
+++ b/tools/profiler/core/platform-linux-android.cpp
@@ -715,11 +715,13 @@
 void TickSample::PopulateContext(void* aContext)
 {
   MOZ_ASSERT(aContext);
+#if defined(__GLIBC__)
   ucontext_t* pContext = reinterpret_cast<ucontext_t*>(aContext);
   if (!getcontext(pContext)) {
     context = pContext;
     SetSampleContext(this,
                      reinterpret_cast<ucontext_t*>(aContext)->uc_mcontext);
   }
+#endif
 }
 
 void OS::SleepMicro(int microseconds)
diff --git a/tools/profiler/lul/LulElf.cpp b/tools/profiler/lul/LulElf.cpp
index a922137..d01dde1 100644
--- a/tools/profiler/lul/LulElf.cpp
+++ b/tools/profiler/lul/LulElf.cpp
@@ -483,10 +483,10 @@ string FormatIdentifier(unsigned char identifier[16]) {
 // Return the non-directory portion of FILENAME: the portion after the
 // last slash, or the whole filename if there are no slashes.
 string BaseFileName(const string &filename) {
-  // Lots of copies!  basename's behavior is less than ideal.
-  char *c_filename = strdup(filename.c_str());
-  string base = basename(c_filename);
-  free(c_filename);
+  // basename's behavior is less than ideal so avoid it
+  const char *c_filename = filename.c_str();
+  const char *p = strrchr(c_filename, '/');
+  string base = p ? p+1 : c_filename;
   return base;
 }
 
diff --git a/testing/firefox/getcontext.patch b/testing/firefox/getcontext.patch
new file mode 100644
index 0000000000..a6e933ba1a
--- /dev/null
+++ b/testing/firefox/getcontext.patch
@@ -0,0 +1,15 @@
--- old/tools/profiler/core/platform-linux-android.cpp
+++ new/tools/profiler/core/platform-linux-android.cpp
@@ -505,8 +505,12 @@
   MOZ_ASSERT(mIsSynchronous);
   MOZ_ASSERT(aContext);
 
+#ifdef __GLIBC__
   if (!getcontext(aContext)) {
+#endif
     FillInSample(*this, aContext);
+#ifdef __GLIBC__
   }
+#endif
 }
 
diff --git a/testing/firefox/tools_profiler_missing_header.patch b/testing/firefox/tools_profiler_missing_header.patch
new file mode 100644
index 0000000000..406dabdf27
--- /dev/null
+++ b/testing/firefox/tools_profiler_missing_header.patch
@@ -0,0 +1,10 @@
--- old/tools/profiler/lul/LulElf.cpp
+++ new/tools/profiler/lul/LulElf.cpp
@@ -53,6 +53,7 @@
 #include <sys/stat.h>
 #include <unistd.h>
 #include <arpa/inet.h>
+#include <libgen.h>
 
 #include <set>
 #include <string>
-- 
2.14.1



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---

[alpine-aports] Re: [PATCH] testing/firefox: Update to version 55.0.1

Marian Buschsieweke <marian.buschsieweke@ovgu.de>
Details
Message ID
<20170817212912.74821078@notebook-marian>
In-Reply-To
<20170816104447.14396-1-marian.buschsieweke@ovgu.de> (view parent)
Sender timestamp
1502998152
DKIM signature
missing
Download raw message
Hi,

sadly there is a bug causing a segfault when website supporting ICE (RFC5245)
features are used [1]. Also Version 55.0.2 has been released right now.
Therefore, I suggest to reject the patch.

Kind regards,
Marian


[1]: https://bugzilla.mozilla.org/show_bug.cgi?id=1391292

On Wed, 16 Aug 2017 12:44:47 +0200
Marian Buschsieweke <marian.buschsieweke@ovgu.de> wrote:

> fix-tools.patch: Can no longer be applied, source changed significantly
> getcontext.patch: Solves the same problem fix-tools.patch solved previously
> tools_profiler_missing_header.patch: Adds a missing #include (obviously this
>                                      header was pulled in on glibc by an
>                                      unrelated #include)
> ---
>  testing/firefox/APKBUILD                           | 13 ++++----
>  testing/firefox/fix-tools.patch                    | 36 ----------------------
>  testing/firefox/getcontext.patch                   | 15 +++++++++
>  .../firefox/tools_profiler_missing_header.patch    | 10 ++++++
>  4 files changed, 32 insertions(+), 42 deletions(-)
>  delete mode 100644 testing/firefox/fix-tools.patch
>  create mode 100644 testing/firefox/getcontext.patch
>  create mode 100644 testing/firefox/tools_profiler_missing_header.patch
> 
> diff --git a/testing/firefox/APKBUILD b/testing/firefox/APKBUILD
> index 39ed093eeb..ebe0c03797 100644
> --- a/testing/firefox/APKBUILD
> +++ b/testing/firefox/APKBUILD
> @@ -1,10 +1,10 @@
>  # Contributor: William Pitcock <nenolod@dereferenced.org>
>  # Maintainer: Natanael Copa <ncopa@alpinelinux.org>
>  pkgname=firefox
> -pkgver=54.0.1
> +pkgver=55.0.1
>  _pkgver=$pkgver
>  _xulver=$pkgver
> -pkgrel=2
> +pkgrel=0
>  pkgdesc="Firefox web browser"
>  url="http://www.firefox.com"
>  # limited by rust and cargo
> @@ -58,7 +58,6 @@ source="https://ftp.mozilla.org/pub/firefox/releases/$pkgver/source/firefox-$pkg
>  	disable-hunspell_hooks.patch
>  	fix-seccomp-bpf.patch
>  	fix-toolkit.patch
> -	fix-tools.patch
>  	mallinfo.patch
>  
>  	fix-fortify-system-wrappers.patch
> @@ -68,6 +67,8 @@ source="https://ftp.mozilla.org/pub/firefox/releases/$pkgver/source/firefox-$pkg
>  	disable-moz-stackwalk.patch
>  	fix-rust-target.patch
>  	fix-bug-1261392.patch
> +	tools_profiler_missing_header.patch
> +	getcontext.patch
>  
>  	firefox.desktop
>  	firefox-safe.desktop"
> @@ -119,7 +120,6 @@ build() {
>  		\
>  		--enable-alsa \
>  		--enable-default-toolkit=cairo-gtk3 \
> -		--enable-gio \
>  		--enable-gold=no \
>  		--enable-official-branding \
>  		--enable-optimize="$CFLAGS" \
> @@ -217,14 +217,13 @@ __EOF__
>  	rm -f "$pkgdir"/${_mozappdirdev}/sdk/lib/libxul.so
>  }
>  
> -sha512sums="43607c2c0af995a21dc7f0f68b24b7e5bdb3faa5ee06025901c826bfe4d169256ea1c9eb5fcc604c4d6426ced53e80787c12fc07cda014eca09199ef3df783a2  firefox-54.0.1.source.tar.xz
> +sha512sums="2c15cb3e1a9f464f63ff7ac8ccf0625ed845a5cb9b186d7acf121c439cec38bcdeee93630e99dbfd336f1b7e60a7c09822a1eba59f308bba8866f155b2ed1c47  firefox-55.0.1.source.tar.xz
>  0b3f1e4b9fdc868e4738b5c81fd6c6128ce8885b260affcb9a65ff9d164d7232626ce1291aaea70132b3e3124f5e13fef4d39326b8e7173e362a823722a85127  stab.h
>  7e123144bc2b1efed149dfb41b255c447d43ea93a63ebe114d01945e6a6d69edc2f2a3c36980a93279106c1842355851b8b6c1d96679ee6be7b9b30513e0b1a8  0002-Use-C99-math-isfinite.patch
>  09bc32cf9ee81b9cc6bb58ddbc66e6cc5c344badff8de3435cde5848e5a451e0172153231db85c2385ff05b5d9c20760cb18e4138dfc99060a9e960de2befbd5  fix-fortify-inline.patch
>  0fcc647af53a3ce21c2bc36e5631eb0935e7243ebb3ab59b5719542cc54a6ac023a4a857b43b75756efb9ed80c0aecaa94dc5679a3b3792f82e87bf2c1af82e1  disable-hunspell_hooks.patch
>  5e7722b8899bf25f5a941980de9a4e9e9a1efcfa9ce12edb70e525e6022bc0e8ef284804be622e8d17eb3bbc1d99c1554b54cfd4d066ea639de810d30005267f  fix-seccomp-bpf.patch
>  a2925045154f4fd34e5fc056656f4f9da100341529e5d4104d249154db0c7863384083f421ce6e47e0f20566a8b20787fa35444c7933c03cd03f96f06dcd4532  fix-toolkit.patch
> -94884c11ff4b38ed33cd890c21956f956fadc9b53c78891074501388cb2c95674a143a0109cbd7cc2b7f4035d6514cd800686f56778fc3711603dc0187c1b703  fix-tools.patch
>  bdcd1b402d2ec94957ba5d08cbad7b1a7f59c251c311be9095208491a05abb05a956c79f27908e1f26b54a3679387b2f33a51e945b650671ad85c0a2d59a5a29  mallinfo.patch
>  2a967e4619fd89a046bd40be7fdcef646fcb1eba5e47afc96aa1d59c25bcc8747cb07baee6c11f3b803db0981aced181534b806778e4d21e74958f5f4d74c784  fix-fortify-system-wrappers.patch
>  ed0d344c66fc8e1cc83a11e9858b32c42e841cbeedd9eb9438811e9fcc3593dc824a8336d00058d55836cedc970aeadd6a82c6dcd7bc0fb746e564d8b478cc6c  fix-arm-atomics-grsec.patch
> @@ -233,5 +232,7 @@ e61664bc93eadce5016a06a4d0684b34a05074f1815e88ef2613380d7b369c6fd305fb34f83b5eb1
>  4797d2d89ac63a57abb826b8ea9f751314ce66946194033deb9d78c2ff377b88106fd2c7bc5034dc13ad03dd5085b1893c3ccae1a9e63fde35655bb0921f7188  disable-moz-stackwalk.patch
>  09d73d9058fbdb0992758e3377d8d4ea0b01cd291bdfbb918ec8b7b1ffb118bba3ff71efaca6107a0510fad295dd8de3102d8933b80d3a3546d7423e707e37b2  fix-rust-target.patch
>  a50b412edf9573a0bd04a43578b1c927967a616b73a5995eefb15bfa78fd2bd14e36ec05315a0703f6370ecd524e6bcb012e7285beb1245e9add9b8553acb79e  fix-bug-1261392.patch
> +67b3587a9434e61b7b013b3dce266d516d3658f094c393b3e02278c4d6b670a7acb391b10e07dce55613338107c18958b209d769aa9cbe05e2dd9eef27f23eeb  tools_profiler_missing_header.patch
> +6bcd37a44e6ba89fe62108a7d96591cf2fe36ce9149ced00e9086f38ef4529c7c4feb1cf9c64693c7ddd9f0a9580a6dc67227ebfbdac925dfb46577a9241d2fa  getcontext.patch
>  f3b7c3e804ce04731012a46cb9e9a6b0769e3772aef9c0a4a8c7520b030fdf6cd703d5e9ff49275f14b7d738fe82a0a4fde3bc3219dff7225d5db0e274987454  firefox.desktop
>  5dcb6288d0444a8a471d669bbaf61cdb1433663eff38b72ee5e980843f5fc07d0d60c91627a2c1159215d0ad77ae3f115dcc5fdfe87e64ca704b641aceaa44ed  firefox-safe.desktop"
> diff --git a/testing/firefox/fix-tools.patch b/testing/firefox/fix-tools.patch
> deleted file mode 100644
> index c7357b6f7e..0000000000
> --- a/testing/firefox/fix-tools.patch
> +++ /dev/null
> @@ -1,36 +0,0 @@
> ---- a/tools/profiler/core/platform-linux-android.cpp
> -+++ b/tools/profiler/core/platform-linux-android.cpp
> -@@ -715,11 +715,13 @@
> - void TickSample::PopulateContext(void* aContext)
> - {
> -   MOZ_ASSERT(aContext);
> -+#if defined(__GLIBC__)
> -   ucontext_t* pContext = reinterpret_cast<ucontext_t*>(aContext);
> -   if (!getcontext(pContext)) {
> -     context = pContext;
> -     SetSampleContext(this,
> -                      reinterpret_cast<ucontext_t*>(aContext)->uc_mcontext);
> -   }
> -+#endif
> - }
> - 
> - void OS::SleepMicro(int microseconds)
> -diff --git a/tools/profiler/lul/LulElf.cpp b/tools/profiler/lul/LulElf.cpp
> -index a922137..d01dde1 100644
> ---- a/tools/profiler/lul/LulElf.cpp
> -+++ b/tools/profiler/lul/LulElf.cpp
> -@@ -483,10 +483,10 @@ string FormatIdentifier(unsigned char identifier[16]) {
> - // Return the non-directory portion of FILENAME: the portion after the
> - // last slash, or the whole filename if there are no slashes.
> - string BaseFileName(const string &filename) {
> --  // Lots of copies!  basename's behavior is less than ideal.
> --  char *c_filename = strdup(filename.c_str());
> --  string base = basename(c_filename);
> --  free(c_filename);
> -+  // basename's behavior is less than ideal so avoid it
> -+  const char *c_filename = filename.c_str();
> -+  const char *p = strrchr(c_filename, '/');
> -+  string base = p ? p+1 : c_filename;
> -   return base;
> - }
> - 
> diff --git a/testing/firefox/getcontext.patch b/testing/firefox/getcontext.patch
> new file mode 100644
> index 0000000000..a6e933ba1a
> --- /dev/null
> +++ b/testing/firefox/getcontext.patch
> @@ -0,0 +1,15 @@
> +--- old/tools/profiler/core/platform-linux-android.cpp
> ++++ new/tools/profiler/core/platform-linux-android.cpp
> +@@ -505,8 +505,12 @@
> +   MOZ_ASSERT(mIsSynchronous);
> +   MOZ_ASSERT(aContext);
> + 
> ++#ifdef __GLIBC__
> +   if (!getcontext(aContext)) {
> ++#endif
> +     FillInSample(*this, aContext);
> ++#ifdef __GLIBC__
> +   }
> ++#endif
> + }
> + 
> diff --git a/testing/firefox/tools_profiler_missing_header.patch b/testing/firefox/tools_profiler_missing_header.patch
> new file mode 100644
> index 0000000000..406dabdf27
> --- /dev/null
> +++ b/testing/firefox/tools_profiler_missing_header.patch
> @@ -0,0 +1,10 @@
> +--- old/tools/profiler/lul/LulElf.cpp
> ++++ new/tools/profiler/lul/LulElf.cpp
> +@@ -53,6 +53,7 @@
> + #include <sys/stat.h>
> + #include <unistd.h>
> + #include <arpa/inet.h>
> ++#include <libgen.h>
> + 
> + #include <set>
> + #include <string>



---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---
Reply to thread Export thread (mbox)