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

Buried headers and includes cleanup #25

Closed
mloskot opened this issue Dec 6, 2012 · 3 comments
Closed

Buried headers and includes cleanup #25

mloskot opened this issue Dec 6, 2012 · 3 comments

Comments

@mloskot
Copy link
Contributor

mloskot commented Dec 6, 2012

Long time ago, there was discussion on soci-users about cleaning up structure of SOCI headers and install headers buried in dedicated folder(s). Here is the original discussion soci-users: includes cleanup

It resulted in patch from @juliantaylor (patch/16 in tracker at SourceForge).
Then, @denisarnaud created branch to maintain the buried headers and submitted two pull requests:

Julian also seems to maintain his patch in a branche at https://github.com/juliantaylor/soci

The buried headers subject has been forgotten since. It would be a good idea to pick it up and release with SOCI 4.0.0, so let's discuss it and make the work happen.

New thread related to this ticket at soci-users: Buried headers and includes cleanup


Related ideas

  • This may be a good opportunity to switch headers extension from .h to .hpp and replace - with _ in all file names.
@hasselmm
Copy link
Contributor

hasselmm commented Feb 6, 2013

Can this please be done? It's pretty nasty that one has to add $prefix/include/soci to the header search path as this pollutes the search path with pretty generally named header files like "error.h", "session.h" or "version.h".

@mloskot
Copy link
Contributor Author

mloskot commented Feb 6, 2013

@hasselmm I certainly agree with you, though we've decided to issue the bugfix release of SOCI 3.2.0 first. This will happen this month yet. Then, I'm getting my hands on preparing SOCI 4.0.0 with new source tree and installation layout.

Here you can read short rationale: release plans and here quick is on Preparing for SOCI 3.2.0 release.

@mloskot
Copy link
Contributor Author

mloskot commented Jan 20, 2014

I'm happy to announce, this has been implemented as part of #125

NOTE: There still may be CMake configuration updates missing (i.e. around install targets), please open new issues or wait until it's caught and fixed in develop branch, before releasing SOCI 4.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants