FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Provide error on gcc < 4.1 when THREADSAFE by ethomson · Pull Request #4135 · libgit2/libgit2 · GitHub

Repository navigation

Provide error on gcc < 4.1 when THREADSAFE - #4135

Merged
carlosmn merged 1 commit into
masterfrom
ethomson/ancient_gcc_not_threadsafe
Feb 22, 2017
Merged

carlosmn merged 1 commit into
masterfrom
ethomson/ancient_gcc_not_threadsafe

Conversation

ethomson commented Feb 22, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

Provide a descriptive error message when compiling THREADSAFE on gcc versions < 4.1. We require the atomic primitives (eg __sync_synchronize) that were introduced in that version.

(Note, clang setes __GNUC__ but appears to set its version > 4.1.)

Copy link
Copy Markdown
Member

(Note, lldb setes GNUC but appears to set its version > 4.1.)

Do you mean llvm/clang? The debugger surely isn't setting the defines.

Provide a descriptive error message when compiling THREADSAFE on gcc
versions < 4.1.  We require the atomic primitives (eg
`__sync_synchronize`) that were introduced in that version.

(Note, clang setes `__GNUC__` but appears to set its version > 4.1.)
ethomson force-pushed the ethomson/ancient_gcc_not_threadsafe branch from 57ac11c to aa5ea47 Compare February 22, 2017 11:22

Copy link
Copy Markdown
Member Author

Do you mean llvm/clang? The debugger surely isn't setting the defines.

Yes, clearly. I've updated the commit message to reflect this.

Copy link
Copy Markdown
Member

The Travis build finished after 25m but it looks like it never sent out the notification.

carlosmn merged commit 86201b9 into master Feb 22, 2017
ethomson deleted the ethomson/ancient_gcc_not_threadsafe branch January 9, 2019 10:17
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL