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

Use FOLLY_DEPRECATED rather than __attribute__((__deprecated__)) #113

Merged
merged 1 commit into from
Jan 21, 2016

Conversation

Orvid
Copy link
Contributor

@Orvid Orvid commented Aug 4, 2015

Because MSVC doesn't support __attribute__ modifiers.

Because MSVC doesn't support `__attribute__` modifiers.
juchem added a commit that referenced this pull request Jan 21, 2016
Use FOLLY_DEPRECATED rather than __attribute__((__deprecated__))
@juchem juchem merged commit 00a6cc5 into facebook:master Jan 21, 2016
juchem pushed a commit that referenced this pull request Jan 22, 2016
This reverts commit 00a6cc5, reversing
changes made to 7c71d6f.
ghost pushed a commit that referenced this pull request Feb 2, 2016
Summary:
Because MSVC doesn't support `__attribute__` modifiers.
Closes #113

Reviewed By: juchem

Differential Revision: D2853366

Pulled By: Orvid

fb-gh-sync-id: dcdb841b1c5eee6461796d81ddf3b204a2ba4450
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
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.

3 participants