This PR follows our Release Process.
It is required to open Transifex translations for v30.0, as scheduled in #32275.
For reference, see the previous similar PR: #31809.
Note for reviewers:
To reproduce the diff in the last commit, run:
0cmake --preset dev-mode
1cmake --build build_dev_mode --target translate
Update the Transifex slug to match the new resource created for the
upcoming 30.x branch.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33152.
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | laanwj, stickies-v |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
Could fix the typo in the source string (found by the llm)?
0diff --git a/src/wallet/feebumper.cpp b/src/wallet/feebumper.cpp
1index 82ea8ee1c9..0b56a23251 100644
2--- a/src/wallet/feebumper.cpp
3+++ b/src/wallet/feebumper.cpp
4@@ -84,7 +84,7 @@ static feebumper::Result CheckFeeRate(const CWallet& wallet, const CMutableTrans
5
6 std::optional<CAmount> combined_bump_fee = wallet.chain().calculateCombinedBumpFee(reused_inputs, newFeerate);
7 if (!combined_bump_fee.has_value()) {
8- errors.push_back(Untranslated(strprintf("Failed to calculate bump fees, because unconfirmed UTXOs depend on enormous cluster of unconfirmed transactions.")));
9+ errors.push_back(Untranslated(strprintf("Failed to calculate bump fees, because unconfirmed UTXOs depend on an enormous cluster of unconfirmed transactions.")));
10 }
11 CAmount new_total_fee = newFeerate.GetFee(maxTxSize) + combined_bump_fee.value();
12
13diff --git a/src/wallet/spend.cpp b/src/wallet/spend.cpp
14index 250377afcf..53a224b46b 100644
15--- a/src/wallet/spend.cpp
16+++ b/src/wallet/spend.cpp
17@@ -762,7 +762,7 @@ util::Result<SelectionResult> ChooseSelectionResult(interfaces::Chain& chain, co
18 }
19 std::optional<CAmount> combined_bump_fee = chain.calculateCombinedBumpFee(outpoints, coin_selection_params.m_effective_feerate);
20 if (!combined_bump_fee.has_value()) {
21- return util::Error{_("Failed to calculate bump fees, because unconfirmed UTXOs depend on enormous cluster of unconfirmed transactions.")};
22+ return util::Error{_("Failed to calculate bump fees, because unconfirmed UTXOs depend on an enormous cluster of unconfirmed transactions.")};
23 }
24 CAmount bump_fee_overestimate = summed_bump_fees - combined_bump_fee.value();
25 if (bump_fee_overestimate) {
26diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp
27index 91a494c379..f86a28c52e 100644
28--- a/src/wallet/wallet.cpp
29+++ b/src/wallet/wallet.cpp
30@@ -2847,7 +2847,7 @@ std::shared_ptr<CWallet> CWallet::Create(WalletContext& context, const std::stri
31 rescan_required = true;
32 } else if (nLoadWalletRet == DBErrors::UNKNOWN_DESCRIPTOR) {
33 error = strprintf(_("Unrecognized descriptor found. Loading wallet %s\n\n"
34- "The wallet might had been created on a newer version.\n"
35+ "The wallet might have been created on a newer version.\n"
36 "Please try running the latest software version.\n"), walletFile);
37 return nullptr;
38 } else if (nLoadWalletRet == DBErrors::UNEXPECTED_LEGACY_ENTRY) {
39@@ -3149,7 +3149,7 @@ bool CWallet::AttachChain(const std::shared_ptr<CWallet>& walletInstance, interf
40 // but fail the rescan with a generic error.
41
42 error = chain.havePruned() ?
43- _("Prune: last wallet synchronisation goes beyond pruned data. You need to -reindex (download the whole blockchain again in case of pruned node)") :
44+ _("Prune: last wallet synchronisation goes beyond pruned data. You need to -reindex (download the whole blockchain again in case of a pruned node)") :
45 strprintf(_(
46 "Error loading wallet. Wallet requires blocks to be downloaded, "
47 "and software does not currently support loading wallets while "
48diff --git a/src/wallet/walletdb.cpp b/src/wallet/walletdb.cpp
49index 2c0073f356..bc5bb0fa0b 100644
50--- a/src/wallet/walletdb.cpp
51+++ b/src/wallet/walletdb.cpp
52@@ -791,7 +791,7 @@ static DBErrors LoadDescriptorWalletRecords(CWallet* pwallet, DatabaseBatch& bat
53 value >> desc;
54 } catch (const std::ios_base::failure& e) {
55 strErr = strprintf("Error: Unrecognized descriptor found in wallet %s. ", pwallet->GetName());
56- strErr += (last_client > CLIENT_VERSION) ? "The wallet might had been created on a newer version. " :
57+ strErr += (last_client > CLIENT_VERSION) ? "The wallet might have been created on a newer version. " :
58 "The database might be corrupted or the software version is not compatible with one of your wallet descriptors. ";
59 strErr += "Please try running the latest software version";
60 // Also include error details
Could fix the typo in the source string (found by the llm)?
0diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp 1index 91a494c379..5aeb39283b 100644 2--- a/src/wallet/wallet.cpp 3+++ b/src/wallet/wallet.cpp 4@@ -3149,7 +3149,7 @@ bool CWallet::AttachChain(const std::shared_ptr<CWallet>& walletInstance, interf 5 // but fail the rescan with a generic error. 6 7 error = chain.havePruned() ? 8- _("Prune: last wallet synchronisation goes beyond pruned data. You need to -reindex (download the whole blockchain again in case of pruned node)") : 9+ _("Prune: last wallet synchronisation goes beyond pruned data. You need to -reindex (download the whole blockchain again in case of a pruned node)") : 10 strprintf(_( 11 "Error loading wallet. Wallet requires blocks to be downloaded, " 12 "and software does not currently support loading wallets while "
Thanks! Fixed.
Steps to reproduce the diff on Ubuntu 25.04:
```
cmake --preset dev-mode
cmake --build build_dev_mode --target translate
```
Could fix the typo in the source string (found by the llm)?
Should be done now.
Code review ACK 656e16aa5e65731a61b77444ac3ab874f7635c50
Last commit reproduces also on Ubuntu 24.04, with only harmless differences in C string splitting like
0"Outbound connections restricted to CJDNS (-onlynet=cjdns) but "
1"-cjdnsreachable is not provided"),
2"Outbound connections restricted to CJDNS (-onlynet=cjdns) but -"
3"cjdnsreachable is not provided"),
Milestone
30.0