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

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

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 [ ]

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
Hi! I have been running into this issue as well while testing new patches for Firefox upstream. I actually first suspected that my build environment was broken, so I generated a fresh chroot. But it seems this is actually a bug. Looking at the build log, this line in particular looks

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