[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Feature Request -- svn commit --unimportant

From: Gabriela Gibson <gabriela.gibson_at_gmail.com>
Date: Mon, 7 Oct 2013 01:29:46 +0100

I can see Eitan's point -- if we went ahead and changed all the obsolete
<tt> tags on the Subversion website, it would remove a lot of useful
blame info.

Is a two stage operation possible?

say, you type:

'svn commit --blame-revert ...'

Part one would be a regular commit, part two would be a second commit that
restores all the original authors that the first commit modified; so one
commit would produce two consecutive revisions.

To be (relatively) safe, this could be a repository admin action only.

Gabriela

On Sun, Oct 6, 2013 at 8:45 PM, Bert Huijben <bert_at_qqmail.nl> wrote:

>
>
> > -----Original Message-----
> > From: Gabriela Gibson [mailto:gabriela.gibson_at_gmail.com]
> > Sent: zaterdag 5 oktober 2013 17:48
> > To: Subversion Development; lists_at_eitanadler.com
> > Subject: Feature Request -- svn commit --unimportant
> >
> > Hi,
> >
> > The following conversation took place on the svn-dev IRC yesterday
> > evening, with Eitan Adler:
> >
> > [22:34] <Eitan> feature request: "svn commit --unimportant" which does
> > not destroy "svn blame" unless you write "svn blame --all"
> > [22:44] <cinnamon> Eitan: what were you trying to do that made you think
> > of this?
> > [22:45] <Eitan> cinnamon: fix up whitespace errors
> > [22:45] <Eitan> cinnamon: the idea is similar to wikipedia's "trivial
> edit"
> > [22:46] <Eitan> cinnamon: I'd love to go en mass through our code and
> > fix up internal whitespace errors but that would destroy "svn blame"
> > of who wrote what
> > [22:46] <cinnamon> nod, that would defeat the purpose of blame somewhat
> > [22:46] <lgo> you can of course use the ignore white space option for
> blame
> > [22:47] <Eitan> lgo: in some cases it isn't just whitespace
> > [22:47] <Eitan> for example, in our documentation we use "&os;" instead
> > of "FreeBSD"
> > [22:47] <Eitan> and I'd love to be able to change all the latter to the
> > former
> > [22:47] <Eitan> etc. etc. etc.
> > [22:48] <Eitan> cinnamon: that is why I propose "blame" and "blame
> > --include-everything/"
> >
> > I think this would be a very useful addition for many users and so
> > that it does not get lost in the IRC chat, I thought I post it here
> > for discussion.
>
> I assume you do know about the -x argument to blame where you can ask blame
> to ignore whitespace only changes?
> (E.g. add "-x -b", see 'svn help blame')
>
>
> In theory something like this could be implemented by adding some revision
> property to specify which commits to ignore. But it might be hard to keep
> things working if there are revisions where this property is supplied, but
> major changes are committed. In that case it would be very hard to show old
> revisions for some lines.
>
> Bert
> >
> > Gabriela
>
>
Received on 2013-10-07 02:30:17 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.