bump-revision: fix for new license syntax

[Rylan12]

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same change?
• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes? Here's an example.
• Have you successfully run brew style with your changes locally?
• Have you successfully run brew tests with your changes locally?

---

Fix bump-revision to work with the new license syntax

bump-revision can now handle license strings and license hashes (I didn't add support for arrays because they are deprecated as of 2.5.0)

I've tested with the following licenses:

license "MIT"

license :public_domain

license "MIT" => { with: "LLVM-exception" }

license any_of: ["MIT", "0BSD"]

license all_of: ["MIT", "0BSD"]

license any_of: ["MIT", all_of: ["0BSD", "Zlib"]]

license any_of: ["GPL-2.0-or-later", "MIT" => { with: "LLVM-exception" }, any_of: ["MIT", "0BSD"]]

[scpeters]

thanks for the quick fix!

have you tested with a symbol like the following:

license :public_domain

• https://github.com/Homebrew/homebrew-core/blob/master/Formula/fmdiff.rb#L6

[Rylan12]

Nope! Good catch. Just pushed a new commit to account for it.

[MikeMcQuaid]

The else case seems a bit hacky/error-prone but doesn't seem any worse than what was there before. Thanks!

[Rylan12]

Agree. I'm open to suggestions on other ways to do it if desired.

[MikeMcQuaid]

Think this is going to be another "use the parser gem" situation.

[scpeters]

thanks again @Rylan12!