RfidResearchGroup / homebrew-proxmark3

Homebrew tap containing proxmark3 software/firmware
MIT License
44 stars 19 forks source link

Support additional build options #22

Closed cynix closed 3 years ago

cynix commented 3 years ago

Allow specifying build options when installing, e.g.:

$ brew install --HEAD --with-generic --with-small --without-standalone --without-hitag proxmark3
==> Installing proxmark3 from cynix/proxmark3
==> Cloning https://github.com/RfidResearchGroup/proxmark3.git
Updating /Users/cynix/Library/Caches/Homebrew/proxmark3--git
==> Checking out branch master
Already on 'master'
Your branch is up to date with 'origin/master'.
HEAD is now at 30fc94da4 Merge pull request #1275 from tharexde/fix_4x50_standalone
==> make clean
==> make all BREW_PREFIX=/usr/local PLATFORM=PM3GENERIC PLATFORM_SIZE=256 SKIP_HITAG=1 STANDALONE=
==> make install PREFIX=/usr/local/Cellar/proxmark3/HEAD-30fc94d BREW_PREFIX=/usr/local PLATFORM=PM3GENERIC PLATFORM_SIZE=256 SKIP_HITAG=1 STANDALONE=
==> Install success!
==> The latest bootloader and firmware binaries are ready and waiting in the current homebrew Cellar within share/firmware.
🍺  /usr/local/Cellar/proxmark3/HEAD-30fc94d: 680 files, 41.3MB, built in 1 minute 45 seconds
iceman1001 commented 3 years ago

Interesting change.

The readme.md will need some update to match the new options suggested here.

cynix commented 3 years ago

The readme.md will need some update to match the new options suggested here.

Updated :)

The motivation for adding these as install options is that Homebrew stores them in INSTALL_RECEIPT.json and will automatically apply them when upgrading, so the user doesn't need to specify them manually each time.

iceman1001 commented 3 years ago

Perfect!

@tcprst, @bettse might wanna have test run before, otherwise I will merge.