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

[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 me that [:space:] is indeed
wrong. Looking at build/autoconf/icu.m4, the m4 code already contains the
correct syntax:

version=`sed -n 's/^[[:space:]]*#[[:space:]]*define[[:space:]][[:space:]]*U_ICU_VERSION_MAJOR_NUM[[:space:]][[:space:]]*\([0-9][0-9]*\)[[:space:]]*$/\1/p'
"$icudir/common/unicode/uvernum.h"`

So, it's probably sufficient to regenerate the old-configure script. Might a good
idea to switch both the firefox and the icedove package to autoreconf [1].

Thanks,
Adrian

> [1] https://wiki.debian.org/Autoreconf

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaubitz at debian.org
`. `'   Freie Universitaet Berlin - glaubitz at physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913