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

more tests, more internal documentation, minor bug fixes #711

Merged
merged 44 commits into from
Dec 19, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
03753c0
now return NC_ENOTBUILT for not built library features
edhartnett Nov 13, 2017
e928964
Merge branch 'master' into ejh_notbuilt_errors
edhartnett Nov 15, 2017
eecade6
Merge branch 'master' into ejh_notbuilt_errors
edhartnett Nov 18, 2017
d64550d
Merge branch 'master' into ejh_notbuilt_errors
edhartnett Nov 22, 2017
34c9248
Merge branch 'master' into ejh_notbuilt_errors
edhartnett Nov 28, 2017
bcfce16
fixed warning
edhartnett Nov 30, 2017
ec3bb22
fixed warning
edhartnett Nov 30, 2017
4f983e5
autotools cleanup
edhartnett Nov 30, 2017
46d3bca
removed non-working BUILD_DLL target
edhartnett Nov 30, 2017
857dda4
fixed two doxygen warnings
edhartnett Nov 30, 2017
0fbfc2b
more warnings
edhartnett Nov 30, 2017
06eb866
applied gsjaardema's suggested change to ocnode.c
edhartnett Nov 30, 2017
5d11415
adding internal documentation
edhartnett Dec 1, 2017
2023ed6
fixed problems with bad inputs to nc_def_Var
edhartnett Dec 1, 2017
ce1df11
more testing for nc4var.c functions
edhartnett Dec 1, 2017
3e45557
more docs, more tests, eliminated two unneeded functions
edhartnett Dec 1, 2017
42c482d
more tests
edhartnett Dec 1, 2017
a8e9c19
more tests, fixed documentation warnings
edhartnett Dec 1, 2017
ded01e0
more tests
edhartnett Dec 1, 2017
acbd3e2
more tests
edhartnett Dec 1, 2017
58f422e
more tests
edhartnett Dec 1, 2017
300236c
more tests
edhartnett Dec 1, 2017
d840bc7
starting adding testing for _int() functions for caching
edhartnett Dec 2, 2017
6c3aa79
more testing of nc_set_var_chunk_cache_ints
edhartnett Dec 2, 2017
7cd344f
removed redundant check in NC4_def_var()
edhartnett Dec 2, 2017
bfeb8d4
more testing
edhartnett Dec 2, 2017
ea63ff5
more tests
edhartnett Dec 2, 2017
1cf8177
more test
edhartnett Dec 2, 2017
996e393
more test
edhartnett Dec 2, 2017
8055c1c
more tests
edhartnett Dec 3, 2017
9e4c564
more tests
edhartnett Dec 3, 2017
bf6dab1
fixed error value from nc4_find_g_var_nc() when ncid is bad
edhartnett Dec 3, 2017
2066232
more tests, more interanl documentation
edhartnett Dec 3, 2017
3514ae9
more internal docs
edhartnett Dec 3, 2017
097a09f
more internal docs
edhartnett Dec 3, 2017
11e3e81
more internal docs
edhartnett Dec 3, 2017
fec74e1
more internal documentation
edhartnett Dec 4, 2017
4de61e2
more docs, more cleaning
edhartnett Dec 4, 2017
652f3b1
more docs
edhartnett Dec 4, 2017
1153f0f
merged in branch ejh_cleanup
edhartnett Dec 4, 2017
ea49cc1
merged ejh_notbuilt_errors
edhartnett Dec 4, 2017
e6785c7
moved declaration out of for loop
edhartnett Dec 4, 2017
e8471c9
Merge branch 'master' into ejh_more_tests
WardF Dec 6, 2017
77cca75
Merge branch 'master' into ejh_more_tests
WardF Dec 8, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 7 additions & 33 deletions Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,12 @@
ACLOCAL_AMFLAGS = -I m4

# These files get added to the distribution.
EXTRA_DIST = README.md COPYRIGHT INSTALL.md test_prog.c \
lib_flags.am cmake CMakeLists.txt COMPILE.cmake.txt \
config.h.cmake.in cmake_uninstall.cmake.in \
FixBundle.cmake.in \
nc-config.cmake.in RELEASE_NOTES.md CTestCustom.cmake \
CTestConfig.cmake.in libnetcdf.settings.in netCDFConfig.cmake.in \
CMakeInstallation.cmake test-driver-verbose test_common.in
EXTRA_DIST = README.md COPYRIGHT INSTALL.md test_prog.c lib_flags.am \
cmake CMakeLists.txt COMPILE.cmake.txt config.h.cmake.in \
cmake_uninstall.cmake.in FixBundle.cmake.in nc-config.cmake.in \
RELEASE_NOTES.md CTestCustom.cmake CTestConfig.cmake.in \
libnetcdf.settings.in netCDFConfig.cmake.in CMakeInstallation.cmake \
test-driver-verbose test_common.in

pkgconfigdir=$(libdir)/pkgconfig
pkgconfig_DATA = netcdf.pc
Expand Down Expand Up @@ -77,7 +76,7 @@ endif
# depend on it.
SUBDIRS = include $(H5_TEST_DIR) libdispatch libsrc $(LIBSRC4_DIR) \
$(LIBSRCP) $(OCLIB) $(DAP2) ${DAP4} liblib $(NCGEN3) $(NCGEN) \
$(NCDUMP) $(TESTDIRS) docs $(EXAMPLES) $(UDUNITS) $(LIBCF)
$(NCDUMP) $(TESTDIRS) docs $(EXAMPLES)

# Remove these generated files, for a distclean.
DISTCLEANFILES = VERSION comps.txt test_prog libnetcdf.settings \
Expand All @@ -101,14 +100,6 @@ ${prefix}/bin/ncgen$(EXEEXT) ${prefix}/bin/ncdump$(EXEEXT) \
${prefix}/share/man/man1/ncgen.1 ${prefix}/share/man/man1/ncdump.1
endif

if BUILD_DLL
BINFILES += bin/libnetcdf-7.dll lib/libnetcdf.dll.a lib/libnetcdf.a \
lib/libnetcdf.la lib/netcdfdll.def
ZIPBINFILES += ${prefix}/bin/libnetcdf-7.dll \
${prefix}/lib/libnetcdf.dll.a ${prefix}/lib/libnetcdf.a \
${prefix}/lib/libnetcdf.la ${prefix}/lib/netcdfdll.def
endif # BUILD_DLL

# install libnetcdf.settings in lib directory.
settingsdir = $(libdir)
settings_DATA = libnetcdf.settings
Expand Down Expand Up @@ -154,29 +145,12 @@ install-netcdf-fortran:

endif

# At Unidata, package up binaries.
ftpbin: install
echo "Getting binaries from ${prefix}"
ls -l ${prefix}
echo "These are the binaries for netCDF @PACKAGE_VERSION@." > ${prefix}/README_BINARIES.txt
echo "For this build: CC=$(CC) CXX=$(CXX)" >> ${prefix}/README_BINARIES.txt
echo "CFLAGS=$(CFLAGS) CXXFLAGS=$(CXXFLAGS)" >> ${prefix}/README_BINARIES.txt
echo "FCFLAGS=$(FCFLAGS) F77FLAGS=$(F77FLAGS) $F90FLAGS=$(F90FLAGS)" >> ${prefix}/README_BINARIES.txt
which tar
echo "PATH: $(PATH)"
tar cf @BINFILE_NAME@ -C ${prefix} ${BINFILES}
gzip -f @BINFILE_NAME@
zip -j netcdf_${VERSION}.zip ${ZIPBINFILES}

check_nc_config:
$(CC) `./nc-config --cflags` test_prog.c -o test_prog `./nc-config --libs`
./test_prog

install-data-hook:

if BUILD_DLL
cp liblib/netcdfdll.def $(DESTDIR)${prefix}/bin
endif # BUILD_DLL
@echo ''
@echo '+-------------------------------------------------------------+'
@echo '| Congratulations! You have successfully installed netCDF! |'
Expand Down
48 changes: 1 addition & 47 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -83,20 +83,6 @@ AC_CONFIG_SRCDIR([include/netcdf.h])

AC_MSG_NOTICE([checking user options])

# If --enable-dll is specified the DLL will be built. This only works
# on mingw.
AC_MSG_CHECKING([whether a win32 DLL is desired])
AC_ARG_ENABLE([dll],
[AS_HELP_STRING([--enable-dll],
[build a win32 DLL (only works on mingw)])])
test "x$enable_dll" = xyes || enable_dll=no
AC_MSG_RESULT([$enable_dll])
if test "x$enable_dll" = xyes; then
AC_DEFINE(DLL_NETCDF, 1, [set this only when building a DLL under MinGW])
AC_DEFINE(DLL_EXPORT, 1, [set this only when building a DLL under MinGW])
fi
AM_CONDITIONAL(BUILD_DLL, [test x$enable_dll = xyes])

