Bug 128041 - Compilation fails for cdf datasource when library is present but not header
Summary: Compilation fails for cdf datasource when library is present but not header
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-25 22:06 UTC by Andrew Walker
Modified: 2006-05-27 11:39 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Walker 2006-05-25 22:07:00 UTC
Version:           HEAD (using KDE KDE 3.5.0)
Installed from:    Compiled From Sources
OS:                Linux

STEPS TO REPRODUCE:
Ensure you have a valid copy of libcdf.a on the LIB path
Ensure you have no copy of cdf.h on the INCLUDE path
Run configure
Make Kst

RESULTS:
Building the cdf datasource fails

EXPECTED RESULTS:
The cdf datasource should not attempt to build
Comment 1 George Staikos 2006-05-27 11:39:44 UTC
SVN commit 545285 by staikos:

quote arguments to avoid shell script errors when the variables are empty
BUG: 128041


 M  +6 -6      configure.in.in  


--- trunk/extragear/graphics/kst/configure.in.in #545284:545285
@@ -97,8 +97,8 @@
 KDE_CHECK_LIB(cdf, CDFlib, CDFLIBS="$ac_cdf_libraries -lcdf $CDFLIBS", , )
 LDFLAGS="$ldflags_save"
 
-if test $ac_cv_lib_cdf_CDFlib = yes; then
-  if test $have_cdf_h = yes; then
+if test "x$ac_cv_lib_cdf_CDFlib" = "xyes"; then
+  if test "x$have_cdf_h" = "xyes"; then
     AC_SUBST(CDFLIBS)
     AC_SUBST(CDFINCLUDES)
     AC_DEFINE(KST_HAVE_CDF, 1, [If we have the CDF libraries.])
@@ -130,8 +130,8 @@
   KDE_CHECK_LIB(netcdf, nc_open, NETCDFLIBS="$ac_netcdf_libraries -lnetcdf_c++ -lnetcdf $NETCDFLIBS", , )
   LDFLAGS="$ldflags_save"
 
-  if test $ac_cv_lib_netcdf_nc_open = yes; then
-    if test $have_netcdfcpp_h = yes; then
+  if test "x$ac_cv_lib_netcdf_nc_open" = "xyes"; then
+    if test "x$have_netcdfcpp_h" = "xyes"; then
       AC_SUBST(NETCDFLIBS)
       AC_SUBST(NETCDFINCLUDES)
       AC_DEFINE(KST_HAVE_NETCDF, 1, [If we have the NetCDF libraries.])
@@ -144,8 +144,8 @@
 # Readline and ncurses check
 KDE_CHECK_HEADER(readline/readline.h, AC_DEFINE(HAVE_READLINE_H, 1, [If we have libreadline headers]) have_readline_h=yes, , )
 KDE_CHECK_LIB(readline, rl_cleanup_after_signal, LIBREADLINE="-lreadline -lncurses", , -lncurses)
-if test $ac_cv_lib_readline_rl_cleanup_after_signal = yes; then
-  if test $have_readline_h = yes; then
+if test "x$ac_cv_lib_readline_rl_cleanup_after_signal" = "xyes"; then
+  if test "x$have_readline_h" = "xyes"; then
     AC_SUBST(LIBREADLINE)
     AC_DEFINE(KST_HAVE_READLINE, 1, [If we have libreadline.])
   else