Skip to content
This repository has been archived by the owner on Mar 28, 2022. It is now read-only.

Bump version of stout-eventuals, fix local build #18

Merged
merged 1 commit into from
Sep 27, 2021

Conversation

while-false
Copy link
Contributor

@while-false while-false commented Sep 24, 2021

Update stout-eventuals in light of recent changes, in particular the
inclusion of Post and Get that relies on curl. The inclusion of curl
introduces two new repo dependencies included in this commit.

As outlined in issue #17, building :grpc locally using gcc is
currently broken by .bazelrc. This commit works around it by removing
the offending compiler flags, allowing the library to be built locally.
This fixes local compilation but it does not fix the issue: we want those
compiler flags in there where possible, i.e., for clang.

@while-false while-false force-pushed the while-false.update_stout_eventuals branch from a2f8da3 to 3401b58 Compare September 24, 2021 11:56
Update stout-eventuals in light of recent changes, in particular the
inclusion of Post and Get that relies on curl. The inclusion of curl
introduces two new repo dependencies included in this commit.

As outlined in issue #17, building `:grpc` locally using `gcc` is
currently broken by `.bazelrc`. This commit works around it by removing
the offending compiler flags, allowing the library to be built locally.
This fixes local compilation but it does not fix the issue: we want
those compiler flags in there where possible, i.e., for `clang`.
@while-false while-false force-pushed the while-false.update_stout_eventuals branch from 3401b58 to d96d19b Compare September 24, 2021 15:29
Copy link
Contributor

@rjhuijsman rjhuijsman left a comment

Choose a reason for hiding this comment

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

Well, that's just, like, your local build man. 😉

@while-false while-false merged commit ba8f6d2 into master Sep 27, 2021
@while-false while-false deleted the while-false.update_stout_eventuals branch September 27, 2021 08:42
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.

2 participants