Updated Mac OS Install Documentation #2392
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Due to security reasons Homebrew has removed direct formula references, this is the workaround.
Arusekk
approved these changes
Apr 18, 2024
|
Thank you! Just for clarification - this for a M1 chip MacBook Pro? There were several people having trouble installing pwntools on the Discord recently. I don't think we need entries in the changelog for changes to the documentation? @Arusekk |
|
M3, I assume the point is more so that it's Apple Silicone :) Edit: Checked with my M1 Mac mini, same issue/fix |
|
Can you revert the changes to the CHANGELOG.md please? I can't push to your fork due to duplicate |
|
All done |
|
Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated the documentation for installing Pwntools on Mac OS.
Firstly, Homebrew no longer supports direct references for brew install "for security reasons". Simple enough fix, just wget the correct .rb and then install from file.
Second issue, there were two requirements missing that had to be installed before I could pip install Pwntools - cmake and pkg-config. I'm coming from a new/fresh set up MacBook Pro, which didn't have either of these installed and I can't see any mention of them in the documentation.