Skip to content

Commit

Permalink
Static bin speed warning
Browse files Browse the repository at this point in the history
  • Loading branch information
tnballo committed Nov 16, 2020
1 parent ae00528 commit 0948f6a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
7 changes: 5 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,9 +94,9 @@ ARGS:
<FILE(S)>... 1+ binaries to gadget search. If > 1: gadgets common to all
```

### CLI Build and Install
### CLI Build and Install (Recommended)

Build from source and install locally:
Build a dynamically-linked binary from source and install it locally:

```bash
cargo install xgadget --features cli-bin # Build on host (pre-req: https://www.rust-lang.org/tools/install)
Expand All @@ -108,6 +108,9 @@ Commits to this repo's `master` branch automatically run integration tests and b
You can [download it here](https://github.com/entropic-security/xgadget/releases) and use the CLI immediately, instead of building from source.
Static binaries for Windows may also be supported in the future.

The statically-linked binary is about 8x slower, presumably due to the built-in memory allocator for target `x86_64-unknown-linux-musl`.
Building a dynamically-linked binary from source with the above `cargo install` command is *highly* recommended.

### ~~Yeah, but can it do 10 OS kernels under 10 seconds?!~~ Repeatable Benchmark Harness

```bash
Expand Down
7 changes: 5 additions & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,9 @@
//! <FILE(S)>... 1+ binaries to gadget search. If > 1: gadgets common to all
//!```
//!
//!### CLI Build and Install
//!### CLI Build and Install (Recommended)
//!
//!Build from source and install locally:
//!Build a dynamically-linked binary from source and install it locally:
//!
//!```bash
//!cargo install xgadget --features cli-bin # Build on host (pre-req: https://www.rust-lang.org/tools/install)
Expand All @@ -105,6 +105,9 @@
//!You can [download it here](https://github.com/entropic-security/xgadget/releases) and use the CLI immediately, instead of building from source.
//!Static binaries for Windows may also be supported in the future.
//!
//!The statically-linked binary is about 8x slower, presumably due to the built-in memory allocator for target `x86_64-unknown-linux-musl`.
//!Building a dynamically-linked binary from source with the above `cargo install` command is *highly* recommended.
//!
//!### ~~Yeah, but can it do 10 OS kernels under 10 seconds?!~~ Repeatable Benchmark Harness
//!
//!```bash
Expand Down

0 comments on commit 0948f6a

Please sign in to comment.