Skip to content

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

Closed
wants to merge 1 commit into from
Closed

ghidra 9.0 #37656

wants to merge 1 commit into from

Conversation

mrjefftang
Copy link

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

@Moisan
Copy link
Member

Moisan commented Mar 7, 2019

@BrewTestBot test this please

@fxcoudert fxcoudert added the new formula PR adds a new formula to Homebrew/homebrew-core label Mar 7, 2019
@ajd394
Copy link

ajd394 commented Mar 10, 2019

@mrjefftang
Copy link
Author

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.

@ajd394
Copy link

ajd394 commented Mar 11, 2019

@mrjefftang
I think that the "no binary" requirement is that the formula be able to be built from source. I also think the open source requirement suggest that all source required to build must be available. Can it be built from source with the source provided?

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

@mrjefftang
Copy link
Author

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

Our policy is that formulae in the core tap (homebrew/core) must be open-source with an OSI-approved license and either built from source or produce cross-platform binaries (e.g. Java, Mono). Binary-only formulae should go to homebrew/cask.

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.

@Gaelan
Copy link

Gaelan commented Mar 16, 2019

IIRC part of Ghidra is written in C++ that isn't open source yet. Maybe wait on that, cask for now?

@SMillerDev
Copy link
Member

Casks don't require an app file but core does require every part to be open source. This will have to be a cask.

@SMillerDev SMillerDev closed this Mar 26, 2019
@ajd394
Copy link

ajd394 commented Apr 9, 2019

i think the new release 9.0.2 now open sourced the whole thing

@lock lock bot added the outdated PR was locked due to age label May 9, 2019
@lock lock bot locked as resolved and limited conversation to collaborators May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new formula PR adds a new formula to Homebrew/homebrew-core outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants