Skip to content

Added a disclaimer so the user is aware of binary#3751

Merged
ebuchman merged 2 commits intotendermint:developfrom
tac0turtle:marko/add_disclaimer-s-w
Jun 24, 2019
Merged

Added a disclaimer so the user is aware of binary#3751
ebuchman merged 2 commits intotendermint:developfrom
tac0turtle:marko/add_disclaimer-s-w

Conversation

@tac0turtle
Copy link
Contributor

  • added disclaimer for -s -w

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- added disclaimer for `-s -w`

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
@codecov-io
Copy link

codecov-io commented Jun 24, 2019

Codecov Report

Merging #3751 into develop will increase coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #3751      +/-   ##
===========================================
+ Coverage    63.92%   64.06%   +0.14%     
===========================================
  Files          242      242              
  Lines        20001    19996       -5     
===========================================
+ Hits         12785    12811      +26     
+ Misses        6165     6147      -18     
+ Partials      1051     1038      -13
Impacted Files Coverage Δ
privval/signer_service_endpoint.go 83.63% <0%> (-1.82%) ⬇️
p2p/conn/connection.go 80.85% <0%> (+0.26%) ⬆️
blockchain/pool.go 80.59% <0%> (+0.32%) ⬆️
consensus/replay.go 71.02% <0%> (+0.81%) ⬆️
consensus/state.go 80.56% <0%> (+0.82%) ⬆️
consensus/reactor.go 72.56% <0%> (+0.93%) ⬆️
blockchain/store.go 95.78% <0%> (+1.29%) ⬆️
privval/signer_remote.go 82% <0%> (+2%) ⬆️
privval/socket_listeners.go 89.65% <0%> (+3.44%) ⬆️
libs/db/util.go 72.72% <0%> (+3.49%) ⬆️
... and 1 more

tendermint node --proxy_app=kvstore
```

_DISCLAIMER_ The binary of tendermint is build/installed without the DWARF symbol table. If you would like to build/install tendermint with the DWARF symbol and debug information, remove `-s -w` from `BUILD_FLAGS` in the make file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably be under Install section?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moved it up to ### Compile section

@ebuchman ebuchman merged commit e93e8e7 into tendermint:develop Jun 24, 2019
unclezoro pushed a commit to unclezoro/tendermint that referenced this pull request Sep 6, 2019
* Added a disaclaimer so the user is aware of binary

- added disclaimer for `-s -w`

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>

* move up to compile
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