[CXX-87] bsondemo.cpp won't build standalone anymore Created: 24/Sep/13 Updated: 18/Feb/15 Resolved: 12/Mar/14 |
|
| Status: | Closed |
| Project: | C++ Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Minor - P4 |
| Reporter: | Dwight Merriman | Assignee: | Unassigned |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | cxxmove, legacy-cxx, neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
comments in the source file indicate you can do a one-line g++ statement. however 1) the include requires (-I) are now broader and as specified wrong; i suppose if there are more grandiose plans for c++ client & bson lib cleanups and such in the meantime the build instruction comment could be eliminated, #3 resolved, and #5 done, and vcxproj files deleted? as it wouldn't make sense to release it broken. but fixing would be easy too. |
| Comments |
| Comment by Andrew Morrow (Inactive) [ 12/Mar/14 ] | |||||||||||
|
A standalone BSON library is not a current goal of the project. | |||||||||||
| Comment by Andrew Morrow (Inactive) [ 10/Mar/14 ] | |||||||||||
|
dwight_10gen It is not clear right now whether we are aiming to support header only use of BSON with the driver as currently constructed or not, so I'm putting this in 'features we aren't sure of' for now. However, the independent repo does build and run the bsondemo example, so we are at least now confirming that it builds and runs. | |||||||||||
| Comment by Dwight Merriman [ 25/Feb/14 ] | |||||||||||
|
it doesn't look like it would throw:
| |||||||||||
| Comment by Tyler Brock [ 08/Oct/13 ] | |||||||||||
|
I got this to compile using:
It throws a "wrong type for field" exception that I wouldn't expect given the comments in the code:
Perhaps we changed how lower case vals() works? Either way, I think its probably bad that the BSON demo needs dbclient.h to compile. Shouldn't this be self contained to simply requiring bson.h? I would be happy to explore further with sverch if others feel the same way. |