Skip to content
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

mateweather doc build segmentation fautl with itstool (With Template) #388

Closed
jmrcpn opened this issue Mar 16, 2019 · 6 comments
Closed

Comments

@jmrcpn
Copy link

jmrcpn commented Mar 16, 2019

Expected behaviour

Successful Build

Actual behaviour

Crash
segmentation fault itstool -m "${mo}" ${d}/C/index.docbook ${d}/C/legal.xml

The problem was pin-point to be within
mate-applets-1.22.0/mateweather/docs/ru/ru.po
replacing ru.po by uk/uk.po (Ukrainian, roughly the same alphabet), make everything compiled.
With a naked eye, the ru.po seems to be quite normal (not a Russian speaking).
I do not believe it is a compilation workbench problem (itstool libraries), as all other foreign documentation (fr,it,zh,zu,etc.) are done without trouble.

Steps to reproduce the behaviour

Build (configure + make)

MATE general version

1.22.0

Package version

1.22.0

Linux Distribution

OSUKISS

Link to downstream report of your Distribution

I am the distribution builder

@raveit65
Copy link
Member

Thanks for honor the template.
It's a known issue with newer itstools.
see mate-desktop/mate-utils#210
itstool/itstool#36
and caused by itstool/itstool@d3adf02
You can try starting the build again, sometimes this helps and you will be lucky after starting it 4-8 times.
For fedora i use this workaround to disable the culprit language in make file, as i am building for different architectures and it fails always.
https://src.fedoraproject.org/cgit/rpms/mate-utils.git/commit/

@jmrcpn
Copy link
Author

jmrcpn commented Mar 16, 2019 via email

@raveit65
Copy link
Member

Sorry, we can`t help. Problem isn't in .pot file, as i said it's a bug in itstools with using parallel builds.
Please read links and follow my suggestion to disable the language.
Or patch itstools, simply revert itstool/itstool@d3adf02
I am waiting for myself of a reaction by itstools developer since 2 months.

@jmrcpn
Copy link
Author

jmrcpn commented Mar 16, 2019 via email

rbuj added a commit that referenced this issue Mar 16, 2019
rbuj added a commit that referenced this issue Mar 16, 2019
@jmrcpn
Copy link
Author

jmrcpn commented Mar 17, 2019

Many Thanks rubj, extracted the 3 ru.po from your commit and the build is now going flawless...
Please, Make sure your "ru-xml" branch will be merged with master.
Note: So itstool is unable to recover to a very simple parsing problem/anomaly....
:) itstool team!?... there is room for improvement ;)

@rbuj
Copy link
Contributor

rbuj commented Mar 17, 2019

f3654da closes the issue in master branch.
c2db442 closes the issue in 1.22 branch.
de7e447 closes the issue in 1.20 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants