Skip to content

Commit

Permalink
Bump to CheckoutV2 (#1654)
Browse files Browse the repository at this point in the history
* Bump version of checkout script and some Python packages

Unicorn, black, and mypy

* Fix typing issue with potential missing file descriptor
  • Loading branch information
Eric Hennenfent authored Apr 6, 2020
1 parent 20c146f commit e28168f
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
- name: Set up Python 3.6
uses: actions/setup-python@v1
with:
Expand Down
10 changes: 8 additions & 2 deletions manticore/core/smtlib/solver.py
Original file line number Diff line number Diff line change
Expand Up @@ -300,8 +300,14 @@ def _send(self, cmd: str):
"""
# logger.debug('>%s', cmd)
try:
self._proc.stdout.flush()
self._proc.stdin.write(f"{cmd}\n")
if self._proc.stdout:
self._proc.stdout.flush()
else:
raise SolverError("Could not flush stdout: file descriptor is None")
if self._proc.stdin:
self._proc.stdin.write(f"{cmd}\n")
else:
raise SolverError("Could not write to stdin: file descriptor is None")
except IOError as e:
raise SolverError(str(e))

Expand Down
4 changes: 2 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,9 @@ def rtd_dependent_deps():

# If you update native_deps please update the `REQUIREMENTS_TO_IMPORTS` dict in `utils/install_helper.py`
# (we need to know how to import a given native dependency so we can check if native dependencies are installed)
native_deps = ["capstone==4.0.1", "pyelftools", "unicorn==1.0.2rc1"]
native_deps = ["capstone==4.0.1", "pyelftools", "unicorn==1.0.2rc2"]

lint_deps = ["black==19.3b0", "mypy==0.740"]
lint_deps = ["black==19.10b0", "mypy==0.770"]

# Development dependencies without keystone
dev_noks = (
Expand Down

0 comments on commit e28168f

Please sign in to comment.