# Did the user specify a default minimum blocksize (NCIO_MINBLOCKSIZE) for posixio?
AC_MSG_CHECKING([whether a NCIO_MINBLOCKSIZE was specified])
AC_ARG_WITH([minblocksize],
Expand Down Expand Up @@ -348,13 +334,8 @@ fi
# See if the user provided us with a curl library
# Do an initial lib test for curl, but suppress the default action
AC_CHECK_LIB([curl],[curl_easy_setopt],[found_curl=yes],[found_curl=no])
#AC_CHECK_LIB([curl.dll],[curl_easy_setopt])
# If curl is required but there is no curl, then complain
if test $require_curl = yes ; then
# Removed. Why assume no curl if we are building DLL?
#if test $enable_dll = yes ; then
# AC_MSG_NOTICE([libcurl not found; continuing])
#elif test $found_curl = no ; then
if test $found_curl = no ; then
AC_MSG_NOTICE([libcurl not found; disabling remote protocol(s) support])
enable_dap=no
Expand Down Expand Up @@ -712,7 +693,7 @@ CFLAGS="$SAVECFLAGS"
# Set up libtool.
AC_MSG_NOTICE([setting up libtool])
LT_PREREQ([2.2])
LT_INIT(win32-dll)
LT_INIT()

# Valgrind tests don't work with shared builds because of some libtool
# weirdness.
Expand Down Expand Up @@ -1236,33 +1217,6 @@ if test "x$enable_logging" = xyes; then
AC_DEFINE([LOGGING], 1, [If true, turn on logging.])
fi


# Like other libraries, udunits and libcf
# are no long part of the netcdf distribution.

#AC_MSG_CHECKING([whether udunits is to be built])
#AC_ARG_WITH([udunits],
# [AS_HELP_STRING([--with-udunits],
# [Build udunits2 package.])])
#test "x$with_udunits" = xyes || with_udunits=no
#AC_MSG_RESULT($with_udunits)
#AM_CONDITIONAL(BUILD_UDUNITS, [test "x$with_udunits" = xyes])


# Does the user want to also build the libcf library?
#AC_MSG_CHECKING([whether libcf is to be built])
#AC_ARG_WITH([libcf],
# [AS_HELP_STRING([--with-libcf],
# [build and install libcf library, a library for \
# handling data in conformance with the Climate and \
# Forecast conventions. (Requires netCDF-4 and HDF5)])])
#test "x$with_libcf" = xyes || with_libcf=no
#AC_MSG_RESULT($with_libcf)
#AM_CONDITIONAL(BUILD_LIBCF, [test "x$with_libcf" = xyes])

#AC_CONFIG_SUBDIRS([udunits libcf])


# Automake conditionals need to be called, whether the answer is yes
# or no.
AM_CONDITIONAL(BUILD_PARALLEL, [test x$enable_parallel = xyes])
Expand Down
11 changes: 11 additions & 0 deletions docs/Doxyfile.in
Original file line number Diff line number Diff line change
Expand Up @@ -788,7 +788,18 @@ INPUT = \
@abs_top_srcdir@/libdispatch/dparallel.c \
@abs_top_srcdir@/libdispatch/derror.c \
@abs_top_srcdir@/libdispatch/dv2i.c \
@abs_top_srcdir@/libdispatch/dcopy.c \
@abs_top_srcdir@/libsrc4/nc4file.c \
@abs_top_srcdir@/libsrc4/nc4var.c \
@abs_top_srcdir@/libsrc4/nc4hdf.c \
@abs_top_srcdir@/libsrc4/nc4internal.c \
@abs_top_srcdir@/libsrc4/nc4type.c \
@abs_top_srcdir@/libsrc4/nc4grp.c \
@abs_top_srcdir@/libsrc4/ncfunc.c \
@abs_top_srcdir@/libsrc4/nc4dim.c \
@abs_top_srcdir@/libsrc4/nc4attr.c \
@abs_top_srcdir@/libsrc4/nc4info.c \
@abs_top_srcdir@/libsrc4/nc4dispatch.c \
@abs_top_srcdir@/examples/C/simple_xy_wr.c \
@abs_top_srcdir@/examples/C/simple_xy_rd.c \
@abs_top_srcdir@/examples/C/sfc_pres_temp_wr.c \
Expand Down
2 changes: 1 addition & 1 deletion h5_test/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ tst_h_atts3 tst_h_atts4 tst_h_vars tst_h_vars2 tst_h_vars3 tst_h_grps \
tst_h_compounds tst_h_compounds2 tst_h_wrt_cmp tst_h_vl tst_h_opaques \
tst_h_strings tst_h_strings1 tst_h_strings2 tst_h_ints \
tst_h_dimscales tst_h_dimscales1 tst_h_dimscales2 tst_h_dimscales3 \
tst_h_enums tst_h_dimscales4 #tst_h_filters
tst_h_enums tst_h_dimscales4

# If benchmarks were turned on, build and run a bunch more tests.
if BUILD_BENCHMARKS
Expand Down
107 changes: 0 additions & 107 deletions h5_test/tst_h_filters.c

This file was deleted.

2 changes: 2 additions & 0 deletions include/nc_tests.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ for conditions of use.
#define THIRTY_TWO_MEG (SIXTEEN_MEG * 2)
#define SIXTY_FOUR_MEG (SIXTEEN_MEG * 4)
#define ONE_TWENTY_EIGHT_MEG (SIXTEEN_MEG * 8)
#define TEST_VAL_42 42
#define BAD_NAME "dd//d/ "
/** \} */

#ifdef USE_PNETCDF
Expand Down
Loading