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

Add doc, fix output bugs #1769

Merged
merged 3 commits into from
Jul 9, 2020
Merged

Add doc, fix output bugs #1769

merged 3 commits into from
Jul 9, 2020

Conversation

feliam
Copy link
Contributor

@feliam feliam commented Jul 6, 2020

Readme updated to link to manticore-verifier documentation
Typos and output cleanups

@feliam feliam requested a review from ehennenfent July 6, 2020 15:46
@@ -9,6 +9,7 @@ Manticore is a symbolic execution tool for analysis of binaries and smart contra
:maxdepth: 2
:caption: Contents:

verifier
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This may need to be CLI-tools
And then inside that we should have manpage-like documentation on how to use the commandline tools for each tool. (So far we have plain manticore and manticore-verifier )

Copy link
Contributor

@ehennenfent ehennenfent left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I suggested a few spelling fixes, but overall the docs looks like a pretty comprehensive introduction to the verifier. If we intend for this to be used externally, I think it'd also be worth adding a section to the README with a sentence or two about manticore-verifier and a link to the documentation on RTD.

manticore/ethereum/verifier.py Outdated Show resolved Hide resolved
manticore/ethereum/verifier.py Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
docs/verifier.rst Outdated Show resolved Hide resolved
Co-authored-by: Eric Hennenfent <eric.hennenfent@trailofbits.com>
@ehennenfent ehennenfent added this to In progress in Manticore Jul 7, 2020
@feliam feliam requested a review from ehennenfent July 7, 2020 19:31
Copy link
Contributor

@ehennenfent ehennenfent left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for adding the README link.

Manticore automation moved this from In progress to Reviewer approved Jul 9, 2020
@ehennenfent ehennenfent merged commit da51351 into master Jul 9, 2020
Manticore automation moved this from Reviewer approved to Done Jul 9, 2020
@ehennenfent ehennenfent deleted the dev-verifier-doc branch July 9, 2020 19:55
ekilmer added a commit that referenced this pull request Jul 17, 2020
* master:
  Create a model for strncpy (#1770)
  Add doc, fix output bugs (#1769)
  Update EVM usage example (#1772)
  New working model of strlen (#1725)
  Typo (#1768)
  Specialized iterative serialization for Array (#1756)
  Enable nightly uploads to PyPI (#1757)
  Manticore 0.3.4 (#1720)
  Manticore verifier (#1717)
  Nightly MacOS Tests (#1614)
  Remove/procrastinate solver query in ether leak detector (#1727)
  Fix constant folding & constraint set slicing (#1706)
ekilmer added a commit that referenced this pull request Aug 5, 2020
* master:
  Change the default to threading (#1779)
  Coveralls Take 2 (#1784)
  Capture return values in run.sh (#1776)
  Create a model for strncpy (#1770)
  Add doc, fix output bugs (#1769)
@ehennenfent ehennenfent added this to the Manticore 0.3.5 milestone Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Manticore
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants