Skip to content

ARROW-858: Remove boost_regex from arrow dependencies#567

Closed
cpcloud wants to merge 1 commit intoapache:masterfrom
cpcloud:decimal-no-regex
Closed

ARROW-858: Remove boost_regex from arrow dependencies#567
cpcloud wants to merge 1 commit intoapache:masterfrom
cpcloud:decimal-no-regex

Conversation

@cpcloud
Copy link
Copy Markdown
Contributor

@cpcloud cpcloud commented Apr 19, 2017

No description provided.

@wesm
Copy link
Copy Markdown
Member

wesm commented Apr 19, 2017

cc @robertnishihara @pcmoritz -- sorry to back track on this but using regexen in decimals seemed like more trouble than it was worth. @cpcloud could you open a JIRA for this? Thanks

@cpcloud
Copy link
Copy Markdown
Contributor Author

cpcloud commented Apr 19, 2017

@cpcloud
Copy link
Copy Markdown
Contributor Author

cpcloud commented Apr 19, 2017

Now to get builds passing.

@cpcloud cpcloud changed the title Remove boost_regex from arrow dependencies ARROW-858: Remove boost_regex from arrow dependencies Apr 19, 2017
Copy link
Copy Markdown
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

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

+1, thanks! Here's the passing Appveyor build: https://ci.appveyor.com/project/cpcloud/arrow/build/1.0.44

@asfgit asfgit closed this in 74f89cf Apr 19, 2017
jeffknupp pushed a commit to jeffknupp/arrow that referenced this pull request Jun 3, 2017
Author: Phillip Cloud <cpcloud@gmail.com>

Closes apache#567 from cpcloud/decimal-no-regex and squashes the following commits:

b5c59bd [Phillip Cloud] ARROW-858: Remove boost_regex from arrow dependencies
@cpcloud cpcloud deleted the decimal-no-regex branch June 23, 2017 04:24
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
…pache#567)

Bumps [io.netty:netty-bom](https://github.com/netty/netty) from
4.1.115.Final to 4.1.117.Final.

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants