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.

Sign up for GitHub

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

Jump to bottom

hyperscan 4.2.0 #2538

Closed
wants to merge 4 commits into from
Closed

hyperscan 4.2.0 #2538

wants to merge 4 commits into from

Conversation

flier
Copy link
Contributor

@flier flier commented Jul 1, 2016

  • Have you followed the guidelines in our Contributing document?
  • [ x 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>)?

@flier flier mentioned this pull request Jul 1, 2016
Closed
@flier
Copy link
Contributor Author

flier commented Jul 1, 2016

build failed at OSX mavericks with clang 6.0, submitted issue to upstream

intel/hyperscan#27

@tdsmith
Copy link
Contributor

tdsmith commented Jul 1, 2016

Please let us know if there's progress on that ticket.

@tdsmith tdsmith added the needs response Needs a response from the issue/PR author label Jul 1, 2016
@DomT4 DomT4 added the upstream issue An upstream issue report is needed label Jul 1, 2016
@UniqMartin UniqMartin changed the title upgrade to Hyperscan v4.2.0 hyperscan 4.2.0 Jul 6, 2016
@ilovezfs
Copy link
Contributor

@flier I think you can apply intel/hyperscan@e9cfbae as a patch rather than waiting for a new release as you were saying here intel/hyperscan#27 (comment)

@ghost ghost removed the needs response Needs a response from the issue/PR author label Jul 21, 2016
@flier
Copy link
Contributor Author

flier commented Jul 22, 2016

seems the patch passed all checks

@dunn
Copy link
Contributor

dunn commented Aug 7, 2016

@BrewTestBot test this please

@dunn dunn closed this in 93d6514 Aug 7, 2016
@dunn
Copy link
Contributor

dunn commented Aug 7, 2016

Thanks! Merged as cb25ddf, 6b97a35, and 93d6514.

@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.
Reviewers
No reviews
Assignees
No one assigned
Labels
upstream issue An upstream issue report is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@flier @tdsmith @ilovezfs @dunn @DomT4