-
-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
searchtool help: build issue with several translations #210
Comments
@rbuj |
For building mate-utils at fedora build server i needed to disable |
@raveit65 I have no problem to build it with ca@valencia enabled on F30 [robert@localhost devel]$ cat /etc/fedora-release [robert@localhost devel]$ cd mate-utils (edit) [robert@localhost mate-utils]$ rpm -q -i itstool | grep "Source RPM" |
@rbuj |
Yes, these are definitely the same bug. I have created itstool issue #36 and just now I've added a bisection. |
I retested it again and i can't reproduce the issue with ca@valencia any more when building master branch for f28/29/30.
Seems this is a random issue. |
|
Yes, but this needs to be fixed at transifex for those languages. |
@raveit65 |
@rbuj |
@raveit65 tx pull -a -f
./autogen.sh && make
find . -name "index.docbook" -exec yelp-check validate {} \;
cd logview/help
LC_ALL=en_US.UTF-8 find * -maxdepth 0 -type d \( ! -name . \) -exec posieve --coloring-type=none --include-name={} -c check-docbook4 -s showmsg ./ \;
cd ../../gsearchtool/help
LC_ALL=en_US.UTF-8 find * -maxdepth 0 -type d \( ! -name . \) -exec posieve --coloring-type=none --include-name={} -c check-docbook4 -s showmsg ./ \;
cd ../../baobab/help
LC_ALL=en_US.UTF-8 find * -maxdepth 0 -type d \( ! -name . \) -exec posieve --coloring-type=none --include-name={} -c check-docbook4 -s showmsg ./ \;
cd ../../mate-dictionary/help
LC_ALL=en_US.UTF-8 find * -maxdepth 0 -type d \( ! -name . \) -exec posieve --coloring-type=none --include-name={} -c check-docbook4 -s showmsg ./ \; |
I'm able to build mate-utils-1.20.2 on slackware-current with itstools-2.0.5 here |
It happened again today and bugreporting tool created a alarm. https://bugzilla.redhat.com/show_bug.cgi?id=1677838
@willysr This is a random issue.
Curiously compiling didn't stop at this point when i started |
I detected some issues... Most translator-credits don't follow the translation rule:
XML tags mismatch: To check that XML/HTML tags have not been translated:
|
For building mate-utils-1.22 at koji build server i had to disable pt language. |
@rbuj
Any idea what could be wrong with translations for pt language in help for searchtool? |
|
Master branch
fr.po
el.po
lt.po
|
1.22 branch
fr.po
el.po
lt.po
|
Looks like you found more issues, but build fails always with pt language. |
Strange thing is that i have no problem building this on Slackware (current) without any patches at all |
Any way i filed out an report. |
Problem string with Portuguese language is
So, i simply dropped the Portuguese translation string from transiflex. |
I am experiencing this when building 1.26.1 on NixOS (build log), sounds like 1.26.1 already have that pt trans string dropped. The failure is a flaky (random?) one for us. Right now we are giving the debian patch some try so far it seems to work (well since the original failure is quite random for us we cannot promise anything 🙃). |
This is still going on, I have to build with --disable-nls which is fine for my own use but if I put the package on archive.org it's not useful outside english-speaking areas. This is the error I am getting:
|
@lukefromdc and all others |
I have itstool from Debian and don't know enough about it to work on it. Hopefully one of these
will be merged, until then I will build with --disable-nls
|
You only need to rebuild debian itstools package with the commit from PR. |
I do very, very little building of Debian's divided-up packages and do NOT know how to build from Debian's scripts after modifying source code, which fails their integrity checks. All my .deb files are single source package ->single .deb package built with checkinstall or packed by setting an install path to a directory for meson. I have never sucessfully built using Debian's scripts with modified source. Thus I will have to wait for the patch unless I want to introduce more manual package handling to the next few updates |
Just had a successful build with nls enabled in Debian Unstable. Since itstool didn't get a recent update I don't know what changed |
Just to weigh in here with my findings. Using itstool-2.0.7 on a Gentoo system we are finding with 1.26.1: GCC13 and 14 compiles fine with no issue. When testing with 1.27.0 there is no issue at all which is slightly strange at the moment as it has thrown me off wondering if fa26078 was the commit that solved it as a first glance of the issue. We are tracking this at https://bugs.gentoo.org/920530 however I will make sure to share anything of interest that our users report. |
Expected behavior
user-guide translations from ca@valencia language shouldn't break building with itstool-2.0.5
Actual behavior
Building with itstool-2.0.5 breaks building the package
Merge issue of ca@valencia language is probably caused by a xml error in
[email protected]
file.Means, that needs to be fixed at transifex.
Building with older itstool-2.0.2 from fedora 28 doesn't cause the build error
For the moment i have disabled the translating help for this language to fix new releases.
e57612b
Steps to reproduce the behavior
./configure && make
MATE general version
master and 1.20 branch
Package version
master and 1.20 branch
Linux Distribution
tested with fedora 29
Link to downstream report of your Distribution
last pull of translations from transifex aren't release.
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: