Merge pull request #24 from chris-ha458/fix_make

Fix make
allenai · Aug 22, 2023 · 02ebf87 · 02ebf87
2 parents b158299 + ac6cf0f
commit 02ebf87
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -23,7 +23,7 @@ setup:
 	$(shell "${PROTOBUF_SETUP}")
 	$(shell "${OPENSSL_SETUP}")
 	which cargo || curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
-	which maturin || pip install maturin
+	which maturin || pip install maturin[patchelf]
 
 publish:
 	maturin publish
@@ -32,7 +32,7 @@ test-python:
 	pytest -vs tests/python
 	rm -rf tests/work/*
 
-test-rust: test-rust-clean test-rust-setup
+test-rust:
 	cargo test -- --nocapture
 	rm -rf tests/work/*
 

diff --git a/README.md b/README.md
@@ -92,6 +92,12 @@ make test
 
 You can choose to run just the Python or Rust tests by calling `make test-python` or `make test-rust` respectively.
 
+## Contributing
+
+Before committing, use the following command
+```shell
+make style
+```
 
 ## Citation
 

diff --git a/pyproject.toml b/pyproject.toml
@@ -107,7 +107,7 @@ dev = [
 ]
 [build-system]
 requires = [
-    "maturin>=1.1,<2.0",
+    "maturin[patchelf]>=1.1,<2.0",
     "setuptools >= 61.0.0",
     "wheel"
 ]