Should we use SVN keyword expansion? (was Re: [xwiki-dev] [Proposal] Subversion configuration)

[Vincent Massol]

On Mar 11, 2007, at 10:38 PM, Sergiu Dumitriu wrote:

> I don't really understand how this works. If on checkout $version $  
> is expanded, what gets commited back? $version $ or the actual number?

@version $Id: $

i.e. no expansion

-Vincent

> On 3/11/07, Vincent Massol <vincent at massol.net> wrote:
>
> On Mar 10, 2007, at 12:52 AM, Artem Melentyev wrote:
>
> > For what purpose we need svn:keywords="Author Date Id Revision"
> > properties?:
>
> To expand svn keywords in source files (especially the "@version $Id:
> $" part). Why do we need this?
>
> To be honest I'm not sure. I've always done and never questioned it
> so I guess now is the right time to do so... :)
>
> Pros I can think of:
> - it allows us to distribute sources and if someone modifies it we
> can know what version of the source they're using
>
> Cons I can think of:
> - more work, developers need to have keyword expansion set in their
> subversion config file (although we would need subversion config
> anyway for eol, mimetypes, etc)
> - for developers, duplicates the information in SVN. It's mostly
> possibly interesting for users and even then it's not sure
>
> Note: in Svn the files are stored in the repo without the expansion;
> it's only on checkout that the expansion happens.
>
> Thus after more thought, I'm more tempted to say we don't need them
> and that we should remove all the @version I've added here and then
> (I'll do it of course should we agree to not use keyword expansion).
>
> Thanks
> -Vincent
>
>
>
> -- 
> http://purl.org/net/sergiu
>
> --
> You receive this message as a subscriber of the xwiki- 
> dev at objectweb.org mailing list.
> To unsubscribe: mailto:xwiki-dev-unsubscribe at objectweb.org
> For general help: mailto:sympa at objectweb.org?subject=help
> ObjectWeb mailing lists service home page: http://www.objectweb.org/ 
> wws

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.xwiki.org/pipermail/devs/attachments/20070311/715b1a5b/attachment.htm