diff options
author | James Laird <jlaird@hdfgroup.org> | 2005-11-17 18:49:28 (GMT) |
---|---|---|
committer | James Laird <jlaird@hdfgroup.org> | 2005-11-17 18:49:28 (GMT) |
commit | 75ea10e5f50c3c23fd73f073adb84b0baf551edc (patch) | |
tree | 86ac2ae00334b79b7aa3d010415cc01e4e2caae4 /config | |
parent | 706885260d43891095c910cd5975390504934c09 (diff) | |
download | hdf5-75ea10e5f50c3c23fd73f073adb84b0baf551edc.zip hdf5-75ea10e5f50c3c23fd73f073adb84b0baf551edc.tar.gz hdf5-75ea10e5f50c3c23fd73f073adb84b0baf551edc.tar.bz2 |
[svn-r11737] Purpose:
Bug fix
Description:
Before this checkin, 'gmake check-s' would fail if there was a file in
the current directory named 'check-s'.
This is fixed under gmake (not sure how to fix for other makes).
Solution:
check, progs, install, etc. are what gmake calls "phony" targets,
which means that no file should be created. These targets can be
specified by a line of the form
.PHONY: check progs install ...
Automake adds this line for targets it knows about, but HDF5 has a
lot of custom rules. This checkin adds a .PHONY line for those rules.
I believe that only gmake recognizes the .PHONY line (at least, pmake
doesn't seem to), but a partial solution is better than none.
This error should occur very rarely anyway (the user has to manually
create files with names like 'build-check-s' or '_test').
Platforms tested:
mir, sleipnir, modi4
Diffstat (limited to 'config')
-rw-r--r-- | config/commence.am | 9 | ||||
-rw-r--r-- | config/conclude.am | 1 |
2 files changed, 9 insertions, 1 deletions
diff --git a/config/commence.am b/config/commence.am index 39bff3c..3a7ef49 100644 --- a/config/commence.am +++ b/config/commence.am @@ -97,3 +97,12 @@ CHECK_CLEANFILES=*.chkexe *.chklog *.clog # normally go) because it should be included in the top-level Makefile.am # as well. check-install: installcheck + +# List all build rules defined by HDF5 Makefiles as "PHONY" targets here. +# This tells the Makefiles that these targets are not files to be built but +# commands that should be executed even if a file with the same name already +# exists. +.PHONY: build-check-clean build-check-p build-check-s build-lib build-progs \ + build-tests check-clean check-install check-p check-s check-vfd \ + install-doc lib progs tests uninstall-doc _exec_check-s _test + diff --git a/config/conclude.am b/config/conclude.am index 0043dbc..479de46 100644 --- a/config/conclude.am +++ b/config/conclude.am @@ -26,7 +26,6 @@ PROGS = $(bin_PROGRAMS) $(bin_SCRIPTS) $(noinst_PROGRAMS) $(noinst_SCRIPTS) \ $(EXTRA_PROG) TESTS = $(check_PROGRAMS) $(check_SCRIPTS) $(EXTRA_TEST) - # lib/progs/tests targets recurse into subdirectories. build-* targets # build files in this directory. # BUILT_SOURCES contain targets that need to be built before anything else |