[CXX-921] legacy driver fails to build on FreeBSD Created: 07/Jun/16 Updated: 21/Jun/16 Resolved: 21/Jun/16 |
|
| Status: | Closed |
| Project: | C++ Driver |
| Component/s: | Build, Portability |
| Affects Version/s: | legacy-1.1.1 |
| Fix Version/s: | legacy-1.1.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Giulio Paci | Assignee: | J Rassi |
| Resolution: | Done | Votes: | 0 |
| Labels: | legacy-cxx | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
Trying to compile mongo-cxx-driver on FreeBSD fails with error The OpenBSD implementation for SecureRandom should work on FreeBSD as well (not checked, as I have no direct access to a FreeBSD installation at the moment). You can see a complete log of the failure here: |
| Comments |
| Comment by Githook User [ 21/Jun/16 ] |
|
Author: {u'username': u'giuliopaci', u'name': u'Giulio Paci', u'email': u'giuliopaci@gmail.com'}Message: Signed-off-by: J. Rassi <rassi@10gen.com> |
| Comment by David Golden [ 13/Jun/16 ] |
|
Thanks! |
| Comment by Giulio Paci [ 13/Jun/16 ] |
|
This also did not work (apparently this macro is for old kernels) however in https://jira.mongodb.org/browse/CXX-924 I attached a patch that makes compilation work. |
| Comment by David Golden [ 09/Jun/16 ] |
|
Apologies for my bad Jira markup – to clarify, that define should be __FreeBSD__. |
| Comment by David Golden [ 09/Jun/16 ] |
|
That error suggests the compiler macro for FreeBSD is incorrect. According to this page, it should probably be __FreeBSD__. If you change this line to check __FreeBSD__, does it compile? If not, what errors do you get? |
| Comment by Andrew Morrow (Inactive) [ 07/Jun/16 ] |
|
Thanks for the bug report. I suspect that you are right that the fix is straightforward. Unfortunately, we do no testing on either OpenBSD or FreeBSD. My recommendation is that you create a pull request with the necessary FreeBSD changes to get it to build and we can start from there. |