! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Patching intltool-merge broken with recent intltool versions
Status:
CLOSED: FIXED
Product:
Xfce4-dev-tools
Component:
General

Comments

Description Enrico Tröger editbugs 2008-10-10 13:48:39 CEST
Since intltool 0.40.x, the scripts intltool-merge/update/extract are not copied into the package directory anymore but instead globally installed ones are used.
For backward compatibility, these files are only touched in the package directory.
(related bugreport http://bugzilla.gnome.org/show_bug.cgi?id=537352)

So, since these files are empty, patching done in xdt-autogen obviously fail. Right now, I don't have a sane solution since patching /usr/bin/intltool-merge is pretty evil and will fail in most cases since people aren't installing with root permissions. Not to mention problems for distro packagers.

But for the long term, was the need of this patching reported upstream? So that it can go into intltool at some point and making the patching unnecessary?
Comment 1 Brian J. Tarricone (not reading bugmail) 2008-10-10 20:09:08 CEST
Intltool is fixed upstream.  I'm not sure if they've made a release since the fix was committed.  Actually yeah, it seems 0.40.5 was released on 8 Oct, and my fix should have gone in around when Pinkie was released, whenever that was.  0.40.4 was released on 19 Sep, so that might have the fix too.

But yeah, there's nothing we can do with unfixed 0.40.x versions.  For the affected packages, I may bump the intltool version requirements... which sucks, since I imagine quite a few distros won't have it yet, but I'm not sure what else to do.
Comment 2 Enrico Tröger editbugs 2008-10-11 16:21:13 CEST
Thanks for the information, Brian.

I manually updated intltool to 0.40.4 and then to 0.40.5, both contain the fix and my generated .desktop files look much better now :).

So, it's 'broken' only for intltool version from 0.40.0 to 0.40.3. Maybe the xdt-autogen script could check for the size of intltool-merge.in and if it is 0, then skip patching as it won't succeed in any case.Not sure about the displayed warning, it's still valid for intltool 0.40.0-3 but it's wrong for 0.40.4 and newer.
But maybe it's not worth it at all.
So, we can close this report.
Comment 3 Brian J. Tarricone (not reading bugmail) 2008-10-11 18:58:07 CEST
Yep, I already made that modification in my local tree last night, just haven't committed it yet.  Let's leave this open to remind me...
Comment 4 Brian J. Tarricone (not reading bugmail) 2008-10-12 08:45:41 CEST
Ok, fix committed.  It won't try to patch for 0.40.4 or greater, and for lower 0.40.x releases it prints out a message saying it can't fix things.
Comment 5 Enrico Tröger editbugs 2008-10-12 15:18:13 CEST
Great!

Bug #4462

Reported by:
Enrico Tröger
Reported on: 2008-10-10
Last modified on: 2009-07-14

People

Assignee:
Brian J. Tarricone (not reading bugmail)
CC List:
1 user

Version

Attachments

Additional information