pkg/48777: "firefox" doesn't build under NetBSD 6.1 with native X11

[tron%zhadum.org.uk@localhost]

>Number:         48777
>Category:       pkg
>Synopsis:       "firefox" doesn't build under NetBSD 6.1 with native X11
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri May 02 09:55:00 +0000 2014
>Originator:     Matthias Scheler
>Release:        NetBSD 6.1_STABLE pkgsrc HEAD as of 2014-05-02
>Organization:
Matthias Scheler                                 https://zhadum.org.uk/
>Environment:
System: NetBSD colwyn.zhadum.org.uk 6.1_STABLE NetBSD 6.1_STABLE (COLWYN.64) 
#1: Tue Apr 22 19:58:32 BST 2014 
tron%colwyn.zhadum.org.uk@localhost:/src/sys/compile/COLWYN.64 amd64
Architecture: x86_64
Machine: amd64
>Description:
=> Bootstrap dependency digest>=20010302: found digest-20121220
=> Checksum SHA1 OK for firefox-29.0.source.tar.bz2
=> Checksum RMD160 OK for firefox-29.0.source.tar.bz2
===> Installing dependencies for firefox-29.0
==========================================================================
The supported build options for firefox are:

        alsa debug gnome mozilla-jemalloc official-mozilla-branding
        pulseaudio webrtc

The currently selected options are:

        pulseaudio

You can select which build options to use by setting PKG_DEFAULT_OPTIONS
or the following variable.  Its current value is shown:

        PKG_OPTIONS.firefox (not defined)

==========================================================================
==========================================================================
The following variables will affect the build process of this package,
firefox-29.0.  Their current value is shown below:

        * JPEG_DEFAULT = jpeg
        * PYTHON_VERSION_DEFAULT = 27
        * SSLBASE = /usr
        * SSLCERTS = /etc/openssl/certs
        * SSLDIR = /etc/openssl
        * SSLKEYS = /etc/openssl/private

Based on these variables, the following variables have been set:

        * JPEGBASE (defined, but empty)
        * JPEG_TYPE = jpeg
        * PYPACKAGE = python27

You may want to abort the process now with CTRL-C and change their value
before continuing.  Be sure to run `/usr/bin/make clean' after
the changes.
==========================================================================
=> Tool dependency autoconf213>=2.13: found autoconf213-2.13nb3
=> Tool dependency gmake>=3.81: found gmake-4.0
=> Tool dependency perl>=5.0: found perl-5.18.2nb1
=> Tool dependency pkg-config>=0.25: found pkg-config-0.28
=> Tool dependency zip-[0-9]*: found zip-3.0nb2
=> Tool dependency zip-[0-9]*: found zip-3.0nb2
=> Tool dependency zip-[0-9]*: found zip-3.0nb2
=> Tool dependency zip-[0-9]*: found zip-3.0nb2
=> Tool dependency checkperms>=1.1: found checkperms-1.11
=> Build dependency py27-sqlite2-[0-9]*: found py27-sqlite2-2.6.3nb7
=> Build dependency x11-links>=0.88: found x11-links-0.88
=> Build dependency python27>=2.7.1nb2: found python27-2.7.6nb3
=> Full dependency sqlite3>=3.8.4.3nb1: found sqlite3-3.8.4.3nb1
=> Full dependency libffi>=3.0.11: found libffi-3.0.13
=> Full dependency nspr>=4.10.2: found nspr-4.10.4
=> Full dependency icu>=53.1: found icu-53.1
=> Full dependency nss>=3.15.4: found nss-3.16nb1
=> Full dependency jpeg>=9: found jpeg-9anb1
=> Full dependency cairo>=1.12.14nb3: found cairo-1.12.16nb1
=> Full dependency graphite2>=1.2: found graphite2-1.2.4
=> Full dependency libvpx>=1.0.0: found libvpx-1.3.0
=> Full dependency libIDL>=0.8.14nb4: found libIDL-0.8.14nb4
=> Full dependency hunspell>=1.3.2: found hunspell-1.3.2nb2
=> Full dependency gtk2+>=2.24.20nb2: found gtk2+-2.24.22nb2
=> Full dependency gstreamer0.10>=0.10.36nb3: found gstreamer0.10-0.10.36nb6
=> Full dependency gst-plugins0.10-base>=0.10.36nb3: found 
gst-plugins0.10-base-0.10.36nb7
=> Full dependency pulseaudio>=2.1nb16: found pulseaudio-2.1nb16
=> Full dependency desktop-file-utils>=0.10nb1: found desktop-file-utils-0.22
===> Overriding tools for firefox-29.0
===> Extracting for firefox-29.0
===> Patching for firefox-29.0
=> Applying pkgsrc patches for firefox-29.0
[...]
===> Configuring for firefox-29.0
[...]
===> Building for firefox-29.0
[...]
c++ -o jsweakmap.o -c  -I../../../../dist/system_wrappers -include 
/objdir/tron/www/firefox/work.colwyn/mozilla-release/js/src/../../config/gcc_hidden.h
 -DEXPORT_JS_API -DJS_HAS_CTYPES -DDLL_PREFIX='"lib"' -DDLL_SUFFIX='".so"' 
-DMOZ_GLUE_IN_PROGRAM -DNO_NSPR_10_SUPPORT -DUSE_ZLIB 
-I/objdir/tron/www/firefox/work.colwyn/mozilla-release/js/src/../../js/src -I. 
-I/usr/pkg/include  
-I/objdir/tron/www/firefox/work.colwyn/mozilla-release/js/src/../../js/src/../../mfbt/double-conversion
 -I/usr/pkg/include  -I../../../../dist/include  -I/usr/pkg/include/nspr        
-fPIC -DPIC  -I/usr/pkg/include -I/usr/include -I/usr/pkg/include/nspr 
-I/usr/pkg/include/nss/nss -I/usr/X11R7/include -I/usr/X11R7/include/libdrm 
-I/usr/X11R7/include/freetype2 -DMOZILLA_CLIENT -include 
../../js/src/js-confdefs.h -MD -MP -MF .deps/jsweakmap.o.pp -I/usr/pkg/include 
-I/usr/include -I/usr/pkg/include/nspr -I/usr/pkg/include/nss/nss 
-I/usr/X11R7/include -I/usr/X11R7/include/libdrm -I/usr/X11R7/include/freetype
 2 -Wall -Wpointer-arith -Woverloaded-virtual -Werror=return-type 
-Werror=int-to-pointer-cast -Wtype-limits -Wempty-body -Werror=conversion-null 
-Wsign-compare -Wno-invalid-offsetof -Wcast-align -O2 -I/usr/pkg/include 
-I/usr/include -I/usr/pkg/include/nspr -I/usr/pkg/include/nss/nss 
-I/usr/X11R7/include -I/usr/X11R7/include/libdrm -I/usr/X11R7/include/freetype2 
-Dunix -fno-rtti -fno-exceptions -fno-math-errno -std=gnu++0x -pthread -pipe  
-DNDEBUG -DTRIMMED -O2 -fomit-frame-pointer -DUSE_SYSTEM_MALLOC=1 
-DENABLE_ASSEMBLER=1 -DENABLE_JIT=1   
/objdir/tron/www/firefox/work.colwyn/mozilla-release/js/src/jsweakmap.cpp
jsworkers.o
In file included from ../../../dist/include/mozilla/layers/LayersTypes.h:11:0,
                 from ../../../dist/include/nsIWidget.h:21,
                 from ../../../dist/include/mozilla/BasicEvents.h:14,
                 from ../../../dist/include/nsEventListenerManager.h:9,
                 from ../../../dist/include/nsDOMEventTargetHelper.h:14,
                 from ../../../dist/include/mozilla/dom/TabChild.h:26,
                 from 
/objdir/tron/www/firefox/work.colwyn/mozilla-release/netwerk/protocol/ftp/FTPChannelChild.cpp:10:
../../../dist/include/nsRegion.h: In member function 'void 
nsRegion::SetEmpty()':
../../../dist/include/nsRegion.h:178:33: error: 'pixman_region32_clear' was not 
declared in this scope
../../../dist/include/nsRegion.h: In member function 'nsRegion& 
nsRegion::Copy(const nsRect&)':
../../../dist/include/nsRegion.h:272:35: error: 'pixman_region32_clear' was not 
declared in this scope
/objdir/tron/www/firefox/work.colwyn/mozilla-release/config/rules.mk:1028: 
recipe for target 'FTPChannelChild.o' failed
gmake[3]: *** [FTPChannelChild.o] Error 1
gmake[3]: Leaving directory 
'/objdir/tron/www/firefox/work.colwyn/build/netwerk/protocol/ftp'
/objdir/tron/www/firefox/work.colwyn/mozilla-release/config/recurse.mk:100: 
recipe for target 'netwerk/protocol/ftp/compile' failed
gmake[2]: *** [netwerk/protocol/ftp/compile] Error 2
gmake[2]: *** Waiting for unfinished jobs....
[...]
gmake[1]: Leaving directory '/objdir/tron/www/firefox/work.colwyn/build'
/objdir/tron/www/firefox/work.colwyn/mozilla-release/config/rules.mk:596: 
recipe for target 'all' failed
gmake: *** [all] Error 2
*** Error code 2

Stop.
make: stopped in /src/NetBSD-current/pkgsrc/www/firefox
*** Error code 1

Stop.
make: stopped in /usr/pkgsrc/www/firefox

>How-To-Repeat:
cd pkgsrc/www/firefox
make

>Fix:
We cannot simply bump the API depends for "pixman" for the "firefox" package
because it pulls in shared libraries (e.g. GTK+ 2) which use "pixman" as well.
We should instead identify the minimum "pixman" version that provides these
new functions, update the API depends of the "pixman" package and perform a
recursive revision bump.