On 2014/02/10 22:34:00, mbruzek wrote: > Just a nit, that you might want to add so the script can be called > non-interactively
> https://codereview.appspot.com/60740045/diff/20001/tests/00_setup.sh > File tests/00_setup.sh (right):
https://codereview.appspot.com/60740045/diff/20001/tests/00_setup.sh#newcode8 > tests/00_setup.sh:8: sudo apt-get install amulet python3-requests > python-requests amulet python3 > Use the -y tag here to install non-interactively.
Patched and uploaded. Cleaned up by removing the Python2 Dependencies since all amulet tests are Python3 at the moment.
https://codereview.appspot.com/60740045/
« Back to merge proposal
On 2014/02/10 22:34:00, mbruzek wrote:
> Just a nit, that you might want to add so the script can be called
> non-interactively
> https:/ /codereview. appspot. com/60740045/ diff/20001/ tests/00_ setup.sh
> File tests/00_setup.sh (right):
https:/ /codereview. appspot. com/60740045/ diff/20001/ tests/00_ setup.sh# newcode8 setup.sh: 8: sudo apt-get install amulet python3-requests
> tests/00_
> python-requests amulet python3
> Use the -y tag here to install non-interactively.
Patched and uploaded. Cleaned up by removing the Python2 Dependencies
since all amulet tests are Python3 at the moment.
https:/ /codereview. appspot. com/60740045/