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

Update translations for bitcoin core #1089

Merged
merged 1 commit into from
Apr 14, 2012

Conversation

laanwj
Copy link
Member

@laanwj laanwj commented Apr 13, 2012

  • Update translations for bitcoin core
  • Move scripts/qt to share/qt, to clean up toplevel directories
  • Update english ts file which is used to source messages for Transifex
  • In extract_strings_qt.py use a glob *.h *.cpp, this is safe now that the Wx UI files are removed

This will not break any current translations, but will allow new messages to be translated.

@laanwj laanwj mentioned this pull request Apr 13, 2012
@Diapolo
Copy link

Diapolo commented Apr 13, 2012

Im fine with wthis change in general (ACK). If it get's merged #1000 (yay ^^) can be closed, too!

@laanwj
Copy link
Member Author

laanwj commented Apr 13, 2012

I'll re-run with -no-obsolete and change Hide to lowercase.

I don't have control over which plural forms are generated.

@Diapolo
Copy link

Diapolo commented Apr 13, 2012

Great!

I'm very sure the plural forms (english base) need to be put in the .ts
directly (manual).

Am 13.04.2012 15:03, schrieb Wladimir J. van der Laan:

I'll re-run with -no-obsolete and change Hide to lowercase.

I don't have control over which plural forms are generated.


Reply to this email directly or view it on GitHub:
#1089 (comment)

@laanwj
Copy link
Member Author

laanwj commented Apr 13, 2012

Can you point to the documentation on that? It sounds unlikely; TS files are supposed to be generated automatically from the source code.
(And plural forms differ per language. They cannot be sourced practically from English)

@Diapolo
Copy link

Diapolo commented Apr 13, 2012

Look at the other plurals or numerusform tags in the en .ts file. I'm sure someone placed the translations for the existing numerusform tags in there.

It seems this can be done with Qt Linguist application (open the en .ts and search for %n second(s) ago in BitcoinGUI), as the existing translations where plural forms are used are displayed with a yellow question mark and the english translation text IS filled in for them, right?

A doc is here, but I'm not sure if that's mentioned:
http://doc.qt.nokia.com/qq/qq19-plurals.html

@Diapolo
Copy link

Diapolo commented Apr 13, 2012

It seems Qt Linguist is the tool of choice :) ... you can even say that a translation is finished, which marks the string with a green check + removes the type="unfinished" tag for finished translations (that's for english master file - translation here means you looked through, added plural forms and marked strings as finished), cool!

@laanwj
Copy link
Member Author

laanwj commented Apr 13, 2012

Ok, you're right. I misunderstood. The _en needs to be manually updated to make the plural mechanism work in English, not to facilitate translations.

https://qt-project.org/doc/qt-4.8/linguist-programmers.html (under "Handling Plural Forms")

@laanwj
Copy link
Member Author

laanwj commented Apr 13, 2012

Fixed; and also marked the plural forms as "finished".

@Diapolo
Copy link

Diapolo commented Apr 13, 2012

Looks pretty good, so ACK!

@Diapolo
Copy link

Diapolo commented Apr 13, 2012

@laanwj 1 more thing I observed while playing around in Qt Linguist, we should set the base language of our en .ts file to only english (currently set to country USA). This can be done under Edit -> Translation File Settings...:

http://i42.tinypic.com/fp5irm.jpg

- Move scripts/qt to share/qt, to clean up toplevel directories
- Update english ts file which is used to source messages for Transifex
- In extract_strings_qt.py use a glob *.h *.cpp, this is safe now that the Wx UI files are removed
@laanwj
Copy link
Member Author

laanwj commented Apr 14, 2012

Ok, fixed too...

laanwj added a commit that referenced this pull request Apr 14, 2012
@laanwj laanwj merged commit 46ba858 into bitcoin:master Apr 14, 2012
coblee pushed a commit to litecoin-project/litecoin that referenced this pull request Jul 17, 2012
@laanwj laanwj deleted the 2012_04_translationupdate branch April 9, 2014 14:31
suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this pull request Dec 5, 2017
* PS: should check for prevout to match inputs

* fix CDarkSendEntry constructor
dagurval pushed a commit to dagurval/bitcoin that referenced this pull request May 27, 2018
* Set the minining max block size in excessiveblock_test.cpp

This makes sure we're starting at the right miningsize in case
it was somehow set to a different value in a previous test.

* fix transaction_tests.cpp

assigne the value of 83 to maxdatacarrierbytes in case it was changed
prior to running this set of tests.

* Fix excessive.py so that it works after the may15th fork

* fix mempool_reorg.py to work after fork activation.

* Fix the may152018_forkactivation_2.py

Since the fork time has passed we need to explicity set the pre
fork time when we mined the first set of blocks.

* small fix for mempool_reorg
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants