Add open and close methods for NcFile #18
Merged
Conversation
This file contains 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
|
Thanks for the issue report and patch; I will review soon and merge if there aren't any problems. Thanks! |
|
Any news on this ? |
|
Thanks for the ping; I've made the merge and am currently adding some tests to our CI system. I will push the update back to github later today. Thanks again, sorry for the delay! |
|
Thank you for merging this ! I see that you are currently porting the build system to CMake, and I need to use netcdf-cxx as part of a Cmake based-build. So if you need some help to make to CMake build fully usable, let me know! |
netbsd-srcmastr
pushed a commit
to NetBSD/pkgsrc
that referenced
this pull request
Jun 3, 2023
\Brief Release notes file for the netcdf-cxx4 package. This file contains a high-level description of this package's evolution. Releases are in reverse chronological order (most recent first). Note that this file was created and maintained starting with the `netcdf-cxx4 4.3.0 release`. ## netCDF-CXX4 4.3.2 TBD ## netCDF-CXX4 v4.3.1 September 11, 2019 ### Requirements * netCDF-C 4.6.0 or greater ### Changes * [Enhancement] Added cmake support to distribution files created by `make dist`. * [Bug Fix] Added `ncFile::create()`, also added a new `open` function and constructor to allow for more flexibility when opening a file. See [GitHub #55](Unidata/netcdf-cxx4#55) for more information. * [Enhancement] Addressed an issue reported with `Intel Compilers 17.0.0`. See [GitHub #41](Unidata/netcdf-cxx4#41) for more information. * [Enhancement] Updated how `configure` determines information about the existing netCDF-C install. See [GitHub pull request #39](Unidata/netcdf-cxx4#39) for more information. * Corrected an issue where cmake-based builds weren't generating `ncxx4-config`. See [GitHub pull request #37](Unidata/netcdf-cxx4#37) for more information. ## netcdf-cxx4 v4.3.0 released May 13, 2016 * Fixed an issue where the tests were failing silently, and the underlying `NcType` class could not properly determine the type name or type size reliably. See [GitHub issue #30](Unidata/netcdf-cxx4#30) for more information. * Changed `NCXX_ENABLE_DOXYGEN` option to an easier-to-remember `ENABLE_DOXYGEN`. * Added `--enable-doxygen`, `-DNCXX_ENABLE_DOXYGEN=ON` options to allow generation of netCDF-CXX4 documentation via doxygen using either `configure` or `cmake`, respectively. * Added `netcdf-cxx4` to the [Coverity Scan Dashboard](https://scan.coverity.com/projects/unidata-netcdf-cxx4?tab=overview). * Added `open` and `close` methods for NcFile. See [Github Pull Request #18](Unidata/netcdf-cxx4#18) for more information. * Added `netcdf-cxx4` to travis-ci.org. See [https://travis-ci.org/Unidata/netcdf-cxx4](https://travis-ci.org/Unidata/netcdf-cxx4) for more details. * Added `NcCompoundType` methods `getMemberName` and `getMemberIndex`. See [Pull Request #19](Unidata/netcdf-cxx4#19) for more details. * Added `cmake` support to `netcdf-cxx4`, which will allow us to create a `CDash` continuous integration dashboard similar to those created for the `netcdf-c` and `netcdf-fortran` projects. * Added a `travis-ci` configuration file, `.travis.yml`. * Created `RELEASE_NOTES.md`.
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #17