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

lib: Texinfo: Translate partial menu node names. #437

Merged

Conversation

Apteryks
Copy link
Contributor

Fixes https://issues.guix.gnu.org/64881.

  • lib/Locale/Po4a/Texinfo.pm (translate_buffer_menuentry): Refine regexp, so that it matches menu entries lacking a description. Only call 'translate_buffer' on the description if it was provided.
  • t/fmt/texinfo/partialmenus.trans: New file.
  • t/fmt/texinfo/partialmenus.texi: Likewise.
  • t/fmt/texinfo/partialmenus.pot: Likewise.
  • t/fmt/texinfo/partialmenus.po: Likewise.
  • t/fmt/texinfo/partialmenus.norm: Likewise.
  • t/fmt-texinfo.t: Register the new 'partialmenus' test.

Fixes <https://issues.guix.gnu.org/64881>.

* lib/Locale/Po4a/Texinfo.pm (translate_buffer_menuentry): Refine
regexp, so that it matches menu entries lacking a description.
Only call 'translate_buffer' on the description if it was provided.
* t/fmt/texinfo/partialmenus.trans: New file.
* t/fmt/texinfo/partialmenus.texi: Likewise.
* t/fmt/texinfo/partialmenus.pot: Likewise.
* t/fmt/texinfo/partialmenus.po: Likewise.
* t/fmt/texinfo/partialmenus.norm: Likewise.
* t/fmt-texinfo.t: Register the new 'partialmenus' test.
trofi added a commit to trofi/nix-guix-gentoo that referenced this pull request Sep 9, 2023
Without the mquinson/po4a#437 change guix build
fails to build translated manpages.
@mquinson
Copy link
Owner

Thanks a lot, merged! Your help is particularly precious because I'm far from being able to update/fix the Texinfo module...

Just a small comment: please also update the NEWS file to document your change, and my pleasure will be perfect.

Thanks again,
Mt

@mquinson mquinson merged commit 51c726f into mquinson:master Oct 16, 2023
@Apteryks
Copy link
Contributor Author

Noted for future contributions, thanks for the review/merge!

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 this pull request may close these issues.

2 participants