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

include msgctxt when generating .po file #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

3dd13
Copy link

@3dd13 3dd13 commented Aug 8, 2011

I am using the function to parse, modify and print out .po file in my project, but I found out that context was not handled in po generation. so, this is my fix.

@nikosd
Copy link

nikosd commented Nov 5, 2011

@3dd13 are you sure about this? Can you give an example of what's not working? I know that you have added a test but I don't get 100% the case cause I'm not so familiar with the GetText::RMsgMerge::PoData api...

@3dd13
Copy link
Author

3dd13 commented Nov 5, 2011

@nikosd

sorry, my test case was not clear enough.
it was the msgctxt "Context" not being printed out.

the story was ... I parsed a .pot file and generate a .po file and found out that the msgctxt was missing in the output .po file.

@nikosd
Copy link

nikosd commented Nov 8, 2011

@3dd13 thanks for the clarification! I will try to co and run the test to cast my vote. This looks useful.

@kou
Copy link

kou commented May 24, 2012

I've merged those changes in ruby-gettext/gettext@3518c4d
Thanks!

fotos pushed a commit to skroutz/gettext that referenced this pull request Oct 29, 2012
rgettext ->
gettext

refs mutoh#12

Reported by mtasaka. Thanks!!!
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.

3 participants