[FFe]Please update to gecko-mediaplayer 1.0.5-1

[Aron Xu]

FFe :
Currently, gnome-mpalyer version in precise is 1.0.5. As gecko-mediaplayer use gnome-mplayer and follow the same developpement, it's better to have the same version for both applications.

Also, this update fix the FTBFS of the current package.

Main changes between 1.0.4 and 1.0.5 are :
- Migration to libgmtk (like gnome-mplayer 1.0.5)
- Some improvements in caching
- Bug fixes and translations updates

The update is a sync with debian testing (see original bug report), + an additional debian/changelog entry :
gecko-mediaplayer (1.0.5-1ubuntu1) precise; urgency=low

  * Sync with Debian. (LP: #928101)
  * debian/rules:
   - Update --with-xulrunner-sdk and add --enable-new-libxul.
     Thanks Brandon Snider (LP: #935501).

Tetsing packages are available in the Lubuntu staging PPA : https://launchpad.net/~lubuntu-dev/+archive/staging

Complete upstream changelog :
1.0.5
    Disable playing of partially cached apple.com files, mplayer seems to crash on these files
    Use the right schema file, can't believe this one has slipped by so long.
    Use better const check from Issue #161, patch from i.stakenvicius
1.0.5b2
    Simpifying const check until something better comes along Issue #162
    Fix Issue #161, improvements over last patch because NPAPI not properly bumped
    Fix support for older xulrunners, Issue #160, issue caused by patch from
    Issue #158
    Support nsapi-sdk, based on patch from Issue #159
    Fix up some const casting so that compilers do not complain Issue #158
    Updated Japanese translation
    Updated Italian translation
1.0.5b1
    When uri is a file:// uri, just play it directly
    Switch from AM_MAINTAINER_MODE to AM_MAINTAINER_MODE([enable]) in configure.in
    Fix Issue #150, return value for function that says it will
    Updated Dutch translation
    Conversion to using libgmtk as separate library
    Improvements to the apple.com/trailers fix, caching now works properly
    Fix regression at apple.com/trailers, side effect of fully caching the file for now

Build log (gecko-mediaplayer-build.log) and install log (gecko-mediaplayer-install.log) attached.

Original bug report :

Please sync gecko-mediaplayer 1.0.5-1 from Debian Testing to Ubuntu universe. The version in Ubuntu (1.0.4) is not compatible with gnome-mplayer 1.0.5 (already in Ubuntu), and deltas are already merged to Debian.

gecko-mediaplayer (1.0.5-1) unstable; urgency=low

   [ Sebastian Ramacher ]
   * New upstream release (Closes: #652152).
   * debian/control:
     - add myself to Uploaders.
     - don't depend on specific browsers (Closes: #567169)
     - Build-Dep on dh-autoreconf instead of autotools-dev.
     - Build-Dep on pkg-config (>= 2.26).
     - Remove unnecessary Build-Dep on libidl0.
   * debian/rules:
     - regenerate configure with dh-autoreconf.
   * debian/patches:
     - add 0002-fix-spelling-error.patch to fix spelling mistake in
       src/plugin.cpp.
     - update 0001-nomozalloc.patch to not include diff for configure.

   [ Julien Lavergne ]
   * debian/patches:
     - force-no-gsettings.patch: Remove, since gnome-mplayer provide the
       gsettings schemas.

   [ Brandon Snider ]
   * debian/rules: convert package to dh short form (Closes: #638336).
   * debian/compat: bump compat to 8.
   * debian/control:
     - Build-Dep on debhelper (>= 8)
     - Build-Dep on libgmlib-dev
   * debian/copyright: make it DEP-5 compliant

   [ Aron Xu ]
   * debian/control:
     - Move to Debian Multimedia Team, add Julien and myself to Uploaders.
       (Closes: #628997)
     - Add Vcs-* fields.

 -- Sebastian Ramacher <email address hidden> Thu, 05 Jan 2012 00:53:13 +0100

PTS: http://packages.qa.debian.org/g/gecko-mediaplayer.html
DSC: http://cdn.debian.net/debian/pool/main/g/gecko-mediaplayer/gecko-mediaplayer_1.0.5-1.dsc

[Daniel Holbach (dholbach)]

I'm afraid this fails to build on amd64 precise:

checking for GLIB... yes
checking for GMLIB... yes
checking for GECKO... no
configure: libxul >= 2 not found, trying another
checking for GECKO... no
configure: libxul >= 1.9.2 not found, trying another
checking for GECKO... no
configure: WARNING: libxul < 1.9.2 not found, trying another
checking for GECKO... no
configure: WARNING: libxul-unstable not found, trying another
checking for GECKO... no
configure: WARNING: mozilla-plugin not found, trying another
checking for GECKO... no
configure: WARNING: firefox-plugin not found, trying another
checking for GECKO... no
configure: WARNING: seamokey-plugin not found, trying another
checking for GECKO... no
configure: WARNING: xulrunner-plugin not found, trying another
checking for GECKO... no
configure: WARNING: iceape-plugin not found, trying another
checking for GECKO... no
configure: error: npapi-sdk not found, all out ot tries

[Aron Xu (happyaron)]

Ah sorry, it looks to caused by the messy status of mozilla packages on both sides, the package was building on Ubuntu when it was uploaded to Sid... Anyway I'll try to provide a working version for you.

[Daniel Holbach (dholbach)]

Until then I'll unsubscribe the sponsors team.

[Aron Xu (happyaron)]

Hi Julien,

Can you help to look at this problem? Ubuntu's mozilla packaging has changed a lot, and the FTBFS can't be resolved by simply installing firefox-dev as build-dep. There are at least directory name change, but I don't have enough knowledge to do with mozilla related software.

Thanks

[Julien Lavergne (gilir)]

I managed to fix this problem, but now I have linker problems with firefox 11 header.
You can use the 2 patches I made to see the problem.

[Julien Lavergne (gilir)]

  CXX memmem_compat.o
  CXX plugin_list.o
  CXX plugin_setup.o
  CXX plugin_dbus.o
  CXX np_entry.o
  CXX npn_gate.o
  CXX npp_gate.o
  CXXLD gecko-mediaplayer.so
plugin.o: In function `operator new':
/usr/include/firefox-11.0/mozilla/mozalloc.h:229: undefined reference to `moz_xmalloc'
plugin.o: In function `ScriptablePluginObject::InvokeDefault(_NPVariant const*, unsigned int, _NPVariant*)':
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2559: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2559: undefined reference to `moz_strdup'
plugin.o: In function `ScriptablePluginObjectError::InvokeDefault(_NPVariant const*, unsigned int, _NPVariant*)':
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2214: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2214: undefined reference to `moz_strdup'
plugin.o: In function `ScriptablePluginObjectSettings::InvokeDefault(_NPVariant const*, unsigned int, _NPVariant*)':
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2116: undefined reference to `moz_strdup'
plugin.o:/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2116: more undefined references to `moz_strdup' follow
plugin.o: In function `operator new':
/usr/include/firefox-11.0/mozilla/mozalloc.h:229: undefined reference to `moz_xmalloc'
/usr/include/firefox-11.0/mozilla/mozalloc.h:229: undefined reference to `moz_xmalloc'
/usr/include/firefox-11.0/mozilla/mozalloc.h:229: undefined reference to `moz_xmalloc'
/usr/include/firefox-11.0/mozilla/mozalloc.h:229: undefined reference to `moz_xmalloc'
plugin.o: In function `ScriptablePluginObjectMedia::Invoke(void*, _NPVariant const*, unsigned int, _NPVariant*)':
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2008: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2008: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2001: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:2001: undefined reference to `moz_strdup'
/tmp/buildd/gecko-mediaplayer-1.0.5/src/plugin.cpp:1997: undefined reference to `moz_strdup'
plugin.o: In function `operator delete':
/usr/include/firefox-11.0/mozilla/mozalloc.h:253: undefined reference to `moz_free'
/usr/include/firefox-11.0/mozilla/mozalloc.h:253: undefined reference to `moz_free'
/usr/include/firefox-11.0/mozilla/mozalloc.h:253: undefined reference to `moz_free'
/usr/include/firefox-11.0/mozilla/mozalloc.h:253: undefined reference to `moz_free'
/usr/include/firefox-11.0/mozilla/mozalloc.h:253: undefined reference to `moz_free'
plugin.o:/usr/include/firefox-11.0/mozilla/mozalloc.h:253: more undefined references to `moz_free' follow
collect2: ld returned 1 exit status
make[4]: *** [gecko-mediaplayer.so] Error 1
make[4]: Leaving directory `/tmp/buildd/gecko-mediaplayer-1.0.5/src'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/tmp/buildd/gecko-mediaplayer-1.0.5/src'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/tmp/buildd/gecko-mediaplayer-1.0.5'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/tmp/buildd/gecko-mediapla...

[Brandon Snider (brandonsnider)]

Fixed here:
https://code.launchpad.net/~brandonsnider/+junk/gecko-mediaplayer

Summary: added "--with-xulrunner-sdk=$(shell pkg-config --variable=sdkdir mozilla-plugin)" to rules which yields "/usr/lib/firefox-devel" as the location of the Gecko dev files, not "/usr/include/firefox-11.0" as above.

Added "--enable-new-libxul", which is on by default starting with gecko-mediaplayer 1.0.6. This fixes a build error relating to uppercase/lowercase that would occur without it.

Ran test builds, they succeed: https://launchpadlibrarian.net/101427031/buildlog_ubuntu-precise-i386.gecko-mediaplayer_1.0.5-1ubuntu1~ppa1_BUILDING.txt.gz

This means two build flags are all that's needed for this to work and not source code patches. FYI, 1.0.6 also builds, and as long as there are no major changes to firefox-dev in the future, such as the removal of /usr/lib/pkgconfig/mozilla-plugin.pc (the only mozilla pkgconfig file left in Ubuntu as far as I can tell) there shouldn't be too much trouble with this package.

[Stefano Rivera (stefanor)]

This missed precise. Unsubscribing the release team.
Any important bug fixes can still be SRUed into precise.
https://wiki.ubuntu.com/StableReleaseUpdates

[Launchpad Janitor (janitor)]

This bug was fixed in the package gecko-mediaplayer - 1.0.6-1ubuntu1

---------------
gecko-mediaplayer (1.0.6-1ubuntu1) quantal; urgency=low

  * Sync with Debian (LP: #928101)
  * Ubuntu remaining changes :
   - debian/rules:
    + Add --with-xulrunner-sdk to fix FTBFS on Ubuntu (LP: #935501)
    + Disable --as-needed to fix another FTBFS on Ubuntu.

gecko-mediaplayer (1.0.6-1) unstable; urgency=low

  * New upstream release.
  * debian/patches:
    - Update 0001-mozalloc.patch.
    - Remove 0002-fix-spelling-error.patch (not needed anymore).
  * debian/control:
    - Bump Standards version to 3.9.3 (no changes required).
    - Bump Depends on gnome-mplayer to 1.0.6 to keep them in sync.
    - Bump Build-Dep on debhelper to >= 9.
    - Drop unnecessary Build-Dep on dpkg-dev.
  * debian/copyright: Update Format URL.
  * debian/rules:
    - Enable verbose build output.
    - Enable parallel build.
    - Drop include of /usr/share/dpkg/buildflags.mk. The flags are exported by
      debhelper with compat level 9.
  * debian/compat: Bump to 9.

gecko-mediaplayer (1.0.5-1) unstable; urgency=low

  [ Sebastian Ramacher ]
  * New upstream release (Closes: #652152).
  * debian/control:
    - add myself to Uploaders.
    - don't depend on specific browsers (Closes: #567169)
    - Build-Dep on dh-autoreconf instead of autotools-dev.
    - Build-Dep on pkg-config (>= 2.26).
    - Remove unnecessary Build-Dep on libidl0.
  * debian/rules:
    - regenerate configure with dh-autoreconf.
  * debian/patches:
    - add 0002-fix-spelling-error.patch to fix spelling mistake in
      src/plugin.cpp.
    - update 0001-nomozalloc.patch to not include diff for configure.

  [ Julien Lavergne ]
  * debian/patches:
    - force-no-gsettings.patch: Remove, since gnome-mplayer provide the
      gsettings schemas.

  [ Brandon Snider ]
  * debian/rules: convert package to dh short form (Closes: #638336).
  * debian/compat: bump compat to 8.
  * debian/control:
    - Build-Dep on debhelper (>= 8)
    - Build-Dep on libgmlib-dev
  * debian/copyright: make it DEP-5 compliant

  [ Aron Xu ]
  * debian/control:
    - Move to Debian Multimedia Team, add Julien and myself to Uploaders.
      (Closes: #628997)
    - Add Vcs-* fields.
 -- Julien Lavergne <email address hidden> Fri, 04 May 2012 19:11:21 +0200

[Aron Xu (happyaron)]

This bug is still relevant for Precise.