Skip to content
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

rocksdb 4.5.1 #621

Closed
wants to merge 2 commits into from
Closed

rocksdb 4.5.1 #621

wants to merge 2 commits into from

Conversation

Jerzerak
Copy link
Contributor

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?
  • Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?
  • Does your submission pass brew audit --strict --online <formula> (after doing brew install <formula>)?

Description

Bump rocksdb stable version from 4.4.1 to 4.5.1

@apjanke
Copy link
Contributor

apjanke commented Apr 26, 2016

Test failure:

    Test Result (3 failures / +3)

    brew-test-bot.el_capitan.linkage --test osquery
    brew-test-bot.mavericks.linkage --test osquery
    brew-test-bot.yosemite.linkage --test osquery

failed: brew linkage --test osquery

Stacktrace

        Missing libraries:
  /usr/local/opt/rocksdb/lib/librocksdb.4.4.dylib

This means that osquery needs to have its revision bumped so it picks up the new rocksdb dylib version. You can add it to this PR in another commit.

Looks like this breakage actually happened in the previous rocksdb version bump; it's just being caught now because the linkage --test check was recently introduced.

@Jerzerak
Copy link
Contributor Author

@apjanke let me know if anything else needs doing

@@ -8,7 +8,7 @@ class Osquery < Formula

bottle do
cellar :any
revision 1
revision 2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bottle revision is different from the formula revision; the latter goes outside the bottle block just below the sha256. But it seems like it's not necessary here since the build is passing as-is.

@apjanke apjanke closed this in 3278618 Apr 26, 2016
@apjanke
Copy link
Contributor

apjanke commented Apr 26, 2016

You're good to go.

Merged. Thank you for your contribution to Homebrew!

For future reference, normally we bump the revision of the formula itself, not the bottle revision. Unlikely to matter here since osquery has no options. We're talking about clearing this terminology up: Homebrew/brew#38.

@Jerzerak
Copy link
Contributor Author

@apjanke Thanks! The documentation was a little confusing on that part so I went with what I thought would work based on some old PR's with revision bumps.

@Jerzerak Jerzerak deleted the update/rocksdb branch April 26, 2016 22:35
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants