Re: CVS update: subversion/subversion/libsvn_wc update.c

[Branko Čibej <br...@hermes.si>]

sussman@tigris.org wrote:
>    /* kff todo: this will want to be somewhere else, and get decided at
>       configure time too probably.  For now let's just get checkout
>       working. */
>   +/* ben sez:  aha!  then this belongs in our "config.h" file! */
>   +
>    #define SVN_DIR_SEPARATOR '/'

Just a small note here: we might want to use a more generic
path parsing library later on. (Do we want Subversion to ever
run on VMS? No? Why not?)

    Brane

-- 
Branko Čibej                 <br...@hermes.si>
HERMES SoftLab, Litijska 51, 1000 Ljubljana, Slovenia
voice: (+386 1) 586 53 49     fax: (+386 1) 586 52 70

Re: CVS update: subversion/subversion/libsvn_wc update.c

["Jonathan S. Shapiro" <sh...@eros-os.org>]

On Windows the directory separator can appear as *either* '/' or '\'
depending on circumstances, and that the drive separator is a different
thing altogether, and that the start of a share begins with '\\'. In the
end, a platform-specific path management library is definitely called for.


----- Original Message -----
From: "Karl Fogel" <kf...@galois.collab.net>
To: <de...@subversion.tigris.org>
Sent: Friday, August 11, 2000 11:06 AM
Subject: Re: CVS update: subversion/subversion/libsvn_wc update.c


> Branko =?iso-8859-2?Q?=C8ibej?= <br...@hermes.si> writes:
> > >   +/* ben sez:  aha!  then this belongs in our "config.h" file! */
> > >   +
> > >    #define SVN_DIR_SEPARATOR '/'
> >
> > Just a small note here: we might want to use a more generic
> > path parsing library later on. (Do we want Subversion to ever
> > run on VMS? No? Why not?)
>
> Yep, definitely.  But: "Premature generalization is the root of all
> evil." :-)
>

Re: CVS update: subversion/subversion/libsvn_wc update.c

[Karl Fogel <kf...@galois.collab.net>]

Karl Fogel <kf...@galois.collab.net> writes:
> > Just a small note here: we might want to use a more generic
> > path parsing library later on. (Do we want Subversion to ever
> > run on VMS? No? Why not?)
> 
> Yep, definitely.  But: "Premature generalization is the root of all
> evil." :-)

To clarify, what I meant by that was:

   a) Such an abstraction is definitely a Good Thing, but

   b) Not working on it just yet because am trying to get other things
      working, and it's not hard to add this later (if one is aware
      it's coming and codes accordingly)

-K

Re: CVS update: subversion/subversion/libsvn_wc update.c

[Karl Fogel <kf...@galois.collab.net>]

Branko =?iso-8859-2?Q?=C8ibej?= <br...@hermes.si> writes:
> >   +/* ben sez:  aha!  then this belongs in our "config.h" file! */
> >   +
> >    #define SVN_DIR_SEPARATOR '/'
> 
> Just a small note here: we might want to use a more generic
> path parsing library later on. (Do we want Subversion to ever
> run on VMS? No? Why not?)

Yep, definitely.  But: "Premature generalization is the root of all
evil." :-)