From 309fca35acfeb776a422d6140ddae8be0b1dbff0 Mon Sep 17 00:00:00 2001 From: Pavel Roskin Date: Tue, 26 Jan 2016 22:42:43 -0800 Subject: Don't use a symlink where #include would suffice Absolute symlinks break when the source is moved to another location. In the best case, the symlink will be replaced with a copy, but that creates code duplication. A user who got the packaged source may decide to change one config.c without realizing that another config.c will get the same changes. Don't ignore config.c, it's a regular source file now, albeit very short. Testing done: compiled inside and outside build directory. --- bootstrap | 1 - 1 file changed, 1 deletion(-) (limited to 'bootstrap') diff --git a/bootstrap b/bootstrap index 1fae2a0e..49aeefef 100755 --- a/bootstrap +++ b/bootstrap @@ -33,5 +33,4 @@ touch NEWS touch AUTHORS touch README touch ChangeLog -ln -s ../config.c $PWD/sesman/tools/config.c autoreconf -fvi -- cgit v1.2.3 From 12cd2f6a1d4aa24fca9046c370cf3b466205220d Mon Sep 17 00:00:00 2001 From: Pavel Roskin Date: Fri, 29 Jan 2016 23:00:44 -0800 Subject: Don't touch any files in bootstrap configure.ac already exists. Its timestamp doesn't need to be updated. autoreconf takes care of the timestamps. Since "foreign" is used in Makefile.am, no standard GNU files are required. Distributing empty files like README could be disappointing for the users. --- bootstrap | 5 ----- 1 file changed, 5 deletions(-) (limited to 'bootstrap') diff --git a/bootstrap b/bootstrap index 49aeefef..be61972e 100755 --- a/bootstrap +++ b/bootstrap @@ -28,9 +28,4 @@ then exit 1 fi -touch configure.ac -touch NEWS -touch AUTHORS -touch README -touch ChangeLog autoreconf -fvi -- cgit v1.2.3