[MPlayer-cvslog] CVS: main get_path.c,1.8,1.9

[Guillaume Poirier]

Hi,

On 4/24/05, Diego Biurrun <diego at biurrun.de> wrote:
> On Fri, Apr 22, 2005 at 05:03:23PM -0400, Rich Felker wrote:
> > On Fri, Apr 22, 2005 at 09:45:41PM +0200, Diego Biurrun wrote:
> > > On Fri, Apr 22, 2005 at 05:31:41PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> > > >
> > > > I vote to ban "-o" altogether.
> > >
> > > Seconded.
> >
> > /me commits the patch:
> >
> > - [all of mplayer.c]
> > + [all of goatse.txt]
> >
> > :))))
> >
> > Aside from stuff like this or mistakes caught within a few seconds of
> > commit, I agree totally, we should abandon -o..
> 
> The command is called admin for a reason, not every user should should
> employ it.  In the case you describe the CVS admin could remove that
> revision (and revoke that person's CVS access).
> 
> What about the others?  Do we have a consensus to abandon 'cvs admin -o'?

I've used it last week as I had screwed a commit on the XML docs. CVS
admin allowed me to redo it again properly, so that people who
translate it afterwards have a chance to work from cleaner commits and
not loose some of their precious time because of me.
So I'd say let's not ban cvs admin when it saves everybody's time.

Regards,
Guillaume