[prev in list] [next in list] [prev in thread] [next in thread] 

List:       opensuse-buildservice
Subject:    Re: [opensuse-buildservice] osc rdiff (was Re: Novell ID's for OSC Plugins)
From:       Adrian =?iso-8859-15?q?Schr=F6ter?= <adrian () suse ! de>
Date:       2007-11-29 20:50:21
Message-ID: 200711292150.22046.adrian () suse ! de
[Download RAW message or body]

On Thursday 29 November 2007 18:25:45 wrote Dr. Peter Poeml:
> On Thu, Nov 29, 2007 at 05:52:24PM +0100, Dr. Peter Poeml wrote:
> > On Thu, Nov 29, 2007 at 05:39:21PM +0100, Adrian Schröter wrote:
> > > On Thursday 29 November 2007 17:21:43 wrote Dr. Peter Poeml:
> > > > On Thu, Nov 29, 2007 at 01:33:47PM +0100, Adrian Schröter wrote:
> > >
> > > ...
> > >
> > > > > an equivalent for cvs rdiff would work without any checked out
> > > > > sources, just
> > > > >
> > > > >  osc rdiff <TO_PROJECT> <TO_PACKAGE> [--package <FROM_PACKAGE>]
> > > > > [--project <FROM_PROJECT>]
> > > >
> > > > It'd also need options for revision and orevision.
> > > >
> > > > How about
> > > >
> > > > osc rdiff [-rN[:M]] [[--oldprj OLD_PROJECT] --oldpkg OLD_PACKAGE]]
> > > > NEW_PROJECT NEW_PACKAGE
> > >
> > > looks even better to me :)
> > >
> > > > One should note that the result is a "pretty diff", which goes into
> > > > tar balls and is handy for reviews -- not a normal diff which yields
> > > > an appliable patch.
> > > >
> > > > > hm, maybe even default to "--project openSUSE:Factory" if no
> > > > > argument is given ...
> > > >
> > > > Maybe a good idea...
> >
> > While experimenting, I noticed that
> > https://api.opensuse.org/source/home:poeml/initviocons?cmd=diff&oproject=
> >home%3Apoeml&opackage=initviocons returns a non-empty diff, even though
> > package=opackage and
> > project=oproject.
> >
> > Is this a diff to the last revision? It looks like that.

yes

> I hacked up an rdiff command, and committed it to svn.

cool thing :)

> I hope it works as expected.
>
> The default behaviour, if opackage and oproject are not specified, is a
> diff against the last revision -- not against Factory. So it behaves
> similar like the backend.

k, maybe we can reconsider this behaviour later, when we have merge request 
mechanism and know where the package is intended to be submitted ...

> The patch is also applied to the package tracking branch.
>
> Peter



-- 

Adrian Schroeter
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)
email: adrian@suse.de

---------------------------------------------------------------------
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org
For additional commands, e-mail: opensuse-buildservice+help@opensuse.org


[prev in list] [next in list] [prev in thread] [next in thread]