-
-
Notifications
You must be signed in to change notification settings - Fork 12.9k
ghidra 9.0 #37656
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ghidra 9.0 #37656
Conversation
@BrewTestBot test this please |
Ghidra is not open source. Does that mean it has to be a cask? |
Most of the source is included in the zip file available for Ghidra. They're all tagged as -src.zip. A few components have only been released as a binary. |
@mrjefftang I would like to see this tool get merged as much as you but it does not appear to meet the acceptable formula requirements. https://docs.brew.sh/Acceptable-Formulae#we-dont-like-binary-formulae |
This project falls somewhere in between formula and cask. It doesn't fit 1 definition or the other since it doesn't build a ".app" for Casks. From Acceptable-Formulae
The app is open-source but not built from source but it does produce cross-platform binaries in the form of Java JARs. The source is in the master zip file, it just hasn't been uploaded as a git repo to GitHub which will change. So it's going to be silly to convert this back to a formula down the line. |
IIRC part of Ghidra is written in C++ that isn't open source yet. Maybe wait on that, cask for now? |
Casks don't require an app file but core does require every part to be open source. This will have to be a cask. |
i think the new release 9.0.2 now open sourced the whole thing |
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install <formula>
)?