Bug 1249051 - intltool-merge sometimes generate translations sometimes not
intltool-merge sometimes generate translations sometimes not
Status: ASSIGNED
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: intltool (Show other bugs)
7.2
Unspecified Unspecified
unspecified Severity unspecified
: rc
: ---
Assigned To: Colin Walters
ecs-bugs
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2015-07-31 08:40 EDT by Lukáš Nykrýn
Modified: 2017-07-01 19:57 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
0001-intltool-merge-Create-cache-file-atomically.patch (1.12 KB, patch)
2015-10-08 16:27 EDT, Colin Walters
no flags Details | Diff
0001-build-Error-out-if-intltool-failed.patch (816 bytes, patch)
2015-10-08 18:20 EDT, Colin Walters
no flags Details | Diff

  None (edit)
Comment 3 Colin Walters 2015-10-08 15:39:57 EDT
Is this a new issue?  intltool hasn't changed in approximately forever.

Do you know if it's affecting Fedora?

Upstream is pretty dead, and now that modern gettext supports most file formats (including the XML here), no one wants to use intltool anymore.

That said, I don't think a rebase of gettext is in the cards, so we can see what we can do to fix this.

There's a change in Fedora to fix
https://bugzilla.redhat.com/show_bug.cgi?id=1233444
I wonder if it's related to the Perl version in the buildroot or something like that?
Comment 4 Colin Walters 2015-10-08 16:14:13 EDT
Between the logs in
http://download.devel.redhat.com/brewroot/packages/systemd/219/8.el7/data/logs/x86_64/build.log
http://download.devel.redhat.com/brewroot/packages/systemd/219/8.el7/data/logs/x86_64/build.log

Hmm.  One thing I notice is that the `sub create_cache` function in intltool-merge is not done as an atomic write, yet systemd's build system has multiple parallel invocations of intltool.

What I would do for now is patch systemd's build to fail if there are no translations.
Comment 5 Colin Walters 2015-10-08 16:27 EDT
Created attachment 1081098 [details]
0001-intltool-merge-Create-cache-file-atomically.patch

Untested patch.
Comment 6 Colin Walters 2015-10-08 18:18:58 EDT
(In reply to Colin Walters from comment #5)
> Created attachment 1081098 [details]
> 0001-intltool-merge-Create-cache-file-atomically.patch
> 
> Untested patch.

Confirmed the patch doesn't break the build at least.
Comment 7 Colin Walters 2015-10-08 18:20 EDT
Created attachment 1081108 [details]
0001-build-Error-out-if-intltool-failed.patch

Tested this doesn't break the build, did not test that it correctly failed if intltool failed.
Comment 8 Lukáš Nykrýn 2015-10-12 04:42:42 EDT
(In reply to Colin Walters from comment #3)
> Is this a new issue?  intltool hasn't changed in approximately forever.
> 
> Do you know if it's affecting Fedora?

Yep, we have the same problem in fedora.
Comment 9 Colin Walters 2015-10-12 10:44:32 EDT
(In reply to Lukáš Nykrýn from comment #8)
> (In reply to Colin Walters from comment #3)
> > Is this a new issue?  intltool hasn't changed in approximately forever.
> > 
> > Do you know if it's affecting Fedora?
> 
> Yep, we have the same problem in fedora.

OK, I pushed the patch there:

http://pkgs.fedoraproject.org/cgit/intltool.git/commit/?id=f0b91355afb6b9d0bdc52c489b8b6a1ce4e78ab4

Note You need to log in before you can comment on or make changes to this bug.