[Patches] [Patch #101275] Issue a warning if Setup.in is newer than Setup

noreply@sourceforge.net noreply@sourceforge.net
Thu, 24 Aug 2000 08:32:28 -0700


Patch #101275 has been updated. 

Project: 
Category: Build
Status: Open
Summary: Issue a warning if Setup.in is newer than Setup

Follow-Ups:

Date: 2000-Aug-24 16:23
By: fdrake

Comment:
This issues a waring during the build if the generated Setup flie is older than the Setup.in file.  This is helpful in getting users to check that they have everything they need in their Setup file.  The limitation is that the warning is issued several times during a build, but it's not clear that's a serious problem, though it is potentially annoying.
-------------------------------------------------------

Date: 2000-Aug-24 17:05
By: montanaro

Comment:
This works fine for me.  My only question is whether
the use of the relatively new "-nt" flag of test is
going to be as portable as we'd like.  I notice it's
not used anywhere else in the Python makefiles or in
the configure script.  Configure only uses =, !=, -gt
and -eq.  It's a script that tries to be painfully
portable.

On the other hand, if we're requiring an ANSI C compiler,
why not a fairly recent incarnation of /bin/sh?

Other than that thought, I say check it in.

-------------------------------------------------------

Date: 2000-Aug-24 17:13
By: fdrake

Comment:
My Mandrake box doesn't have a man page for sh(1), so it may be that -nt is specific to bash; Mandrake uses a symlink to use bash as sh, so there's no way for me to tell if -nt is available under a traditional sh.  This could be a showstopper for this patch.
-------------------------------------------------------

Date: 2000-Aug-24 17:19
By: montanaro

Comment:
try "man 1 test".  -nt is actually an argument to
the test command.  Bash's builtin test command does
support -nt.

-------------------------------------------------------

Date: 2000-Aug-24 17:32
By: twouters

Comment:
BSDI 'test' does not support -nt.

-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=101275&group_id=5470