New upstream release available for cairomm

Bug #566986 reported by Daniel Elstner
22
This bug affects 2 people
Affects Status Importance Assigned to Milestone
cairomm (Ubuntu)
Fix Released
Wishlist
Unassigned
Lucid
Fix Released
Wishlist
Sebastien Bacher

Bug Description

Ubuntu (and Debian) currently package version 1.8.0 of cairomm. This version is heavily outdated. The latest stable release 1.8.4 was made in October 2009. Due to the revamped build system based on mm-common, the 1.8.4 release of cairomm fixes quite a number of problems related to packaging and system integration, including the bugs #452733 and #515638.

Apart from that, the current libcairomm-1.0-dev development package in Lucid is completely unusable because it ships with an utterly broken cairommconfig.h file which redefines macros such as PACKAGE_NAME or VERSION. This, too, was fixed in cairomm 1.8.4.

Please update the package to version 1.8.4 as soon as possible.

Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

This bug should be fowarded to debian also

Revision history for this message
Daniel Elstner (daniel-elstner) wrote :

An updated cairomm package for Lucid can now be found in my PPA: https://edge.launchpad.net/~daniel-elstner/+archive/ppa

Revision history for this message
Daniel Elstner (daniel-elstner) wrote :
Revision history for this message
Daniel Elstner (daniel-elstner) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report, lucid is frozen now and such change will not be approved between rc and lucid but we will get the update in a stable update after lucid

Changed in cairomm (Ubuntu):
importance: Undecided → Wishlist
status: New → Confirmed
Changed in cairomm (Ubuntu Lucid):
milestone: none → lucid-updates
Revision history for this message
Sebastien Bacher (seb128) wrote :
Changed in cairomm (Ubuntu Lucid):
assignee: nobody → Sebastien Bacher (seb128)
status: Confirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

It's a fairly intrusive change, but it keeps API stability, so it looks okay. However, I'm not sure how this can be tested? Are there some prominent applications which use cairomm?

tags: added: verification-needed
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted cairomm into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

summary: - New upstream release available
+ New upstream release available for cariomm
summary: - New upstream release available for cariomm
+ New upstream release available for cairomm
Revision history for this message
Daniel Elstner (daniel-elstner) wrote :

Regarding API and ABI: I confirmed with Jonathon Jongsma (upstream maintainer) that no API or ABI that existed in 1.8.0 has been changed in 1.8.4. There have only been additions to undo an accidental removal of API that existed before 1.8.0.

I just had a look at the new cairomm packages in proposed and they mostly look good. However, the documentation package is broken: The refererence documentation should be accessible via the /usr/share/doc/cairomm-1.0 directory name. It can be a symlink to a package-specific directory if necessary, as long as other packages using the cairomm-1.0 pkg-config information continue to work (pkg-config --variable docdir cairomm-1.0).

The installed Devhelp file also refers to that directory. Unfortunately it seems to be completely missing from the new packages. Also, the automatic compression of the Doxygen tag file must be suppressed because Doxygen currently cannot cope with it, and it won't be found by the documentation cross-reference logic in mm-common.

The cairomm package uploaded to my PPA gets these details right, which was the main reason I modified it heavily. It also adds a debug binary package. Guess it's my fault for not documenting my changes in detail...

Revision history for this message
Sebastien Bacher (seb128) wrote :

thank you for your comment, the changelog in your ppa is not really clear about those changes, could you maybe add a debdiff over the current sru version to fix the issue you listed? the version uploaded is similar to what has been uploaded to debian, could you explain the tag compression issue?

Revision history for this message
Sebastien Bacher (seb128) wrote :

Daniel, could you reply to my questions? It's blocking further work to get that update in lucid

Revision history for this message
Pedro Villavicencio (pedro) wrote :

I've been using the proposed package for a few days already and did not noticed any regression so far, I'm marking this as verification done, thanks all.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Martin Pitt (pitti) wrote :

Maverick has 1.8.4-3, closing task.

Changed in cairomm (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cairomm - 1.8.4-0ubuntu1

---------------
cairomm (1.8.4-0ubuntu1) lucid-proposed; urgency=low

  * New upstream version (lp: #566986)
  * Updated install and rules files for the new version changes
 -- Sebastien Bacher <email address hidden> Wed, 05 May 2010 16:20:31 +0200

Changed in cairomm (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Daniel Elstner (daniel-elstner) wrote :

Sorry for the delay. I have now created a debdiff on top of the current Debian sid package, with all unrelated changes and cleanups removed. There is also a more detailed changelog:

  * debian/control:
    - Drop build dependencies on doxygen and graphviz, since upstream now
      ships the generated documentation
  * debian/libcairomm-1.0-doc.install:
    - Add the Devhelp book to the package
  * debian/libcairomm-1.0-doc.links:
    - Adjust the symlink for the upstream documentation directory, which has
      has changed to /usr/share/doc/cairomm-1.0
  * debian/rules:
    - Remove the clean:: target, as it is no longer needed
    - Add DEB_COMPRESS_EXCLUDE for .tag files since Doxygen cannot handle
      compressed tag files
    - Inlude clean-la.mk to purge dependency information from .la files

I sent the same debdiff to Debian, too.

Revision history for this message
Sebastien Bacher (seb128) wrote :

thank you for the detailled summary and debdiff, the current symlink is correct in lucid, not sure why you changed it there? otherwise out of the devhelp installation the change seem cleaning not worth a lucid change, we will get this change if the soure get uploaded for other reason though. Did you send the corresponding debdiff to debian as well?

Revision history for this message
Daniel Elstner (daniel-elstner) wrote :

No, the symlink is not correct. I just checked again with "dpkg -L libcairomm-1.0-doc". The symlink is named "libcairomm-1.0", but it should be cairomm-1.0 to match the location where "make install" would put it. This is important because, among other things, this location is advertised in the pkg-config (.pc) file that is part of the installation:

    pkg-config --variable=docdir cairomm-1.0

As to the Doxygen tag file, it is used when building the documentation of modules depending on cairomm. With the .gz extension, the file cannot be found by the build scripts, and even if it could be found, Doxygen would be unable to process the gzipped file (I tried that already).

And then, as you say, there is the missing Devhelp book (bug #515638).

As I said, I sent the same debdiff to Debian, as an attachment to this bug:

    http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=579261

Revision history for this message
Sebastien Bacher (seb128) wrote :

ok, I've built an updated version based on your change now, the base url being an online one seems to confuse devhelp though, should it be replaced by "/usr/share/doc/libcairomm-1.0-doc/reference" rather?

Revision history for this message
Daniel Elstner (daniel-elstner) wrote :

The base URL in the Devhelp file should normally be translated automatically when running "make install". I've just confirmed on my own system that the doc-install.pl script for some reason isn't invoked for the devhelp file during installation. It might be that the mm-common files included with the cairomm tarball are out of date; I'll have a look at this later.

Revision history for this message
Sebastien Bacher (seb128) wrote :

do you have any news about this issue?

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.