Commit Graph

3263 Commits

Author SHA1 Message Date
Ward Fisher
08e8374b02 Tentative fix for initial issue at http://github.com/Unidata/netcdf-c/issues/221, investigating knock-on issues now. 2016-02-18 14:45:57 -07:00
Ward Fisher
e2eda755e3 Tweaked empty test to check for limited, unlimited dimensions. Also added a gdb command file for ease of debugging. 2016-02-18 13:59:53 -07:00
Ward Fisher
791800145a Added test file for debugging issue. 2016-02-18 17:33:49 +00:00
Ward Fisher
845f3b2282 Merge branch 'gspatch8' 2016-02-17 14:20:03 -07:00
Ward Fisher
ecc635014c Merge branch 'patch-8' of https://github.com/gsjaardema/netcdf-c into gspatch8 2016-02-17 12:40:20 -07:00
Ward Fisher
e9f07bd681 Merge branch 'patch-7' of https://github.com/gsjaardema/netcdf-c into gsjaardema-patch-7 2016-02-16 15:34:26 -07:00
Ward Fisher
0eff434098 Renamed test file to be more reflective for what it is doing. 2016-02-16 14:54:10 -07:00
Ward Fisher
4ccad07a6b Have duplicated issue reported in #221 at github. 2016-02-16 14:52:03 -07:00
Ward Fisher
654a6e39e3 Additional work debugging issue. 2016-02-16 14:40:09 -07:00
Ward Fisher
1807cb7aa2 Roughing in a test to create the failure described in https://github.com/Unidata/netcdf-c/issues/221 2016-02-16 11:00:09 -07:00
Ward Fisher
47dcd00bb7 Added test file for debugging https://github.com/Unidata/netcdf-c/issues/221. 2016-02-16 10:23:53 -07:00
Ward Fisher
7e7fa31b5f Merge pull request #215 from gsjaardema/patch-5
Support user-specified hdf5 and hdf5_hl libraries
2016-02-16 10:19:15 -07:00
Greg Sjaardema
2d3278c58b Update comment
If pnetcdf support is enabled, then  NC_HAS_PARALLEL is defined to 1 even if HDF5 is not enabled.  Fix comment to make this clearer.
2016-02-10 15:07:19 -07:00
Greg Sjaardema
e8280efcb2 Generate correct results from cmake-generated nc-config
A couple of the `has_XX` variables seem to have values of OFF or ON instead of `empty` or ON which causes incorrect output from the nc-config script since it is using `-z` to test whether the variable is non-empty.  This causes nc-config to incorrectly report that the library supports HDF5 or has the version 2 api.
2016-02-10 15:03:19 -07:00
Greg Sjaardema
ebc25610ef Merge branch 'master' into patch-5 2016-02-09 13:43:06 -07:00
Greg Sjaardema
126a319230 Fix version dependency.
Required PNetCDF version is 1.6.1 or greater; not 1.6.0 as I previously thought.

Note that this is pull request may (partially) satisfy Issue #194 which I just found.
2016-02-09 11:23:33 -07:00
Greg Sjaardema
d123157771 Enable PNetCDF settings in cmake build
One possible solution to finding the PNetCDF library and testing for the correct version in the cmake build.
2016-02-09 07:23:09 -07:00
Greg Sjaardema
edfffb4b14 Merge branch 'master' into patch-5 2016-02-09 07:05:54 -07:00
Ward Fisher
1a42d4ff99 Corrected version string to reflect development branch is 4.4.1 in support of correcting https://github.com/Unidata/netcdf-c/issues/214. 2016-02-08 17:36:04 -07:00
Greg Sjaardema
df58364f00 Support user-specified hdf5 and hdf5_hl libraries
If the netcdf cmake build is configured with explicitly specified hdf5 c and hl libraries:
```
    -DHDF5_C_LIBRARY:PATH=${ACCESS}/lib/libhdf5.dylib \
    -DHDF5_HL_LIBRARY:PATH=${ACCESS}/lib/libhdf5_hl.dylib \
    -DHDF5_INCLUDE_DIR:PATH=${ACCESS}/include \
```
Then it looks like the HDF5_C_LIBRARIES and HDF5_HL_LIBRARIES variables do not get set and then this causes the HDF5 libraries to be omitted from the `TLL_LIBS` symbol in liblib/CMakeLists.txt.  This is subsequently used to build `ALL_TLL_LIBS` which is used for `LIBS` which is output as the `Extra libraries:        @LIBS@` in libnetcdf.settings.in 

The output for `-- Linking against:` is also incorrect as it omits the hdf5 libraries:   Output without suggested change:
```
-- Linking against:      /usr/lib/libdl.dylib;/usr/lib/libm.dylib;/opt/local/lib/libz.dylib;/opt/local/lib/libcurl.dylib
(later in file)
Extra libraries:	-ldl -lm -lz -lcurl
```
versus output with suggested change:
```
-- Linking against:      /Users/gdsjaar/src/seacas-parallel/lib/libhdf5_hl.dylib;/Users/gdsjaar/src/seacas-parallel/lib/libhdf5.dylib;/usr/lib/libdl.dylib;/usr/lib/libm.dylib;/opt/local/lib/libz.dylib;/opt/local/lib/libcurl.dylib
(later in file)
Extra libraries:	-lhdf5_hl -lhdf5 -ldl -lm -lz -lcurl
```
This also causes a link failure when building libnetcdf.dylib
2016-02-08 17:20:48 -07:00
Greg Sjaardema
8de0d10803 Merge remote-tracking branch 'upstream/master' 2016-02-08 11:04:45 -07:00
Ward Fisher
55b24e2964 Added tst_mode to CMakeLists.txt 2016-02-04 19:12:15 +00:00
Ward Fisher
03b7a103ad Merge pull request #210 from Unidata/gh206
merging work @dmh did to address #206
2016-02-03 10:47:01 -07:00
Dennis Heimbigner
e03785069a Merge branch 'gh206' of https://github.com/Unidata/netcdf-c into gh206 2016-02-02 13:07:15 -07:00
Dennis Heimbigner
66fcf79d77 Github issue: https://github.com/Unidata/netcdf-c/issues/206
Re e-eupport VGQ-678069
It was noticed that an attribute value of "nan." was being treated as
legal (it should be "nan"). The reason is that sscanf was not be checked
to see that all the attribute value characters were being read.
Solution is to verify that all characters were being consumed.
2016-02-02 13:06:57 -07:00
Ward Fisher
20ce7edf32 Merge pull request #209 from Unidata/gh208
Fix github issue: https://github.com/Unidata/netcdf-c/issues/208
2016-02-02 12:42:00 -07:00
Dennis Heimbigner
80a07f2375 Merge branch 'gh206' of https://github.com/Unidata/netcdf-c into gh206 2016-02-01 16:21:32 -07:00
Dennis Heimbigner
b2a25bde13 Github issue: https://github.com/Unidata/netcdf-c/issues/206
Re e-eupport VGQ-678069
It was noticed that an attribute value of "nan." was being treated as
legal (it should be "nan"). The reason is that sscanf was not be checked
to see that all the attribute value characters were being read.
Solution is to verify that all characters were being consumed.
2016-02-01 16:19:34 -07:00
Dennis Heimbigner
45572f5971 Fix github issue: https://github.com/Unidata/netcdf-c/issues/208
Return an error when specifying deflation (compression) or fletcher32 on
a file created for parallel IO in netcdf-4.
2016-02-01 16:15:58 -07:00
Ward Fisher
990296c671 Merge pull request #205 from Unidata/hdf5init
Clean up the handling of hdf5 initialization
2016-01-29 11:07:54 -07:00
Ward Fisher
7c828bdb5e Merge branch 'master' into hdf5init 2016-01-29 10:31:52 -07:00
Ward Fisher
83df40b6f9 Merge pull request #204 from Unidata/contributing-toc
Added a table-of-contents to the contributing guideline file.
2016-01-29 10:12:43 -07:00
Dennis Heimbigner
d91ccdf54c Github issue: https://github.com/Unidata/netcdf-c/issues/206
Re e-eupport VGQ-678069
It was noticed that an attribute value of "nan." was being treated as
legal (it should be "nan"). The reason is that sscanf was not be checked
to see that all the attribute value characters were being read.
Solution is to verify that all characters were being consumed.
2016-01-28 22:18:51 -07:00
Dennis Heimbigner
b5ba424793 Clean up the handling of hdf5 initialization by
creating an nc4_hdf5_initialize(void) function
plus nc4_hdf5_initialized flag.
Also fix potential null exception in nc4internal.c
2016-01-28 16:19:38 -07:00
Ward Fisher
5b18f955dc Added a table-of-contents to the contributing guideline file. 2016-01-27 15:56:34 -07:00
Ward Fisher
f85d8a0ead Merge pull request #203 from Unidata/contributing
Add contributing guideline document based on `contributing.html` file added by @dmh.
2016-01-27 15:49:14 -07:00
Ward Fisher
93560f7555 Updated contributing markdown document. 2016-01-27 14:29:58 -07:00
Ward Fisher
bfb18e4f57 First draft of contributing.md 2016-01-26 16:06:19 -07:00
Ward Fisher
e81f5ed7d6 Adding a contributing guideline file. 2016-01-26 15:23:04 -07:00
Greg Sjaardema
e967eda145 Merge remote-tracking branch 'upstream/master' 2016-01-26 11:04:43 -07:00
Ward Fisher
427577be8d Merge branch 'gsjaardema-patch-6' into pull-consolidation 2016-01-26 10:10:54 -07:00
Ward Fisher
30e70df5b6 Merge branch 'gsjaardema-patch-5' into pull-consolidation 2016-01-26 10:09:51 -07:00
Ward Fisher
cf9a477daa Modified spacing in configure.ac. 2016-01-26 10:09:14 -07:00
Greg Sjaardema
e1696187ec Replace NC_MAX_DIMS with NC_MAX_VAR_DIMS
Same issue as in #197.
2016-01-25 14:55:29 -07:00
Greg Sjaardema
ba7d9c54bd Replace NC_MAX_DIMS with NC_TESTS_MAX_DIMS
Missed this file earlier somehow.  Same situation as fixed in fbc210f
2016-01-25 14:47:52 -07:00
Greg Sjaardema
12c94a2668 Merge remote-tracking branch 'upstream/master' 2016-01-25 14:39:35 -07:00
Ward Fisher
fbc210f511 Alternative solution to https://github.com/Unidata/netcdf-c/pull/199 and https://github.com/Unidata/netcdf-c/pull/200. 2016-01-25 14:09:03 -07:00
Ward Fisher
f3cae2e6ce Merge branch 'master' of https://github.com/Unidata/netcdf-c 2016-01-25 11:29:12 -07:00
Ward Fisher
3769de2669 Merge branch 'gsjaardema-patch-2' 2016-01-25 11:10:32 -07:00
Ward Fisher
dc78c6a4a5 Merge pull request #197 from gsjaardema/patch-1
Replace NC_MAX_DIMS with correct NC_MAX_VAR_DIMS
2016-01-25 10:23:13 -07:00