problems with cvs -z9 update

Gerald Pfeifer pfeifer@dbai.tuwien.ac.at
Mon Aug 10 12:33:00 GMT 1998


On 10 Aug 1998, Alexandre Oliva wrote:
> Please insert a space before the slash; some systems interpret `#! /'
> as a magic number:
> 
> #! /bin/sh

Thanks for the hint. Done.


>> + if [ ! -d CVS  -o  ! -f gcc/version.c ]; then
> 
> Please make that
> 
> if [ ! -d CVS ] || [ ! -f gcc/version.c ]; then

Again, thanks for the hint. Done.  (Interestingly ./configure does use
-o as well!)


>> + cvs -q update -P -d -A
> I don't think the `-A' should be here; it defeats the purpose of
> egcs_latest_snapshot, for instance

Indeed I thought about it and my idea was that for new users it might
be useful. I have now made this easily configurable in my local version:

  UPDATE_OPTIONS=-P
  # Add -d to create any directories that exist in the repository but not
  #        locally. 
  # Add -A to reset any sticky tags, dates, or `-k' options.


Oliva, thanks for your comments!
Jeff, okay to install?

Gerald
-- 
Gerald Pfeifer (Jerry)      Vienna University of Technology
pfeifer@dbai.tuwien.ac.at   http://www.dbai.tuwien.ac.at/~pfeifer/




More information about the Gcc-patches mailing list