Bug#850984: Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread John Paul Adrian Glaubitz
On 01/12/2017 10:48 PM, Mike Hommey wrote: >> What about Icedove? Should we cherry-pick the patch in this case? > > Icedove uses Firefox code base upstream, so it will end up there > mechanically. Yep, I'm aware of that. I was actually referring to the Debian icedove package but as it seems Carst

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread Mike Hommey
On Thu, Jan 12, 2017 at 01:35:35PM +0100, John Paul Adrian Glaubitz wrote: > Hi Mike! > > On 01/12/2017 01:30 PM, Mike Hommey wrote: > > It's more subtle than that. Also, firefox and icedove already do an > > autoreconf on their own. > > > > icu.m4 is autoconf m4, iow, m4 with [ and ] as quote ch

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread Carsten Schoenert
Hello John, On 12.01.2017 13:35, John Paul Adrian Glaubitz wrote: ... >> This should be fixed in next upstream release, in less than two weeks. > > What about Icedove? Should we cherry-pick the patch in this case? this was already fixed in src:icedove with the upload of 1:45.6.0-2 which happend

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread John Paul Adrian Glaubitz
Hi Mike! On 01/12/2017 01:30 PM, Mike Hommey wrote: > It's more subtle than that. Also, firefox and icedove already do an > autoreconf on their own. > > icu.m4 is autoconf m4, iow, m4 with [ and ] as quote characters. Which > means the [[ and ]] in that expression are replaced with literal [ ] in

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread Mike Hommey
On Thu, Jan 12, 2017 at 12:25:57PM +0100, John Paul Adrian Glaubitz wrote: > On 01/12/2017 12:02 PM, John Paul Adrian Glaubitz wrote: > >> sed: character class syntax is [[:space:]], not [:space:] > > > > Almost looks like sed changed its syntax although I can't really believe > > this > > would

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-12 Thread John Paul Adrian Glaubitz
On 01/12/2017 12:02 PM, John Paul Adrian Glaubitz wrote: >> sed: character class syntax is [[:space:]], not [:space:] > > Almost looks like sed changed its syntax although I can't really believe this > would happen without a major version bump. So, a quick Google research and #debian-devel tell m

Bug#850972: firefox: FTBFS: configure: error: cannot determine icu version number from uvernum.h header file

2017-01-11 Thread Lucas Nussbaum
Source: firefox Version: 50.1.0-1 Severity: serious Tags: stretch sid User: debian...@lists.debian.org Usertags: qa-ftbfs-20170111 qa-ftbfs Justification: FTBFS on amd64 Hi, During a rebuild of all packages in sid, your package failed to build on amd64. Relevant part (hopefully): > checking for