Merge pull request #517 from lukemartinlogan/master

Update readme

README.md

@@ -72,6 +72,14 @@ After successfully building Hermes, it's a good idea to run the test suite via t
 
 Finally, install the library with `make install`. You can find a complete example build script [here](https://github.com/HDFGroup/hermes/blob/master/ci/install_hermes.sh).
 
+Before running ctests, you must install jarvis-util, which is used to execute
+the tests.
+```
+cd $HERMES_ROOT/ci/jarvis-util
+python3 -m pip install -r requirements.txt
+python3 -m pip install -e .
+```
+
 ## Contributing
 
 We follow the [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html). You can run `make lint` to ensure that your code conforms to the style. This requires the `cpplint` Python module (`pip install cpplint`). Alternatively, you can let the CI build inform you of required style changes.

ci/hermes/packages/hermes/package.py

@@ -16,6 +16,9 @@ class Hermes(CMakePackage):
     depends_on('mpich@3.3.2:')
     depends_on('yaml-cpp')
     depends_on('boost@1.7:')
+    depends_on('python3')
+    # TODO(): we need to figure out how to add a python library as a dependency
+    # depends_on('jarvis-util')
     depends_on('libfabric@1.14.1 fabrics=mlx,rxd,rxm,shm,sockets,tcp,udp,verbs,xpmem',
                when='+ares')
     depends_on('hdf5@1.13.0:', when='+vfd')