Please update uim to version 1.5.6 as it fixes a FTBFS

Bug #409034 reported by Fabrice Coutadeur
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
uim (Debian)
Fix Released
Undecided
Unassigned
uim (Ubuntu)
Fix Released
High
Chris Coulson

Bug Description

Binary package hint: uim

Hi,

This package FTBFS because of missing gnome.h header.
This is because this package is not aware of libgnomeui-2.0 package, and does not include the header and the lib.

Updating to version 1.5.6 fixes this problem

Cheers,
Fabrice

Related branches

Changed in uim (Ubuntu):
assignee: nobody → fabrice_sp (fabricesp)
status: New → In Progress
summary: - FTBFS because of tha package does not detect libgnomeui-2.0
+ FTBFS because of the package does not detect libgnomeui-2.0
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

debdiff with the definition of LDFLAGS and CFLAGS in debian/rules. Also, changed build dependency from x-dev to x11proto-core-dev as stated at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=515522

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote : Re: FTBFS because of the package does not detect libgnomeui-2.0

Here the log of the build

summary: - FTBFS because of the package does not detect libgnomeui-2.0
+ FTBFS because uim does not detect libgnomeui-2.0
Changed in uim (Ubuntu):
assignee: fabrice_sp (fabricesp) → nobody
status: In Progress → Confirmed
Changed in uim (Debian):
status: Unknown → New
Revision history for this message
Chris Coulson (chrisccoulson) wrote : Re: FTBFS because uim does not detect libgnomeui-2.0

Thanks for your work there. However, whilst forcing CFLAGS and LDFLAGS in debian/rules will make the package build, it will cause every binary in the package to be linked against libgnomeui-2.0, won't it? This will then mean that all the final binary packages will pull in the libgnomeui dependency even if they don't need them.

I think the correct way to fix this is to modify configure.ac and use PKG_CHECK_MODULES to get the LDFLAGS and CFLAGS for libgnomeui-2.0 and export them in such a way that only uim-toolbar-applet is linked against this library. With this method, the only binary package which will depend on libgnomeui will be uim-applet-gnome. Do you agree?

Would you mind working on this? (or perhaps checking if it is still an issue in uim 1.5.6)

Thanks :)

Changed in uim (Ubuntu):
assignee: nobody → fabrice_sp (fabricesp)
importance: Undecided → High
status: Confirmed → Incomplete
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Hi,

Version 1.5.6 does not have this problem, so I'll package it. Anyway, I found the way upstream fixed it, so if the packaging of 1.5.6 is not successful, I can retrofit the patch.

Thanks,
Fabrice

Changed in uim (Ubuntu):
status: Incomplete → In Progress
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Hi,

Here is the diff file with debian directory content.

The source tarball for 1.5.6 can be downloaded from http://code.google.com/p/uim/

I've made the following changes in the packaging:
  * debian/control:
    - added Homepage tag
    - added watch file
    - bump Standards-Version to 3.8.2
    - changed build dependency on x-dev to x11proto-core-dev to avoid a lintian
      error (Debian bug #515522)
    - Added ${misc:Depends} to packages, to avoid several lintian warnings
    - Update build dependency on libgtk2.0-dev to delete the -1 revision and
      fix this way another lintian warning
   * debian/uim-qt3.install: changed uim-chardict-qt.mo to uim.mo as upstream
     renamed it.
   * Added debian/README.source as this package uses dpatch to apply patches

Thanks,

Fabrice

summary: - FTBFS because uim does not detect libgnomeui-2.0
+ Please update uim to version 1.5.6 as it fixes a FTBFS
description: updated
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Thank you! I'll take this for sponsoring now

Changed in uim (Ubuntu):
assignee: fabrice_sp (fabricesp) → Chris Coulson (chrisccoulson)
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Hi,

I've just uploaded this now. One small comment - you left the previous CFLAGS/LDFLAGS workaround in debian/rules. I assume this wasn't intentional so I removed it for you, and it's still building ok.

Other than that, great work! Thanks you :)

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package uim - 1:1.5.6-0ubuntu1

---------------
uim (1:1.5.6-0ubuntu1) karmic; urgency=low

  * New upstream release (fix FTBFS because of missing gnome.h) (LP: #409034)
  * debian/control:
    - added homepage tag
    - added watch file
    - bump Standards-Version to 3.8.2
    - changed build dependency on x-dev to x11proto-core-dev to avoid a lintian
      error (Debian bug #515522)
    - Added ${misc:Depends} to packages, to avoid several lintian warnings
    - Update build dependency on libgtk2.0-dev to delete the -1 revision and
      fix this way another lintian warning
   * debian/uim-qt3.install: changed uim-chardict-qt.mo to uim.mo as upstream
     renamed it.
   * Added debian/README.source as this package uses dpatch to apply patches

 -- Fabrice Coutadeur <email address hidden> Mon, 10 Aug 2009 03:55:11 +0000

Changed in uim (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Hi,

You're right: I forgot to remove the workaround in debian/rules. Sorry about that.

Thanks for sponsoring it,

Fabrice

Revision history for this message
Artur Rona (ari-tczew) wrote :

 uim (1:1.5.6-1) unstable; urgency=high

   * New upstream release.
     - configure.ac : Fix not found gnome.h header (patch from Fedora).
       See <http://code.google.com/p/uim/source/detail?r=5963>; for more details.
       (closes: Bug#536944)
   * New maintainer (acknowledged from former maintainer Masahiro Omote).
   * 07_fix_configure_notoverwrite_po_makefile.dpatch:
     patch for not install uim-chardict-qt.mo file.
     See <http://groups.google.com/group/uim-ja/browse_thread/thread/31a942b9e3bfc2b7/47e2601cd04b0f3c#47e2601cd04b0f3c>; for more details (in Japanese).

 -- Kiwamu Okabe <email address hidden> Tue, 08 Sep 2009 16:44:29 +0900

Changed in uim (Debian):
importance: Unknown → Undecided
status: New → Fix Released
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.