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.

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

Missed ./acceptor/EvbHandshakeHelper.cpp in CMakeLists.txt #113

Closed
wants to merge 1 commit into from

Conversation

capveg
Copy link

@capveg capveg commented Sep 8, 2017

@mingtaoy 's recent change broken open source wangle, which broke open source fbthrift, which broke open source fboss :-( This patch fixes it.

FYI: @eduardo-elizondo

@capveg
Copy link
Author

capveg commented Sep 8, 2017

this was merged internally; closing.

@capveg capveg closed this Sep 8, 2017
facebook-github-bot pushed a commit that referenced this pull request Feb 20, 2022
Summary:
Pull Request resolved: facebook/sapling#113

GitHub Actions was failing at apt-get stage,  error message suggested adding an apt-get update as remedy.  Added in this diff and it indeed works.

The actions build fails later on missing mio::net module when building mysql async, but that's unrelated to this change.

Reviewed By: farnz

Differential Revision: D34368662

fbshipit-source-id: f0a00da3ee740ae4443a328616e792ea615c922c
dgrnbrg-meta added a commit to dgrnbrg-meta/wangle that referenced this pull request Mar 11, 2022
Summary:
X-link: facebook/fboss#113

Adds an environment variable to getdeps to provide `hg` info to avoid calling `hg` directly.

When using `getdeps` inside a containerized environment (which we need to build Research Super Cluster tooling with the correct linker attributes), `getdeps` fails because of unregistered mercurial extensions in the `hgrc`.

This allows `getdeps` to be useable in an environment where the mercurial extensions used in a project aren't installed/available.

Reviewed By: vivekspai

Differential Revision: D34732506

fbshipit-source-id: c7d753c0a4bcc33870a6019b2f2936d489595f3d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants