Fix bug in run_newformat.sh

RELEASE_NOTES.md

@@ -7,7 +7,7 @@ This file contains a high-level description of this package's evolution. Release
 
 ## 4.9.3 - TBD
 
-* Provide better documentation for the .rc file mechanism and API. See [Github #????](https://github.com/Unidata/netcdf-c/pull/????) for more information.
+* Provide better documentation for the .rc file mechanism and API. See [Github #2956](https://github.com/Unidata/netcdf-c/pull/2956) for more information.
 * Convert NCZarr V2 to store all netcdf-4 specific info as attributes. This improves interoperability with other Zarr implementations by no longer using non-standard keys. The price to be paid is that lazy attribute reading cannot be supported. See [Github #2836](https://github.com/Unidata/netcdf-c/pull/2936) for more information.
 * Cleanup the option code for NETCDF_ENABLE_SET_LOG_LEVEL\[_FUNC\] See [Github #2931](https://github.com/Unidata/netcdf-c/pull/2931) for more information.
 * 

libnczarr/zsync.c

@@ -1513,7 +1513,7 @@ define_var1(NC_FILE_INFO_T* file, NC_GRP_INFO_T* grp, const char* varname)
  if(jvalue != NULL)
  var->storage = NC_CHUNKED;
  /* Extract dimrefs list */
- if((stat = dictgetalt(jncvar,"dimension_references","dimensions",&jdimrefs))) goto done;
+ if((stat = dictgetalt(jncvar,"dimension_references","dimrefs",&jdimrefs))) goto done;
  if(jdimrefs != NULL) { /* Extract the dimref names */
  assert((NCJsort(jdimrefs) == NCJ_ARRAY));
  if(zvar->scalar) {
@@ -1851,7 +1851,7 @@ ncz_read_superblock(NC_FILE_INFO_T* file, char** nczarrvp, char** zarrfp)
 
  if(jsuper != NULL) {
  if(jsuper->sort != NCJ_DICT) {stat = NC_ENCZARR; goto done;}
- if((stat = NCJdictget(jsuper,"version",&jtmp))<0) {stat = NC_EINVAL; goto done;}
+ if((stat = dictgetalt(jsuper,"nczarr_version","version",&jtmp))<0) {stat = NC_EINVAL; goto done;}
  nczarr_version = nulldup(NCJstring(jtmp));
  }
 
@@ -2572,14 +2572,13 @@ getnczarrkey(NC_OBJ* container, const char* name, const NCjson** jncxxxp)
  jxxx = NULL;
  if((stat = NCJdictget(zobj->atts,name,&jxxx))<0) {stat = NC_EINVAL; goto done;}
  }
- if(name == NULL) {
- jxxx = NULL;
+ if(jxxx == NULL) {
  /* Try .zxxx second */
  if(zobj->obj != NULL) {
  if((stat = NCJdictget(zobj->obj,name,&jxxx))<0) {stat = NC_EINVAL; goto done;}
  }
- /* Mark as old style with _nczarr_xxx in obj not attributes */
- zobj->nczv1 = 1;
+ if(jxxx != NULL)
+ zobj->nczv1 = 1; /* Mark as old style with _nczarr_xxx in obj not attributes */
  }
  if(jncxxxp) *jncxxxp = jxxx;
 done:

nc_test/Makefile.am

@@ -126,13 +126,13 @@ tst_http_nc3.cdl tst_http_nc4?.cdl tmp*.cdl tmp*.nc
 
 EXTRA_DIST += bad_cdf5_begin.nc run_cdf5.sh nc_enddef.cdl
 if NETCDF_ENABLE_CDF5
- # bad_cdf5_begin.nc is a corrupted CDF-5 file with bad variable starting
- # file offsets. It is to be used by tst_open_cdf5.c to check if it can
- # detect and report error code NC_ENOTNC.
- TESTS += run_cdf5.sh
- check_PROGRAMS += tst_open_cdf5
+# bad_cdf5_begin.nc is a corrupted CDF-5 file with bad variable starting
+# file offsets. It is to be used by tst_open_cdf5.c to check if it can
+# detect and report error code NC_ENOTNC.
+TESTS += run_cdf5.sh
+check_PROGRAMS += tst_open_cdf5
 if LARGE_FILE_TESTS
- TESTPROGRAMS += tst_large_cdf5 tst_cdf5_begin
+TESTPROGRAMS += tst_large_cdf5 tst_cdf5_begin
 endif
 endif
 

ncdap_test/Makefile.am

@@ -60,15 +60,15 @@ TESTS += tst_hyrax.sh
 TESTS += test_partvar
 
 # Various
- TESTS += tst_longremote3.sh
- tst_longremote3.log: tst_remote3.log
+TESTS += tst_longremote3.sh
+tst_longremote3.log: tst_remote3.log
 endif
 
 if NETCDF_ENABLE_DAP_LONG_TESTS
- test_manyurls_SOURCES = test_manyurls.c manyurls.h
- check_PROGRAMS += test_manyurls
- test_manyurls.log: tst_longremote3.log
- TESTS += test_manyurls
+test_manyurls_SOURCES = test_manyurls.c manyurls.h
+check_PROGRAMS += test_manyurls
+test_manyurls.log: tst_longremote3.log
+TESTS += test_manyurls
 endif
 
 test_partvar_SOURCES = test_partvar.c

nczarr_test/Makefile.am

@@ -96,16 +96,16 @@ TESTS += run_unlim_io.sh run_nccopyz.sh
 endif
 
 if LARGE_FILE_TESTS
- check_PROGRAMS += test_writecaching test_readcaching
- TESTS += run_cachetest.sh
+check_PROGRAMS += test_writecaching test_readcaching
+TESTS += run_cachetest.sh
 endif
 
 endif #NETCDF_BUILD_UTILITIES
 
 if NETCDF_BUILD_UTILITIES 
 
 if NETCDF_ENABLE_NCZARR_ZIP
- TESTS += run_newformat.sh
+TESTS += run_newformat.sh
 endif
 
 if LARGE_FILE_TESTS