Re: [PATCH] fixes empty $NGX_SBIN_PATH in the root Makefile

[Maxim Dounin <mdounin@xxxxxxxxxx>]

Hello!

On Sat, May 16, 2009 at 06:23:28PM +0700, Denis F. Latypoff wrote:

> Hello Igor,
> 
>   Начиная с версии 0.7.53 сломался make upgrade. Патч.

Good catch, thanks.

IMHO, было бы наверное правильнее сделать запись Makefile 
отдельным скриптом, и звать его в самом конце - чтобы любые 
ошибки приводили к отсутствию Makefile.  Заодно это справит 
довольно типичную проблему с

./configure ; make

вместо

./configure && make

Единственная потенциальная проблема которую я вижу (по сравнению с 
прошлым поведение) - это отсутствие возможности сделать make clean 
после неуспешного ./configure.  Может быть для этого имеет смысл в 
самом начале писать совсем простой Makefile, умеющий только clean, 
и ругающийся на всё остальное.

Maxim Dounin

p.s. Патч добавляющий запись Makefile'а в самом конце - 
прилагается.

diff --git a/auto/init b/auto/init
--- a/auto/init
+++ b/auto/init
@@ -37,42 +37,3 @@ else
     ngx_n=
     ngx_c='\c'
 fi
-
-
-# create Makefile
-
-cat << END > Makefile
-
-build:
-       \$(MAKE) -f $NGX_MAKEFILE
-
-install:
-       \$(MAKE) -f $NGX_MAKEFILE install
-
-clean:
-       rm -rf Makefile $NGX_OBJS
-
-upgrade:
-       $NGX_SBIN_PATH -t
-
-       kill -USR2 \`cat $NGX_PID_PATH\`
-       sleep 1
-       test -f $NGX_PID_PATH.oldbin
-
-       kill -QUIT \`cat $NGX_PID_PATH.oldbin\`
-
-upgrade1:
-       # upgrade 0.1.x to 0.2+
-
-       $NGX_SBIN_PATH -t
-
-       cp $NGX_PID_PATH $NGX_PID_PATH.oldbin
-
-       kill -USR2 \`cat $NGX_PID_PATH\`
-       sleep 1
-       test -f $NGX_PID_PATH.oldbin
-
-       cp $NGX_PID_PATH $NGX_PID_PATH.newbin
-
-       kill -QUIT \`cat $NGX_PID_PATH.oldbin\`
-END
diff --git a/auto/makefile b/auto/makefile
new file mode 100644
--- /dev/null
+++ b/auto/makefile
@@ -0,0 +1,38 @@
+
+# Copyright (C) Igor Sysoev
+
+cat << END > Makefile
+
+build:
+       \$(MAKE) -f $NGX_MAKEFILE
+
+install:
+       \$(MAKE) -f $NGX_MAKEFILE install
+
+clean:
+       rm -rf Makefile $NGX_OBJS
+
+upgrade:
+       $NGX_SBIN_PATH -t
+
+       kill -USR2 \`cat $NGX_PID_PATH\`
+       sleep 1
+       test -f $NGX_PID_PATH.oldbin
+
+       kill -QUIT \`cat $NGX_PID_PATH.oldbin\`
+
+upgrade1:
+       # upgrade 0.1.x to 0.2+
+
+       $NGX_SBIN_PATH -t
+
+       cp $NGX_PID_PATH $NGX_PID_PATH.oldbin
+
+       kill -USR2 \`cat $NGX_PID_PATH\`
+       sleep 1
+       test -f $NGX_PID_PATH.oldbin
+
+       cp $NGX_PID_PATH $NGX_PID_PATH.newbin
+
+       kill -QUIT \`cat $NGX_PID_PATH.oldbin\`
+END
diff --git a/configure b/configure
--- a/configure
+++ b/configure
@@ -106,3 +106,4 @@ have=NGX_USER value="\"$NGX_USER\"" . au
 have=NGX_GROUP value="\"$NGX_GROUP\"" . auto/define
 
 . auto/summary
+. auto/makefile