Skip to content

Fix build under linux/gcc #2116

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

Merged
merged 1 commit into from
Nov 23, 2018
Merged

Fix build under linux/gcc #2116

merged 1 commit into from
Nov 23, 2018

Conversation

rsgowman
Copy link
Member

grpc_stream_tester.cc relies on f:f:model::DatabaseId, but wasn't
depending on it within the cc_library f_f_remote_test_util rule, causing
the build to fail. (Presumably, a mac specific library indirectly pulls
this in.)

grpc_stream_tester.cc relies on f:f:model::DatabaseId, but wasn't
depending on it within the cc_library f_f_remote_test_util rule, causing
the build to fail. (Presumably, a mac specific library indirectly pulls
this in.)
Copy link
Contributor

@var-const var-const left a comment

Choose a reason for hiding this comment

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

Thanks!

@rsgowman rsgowman merged commit b0a4b6c into master Nov 23, 2018
@rsgowman rsgowman deleted the fix_build_grpc branch November 23, 2018 21:50
bstpierr added a commit that referenced this pull request Nov 27, 2018
bstpierr added a commit that referenced this pull request Nov 27, 2018
* Revert "Custom fdl domain (#2121)"

This reverts commit ebea1ef.

* Revert "enable disabled passed integration test (#2120)"

This reverts commit 48e5f7a.

* Revert "Wire together LRU GC (#1905)"

This reverts commit 305f872.

* Revert "Fix build under linux/gcc (#2116)"

This reverts commit b0a4b6c.

* Revert "Implement PatchMutation::ApplyToLocalView (#1973)"

This reverts commit f66b5b5.

* Revert "Add Firebase Source to Header Search Path (#2114)"

This reverts commit 0540361.
@firebase firebase locked and limited conversation to collaborators Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants