Skip to content

Replace boost::regex library with std::regex#6997

Closed
djuanit0x wants to merge 2 commits intoargotorg:developfrom
djuanit0x:stdregex
Closed

Replace boost::regex library with std::regex#6997
djuanit0x wants to merge 2 commits intoargotorg:developfrom
djuanit0x:stdregex

Conversation

@djuanit0x
Copy link
Copy Markdown

Description

I address #6971

@chriseth
Copy link
Copy Markdown
Contributor

There still seem to be some problems here. Did you try to compile locally?

@djuanit0x
Copy link
Copy Markdown
Author

Nope, I keep getting an error when running ./scripts/tests.sh. The Failing CI logs, however, consistently produce these errors error: no matching function for call to ‘regex_replace. After reading this question, It appears that the compiler should be updated to GCC 4.9 to remove the errors.

@chriseth
Copy link
Copy Markdown
Contributor

It looks like stdc++ does not support regex_replace where the replacement is performed by a lambda function. So you would have to write some glue code. This might be more complicated than I initially though - sorry about that!

@djuanit0x
Copy link
Copy Markdown
Author

No problem:smile:. I am closing this PR, since I have some troubles with the testing.

@djuanit0x djuanit0x closed this Jun 29, 2019
@chriseth
Copy link
Copy Markdown
Contributor

chriseth commented Jul 1, 2019

@djuanit0x thanks a lot for trying!

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