This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: problems with cvs -z9 update


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/



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]