diff mbox

configure: remove confusing file manipulation

Message ID 1369228899-74124-1-git-send-email-emaste@freebsd.org
State New
Headers show

Commit Message

Ed Maste May 22, 2013, 1:21 p.m. UTC
The configure script had some code to manipulate config-host.ld~ (i.e.,
a common backup filename), comparing it with the newly-generated file.
I believe the sense of the comparison was backwards.

Since it seemed to serve little purpose anyway, remove it to avoid any
confusion.

Signed-off-by: Ed Maste <emaste@freebsd.org>
---
 configure | 9 ---------
 1 file changed, 9 deletions(-)

Comments

Michael Tokarev May 25, 2013, 9:31 a.m. UTC | #1
22.05.2013 17:21, Ed Maste wrote:
> The configure script had some code to manipulate config-host.ld~ (i.e.,
> a common backup filename), comparing it with the newly-generated file.
> I believe the sense of the comparison was backwards.
> 
> Since it seemed to serve little purpose anyway, remove it to avoid any
> confusion.

Thanks, applied to the trivial patches queue.

/mjt
diff mbox

Patch

diff --git a/configure b/configure
index 5ae7e4a..cfdb564 100755
--- a/configure
+++ b/configure
@@ -4104,17 +4104,8 @@  if test "$gcov" = "yes" ; then
 fi
 
 # generate list of library paths for linker script
-
 $ld --verbose -v 2> /dev/null | grep SEARCH_DIR > ${config_host_ld}
 
-if test -f ${config_host_ld}~ ; then
-  if cmp -s $config_host_ld ${config_host_ld}~ ; then
-    mv ${config_host_ld}~ $config_host_ld
-  else
-    rm ${config_host_ld}~
-  fi
-fi
-
 # use included Linux headers
 if test "$linux" = "yes" ; then
   mkdir -p linux-headers