Skip to content

Conversation

codler
Copy link
Contributor

@codler codler commented Jun 11, 2011

No description provided.

@jgarzik
Copy link
Contributor

jgarzik commented Jun 11, 2011

Can you redo this pull request, and delete the *.mo files please? Thanks.

@codler
Copy link
Contributor Author

codler commented Jun 11, 2011

I removed mo now, why do it need to be removed?

@jgarzik
Copy link
Contributor

jgarzik commented Jun 11, 2011

Please rebase your commits, rather than add a new one. We do not want to add two commits to the bitcoin history, one undoing 50% of the last commit :)

Why remove *.mo? It is a binary file, impossible to review. Your *.po could be correct, but an attacker may rewrite *.mo to add their own messages.

@codler
Copy link
Contributor Author

codler commented Jun 11, 2011

I hope it is right now, In my fork repo I used

git reset --hard HEAD@{2}

and commited, then

git push -f

jgarzik pushed a commit that referenced this pull request Jun 11, 2011
Added Swedish translation
@jgarzik jgarzik merged commit 18e3930 into bitcoin:master Jun 11, 2011
classesjack pushed a commit to classesjack/bitcoin that referenced this pull request Jan 2, 2018
Extend staker to properly prioritize contract transactions
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Oct 16, 2019
Update init.cpp: add missing semicolon to fix compilation with MinGW
@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.

2 participants