hebasto
commented at 1:05 PM on May 17, 2021:
member
This PR removes multiple "Qualifying with unknown namespace/class" warnings in make -C src translate output.
Also all of the lupdate options are moved before input files (as documented).
The remaining warnings are fixed in Qt 5.12.2 (see QTBUG-42736).
fanquake added the label Build system on May 17, 2021
hebasto
commented at 1:21 PM on May 17, 2021:
member
Not seeing any difference in make translate output here:
Sorry for asking, but did you do ./autogen.sh && ./configure after switching to this branch from the master one?
laanwj
commented at 6:03 PM on May 17, 2021:
member
No, but I did verify that the lupdate command line changed (see the diff in previous message). Shouldn't that be enough?
hebasto
commented at 6:51 PM on May 17, 2021:
member
No, but I did verify that the lupdate command line changed (see the diff in previous message). Shouldn't that be enough?
It seems I misunderstand you.
If "output" is the result of make translate in generated and modified files then "Not seeing any difference" is expected.
If "output" are messages printed by make translate to console -- "Not seeing any difference" is not what expected as warnings should gone.
That's why I was confused by your comment. Sorry for that.
jarolrod
commented at 1:09 AM on May 18, 2021:
member
ACKaa9702b3ae937e72b8c96fb9b12f2db75f7cffc5
This silences the warnings for me on macOS 11.3 clang 12.0.5, these warnings appear harmless.
laanwj
commented at 11:05 AM on May 18, 2021:
member
I'm really spooked.
Could it have to do with something else, like the order of the input files? Can anyone post their lupdate command line from make V=1 translate please? (with this PR)
hebasto
commented at 11:12 AM on May 18, 2021:
member
laanwj
commented at 12:32 PM on May 18, 2021:
member
I figured it out but it is pretty strange: It depends on what directory it is run from. I am using out-of-tree builds so in my case, lupdate is run from an external build directory with absolute paths for all its input files. Say, you have the following script lupdate.sh:
# from the source tree
$ cd …/bitcoin/src
$ lupdate.sh ..
…/bitcoin/src/qt/guiutil.cpp:767: Unterminated C++ character
…/bitcoin/src/qt/guiutil.cpp:770: Unterminated C++ character
…/bitcoin/src/qt/guiutil.cpp:771: Unterminated C++ character
…/bitcoin/src/qt/guiutil.cpp:774: Unterminated C++ character
…/src/qt/guiutil.cpp:767: Unbalanced opening parenthesis in C++ code (or abuse of the C++ preprocessor)
Updating 'qt/locale/bitcoin_en.ts'...
Found 1048 source text(s) (0 new and 1048 already existing)
# any other directory
$ cd ../build/src
$ lupdate.sh /full/path/to/bitcoin
…/bitcoin/src/qt/bantablemodel.cpp:85: Qualifying with unknown namespace/class ::BanTableModel
…/bitcoin/src/qt/bitcoin.cpp:421: Qualifying with unknown namespace/class ::BitcoinApplication
…/bitcoin/src/qt/bitcoin.cpp:431: Qualifying with unknown namespace/class ::BitcoinApplication
…/bitcoin/src/qt/bitcoingui.cpp:247: Qualifying with unknown namespace/class ::BitcoinGUI
…/bitcoin/src/qt/bitcoingui.cpp:457: Qualifying with unknown namespace/class ::BitcoinGUI
…/bitcoin/src/qt/bitcoingui.cpp:547: Qualifying with unknown namespace/class ::BitcoinGUI
⋮ (lots more!)
Updating '…/bitcoin/src/qt/locale/bitcoin_en.ts'...
Found 1048 source text(s) (0 new and 1048 already existing)
So I guess the good news is that I can confirm that the change in this PR works, given you're not doing an out-of-tree build :smile:
I don't think it matters much: the generated ts is exactly the same.
hebasto force-pushed on May 18, 2021
hebasto
commented at 2:59 PM on May 18, 2021:
member
I figured it out but it is pretty strange: It depends on what directory it is run from. I am using out-of-tree builds so in my case, lupdate is run from an external build directory with absolute paths for all its input files.
This is a metadata mirror of the GitHub repository
bitcoin/bitcoin.
This site is not affiliated with GitHub.
Content is generated from a GitHub metadata backup.
generated: 2026-04-21 18:14 UTC
This site is hosted by @0xB10C More mirrored repositories can be found on mirror.b10c.me