#245 Doesn't build with xulrunner 9

closed-fixed
nobody
None
5
2012-07-13
2012-01-10
Debian Package
No

Hi,

gpac doesn't build with xulrunner 9 :

ccache cc -O3 -fPIC -DPIC -DXP_UNIX -Wall -g -Wall -fno-strict-aliasing -Wno-pointer-sign -DGPAC_HAVE_CONFIG_H -I"/home/marillat/src/gpac-0.4.6~svn20120110" -I"/home/marillat/src/gpac-0.4.6~svn20120110/include" -DSPIDERMONKEY_NEW_API -DXP_UNIX -DJS_THREADSAFE -I/usr/include/nspr -I/usr/include/mozjs -c -o scenegraph/vrml_smjs.o scenegraph/vrml_smjs.c
scenegraph/vrml_smjs.c: In function ‘on_route_to_object’:
scenegraph/vrml_smjs.c:703:7: error: used union type value where scalar is required
scenegraph/vrml_smjs.c:705:10: error: incompatible types when assigning to type ‘jsval’ from type ‘int’
scenegraph/vrml_smjs.c: In function ‘deleteRoute’:
scenegraph/vrml_smjs.c:868:42: error: invalid operands to binary == (have ‘jsval’ and ‘jsval’)
scenegraph/vrml_smjs.c:868:67: error: invalid operands to binary == (have ‘jsval’ and ‘jsval’)
scenegraph/vrml_smjs.c: In function ‘gf_sg_script_eventout_set_prop’:
scenegraph/vrml_smjs.c:1146:2: error: incompatible type for argument 2 of ‘JS_ValueToString’
/usr/include/mozjs/jsapi.h:1835:1: note: expected ‘jsval’ but argument is of type ‘jsid’
make[1]: *** [scenegraph/vrml_smjs.o] Erreur 1

Discussion

  • Hi,

    Are you a Debian maintainer? The xulrunner APIs are moving too fast for us to follow, do you know if your package base is about to include any old xulrunner/libjs/mozjs version?

    Romain

     
  • Debian Package
    Debian Package
    2012-01-12

    Yes, I'm a Debian maintainer.

    Old release are removed. For now we only have xulrunner 8 and 9 for runtime library and only 9 for as dev package.

     
  • Debian Package
    Debian Package
    2012-01-25

    Hi,

    FYI, xulrunner 8 has been removed from Debian (and probably from Ubuntu) and thus gpac isn't installable...

    Christian

     
  • Jean Le Feuvre
    Jean Le Feuvre
    2012-01-26

    Thanks for the info. We're actually thinking of getting rid of the xulrunner dependency, this confirms the plan is right ... We'll keep you posted on that topic

    jean

     
  • Christian,

    In the gpac package, there are tools for content packaging (MP4Box) that are widely used and does not require spidermonkey. There is also a player (less used) that *may* require spidermonkey for javascript support.

    What I suggest is to temporally remove the javascript support. To do so, execute 'configure --use-js=no', and you'll be able to remove the spidermonkey package dependency. Does that sound ok to you?

    Romain

     
  • Debian Package
    Debian Package
    2012-01-27

    Hi,

    Yes I agree, I'll do that.

    Christian

     
  • Jean Le Feuvre
    Jean Le Feuvre
    2012-03-05

    Compilation with latest xulrunner (tested on ubuntu 11.10, 12.04 & debian wheezy) should now be fixed and JS support should work. Could you test that ?

     
  • Jean Le Feuvre
    Jean Le Feuvre
    2012-07-13

    • status: open --> closed-fixed