singrep
singrep copied to clipboard
Add build/install/test options in makefile
- Add Makefile
- Update Authors
- Readme updates
Thank you Christo!
Some changes please:
- Can you change the Makefile's
build
to usecargo build --release
to generate release rather than debug builds. - On my install
cargo install --path .
installed to/Users/<user>/.cargo/bin/singrep
it would be good to make the clear in the README. - Please put installation instructions in the README under a heading "Installation", and make the specific install steps sub headings (##).
- Please add a
clean
directive to the Makefile withcargo clean
Some questions:
- Why are you using
CARGO_NET_GIT_FETCH_WITH_CLI
I'm not aware of any special authentication requirements? - What is the purpose of the
'libc'
in thecargo run
directive? libc is included as a crate.