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

List:       kde-core-devel
Subject:    Re: Code duplication in KWrite, comment/uncomment stuff
From:       Stephan Kulow <coolo () kde ! org>
Date:       2000-05-29 9:32:50
[Download RAW message or body]

Simon Hausmann wrote:
> 
> On Sun, 28 May 2000, David Faure wrote:
> 
> > On Sun, May 28, 2000 at 06:23:05PM +0200, Simon Hausmann wrote:
> > >
> > >
> > > On Sat, 27 May 2000, Ralf Nolden wrote:
> > >
> > > > Simon Hausmann wrote:
> > > > >
> > > > > I neither can nor want to decide on this issue :-}
> > > > >
> > > > > There are two valid points:
> > > > >
> > > > > - KWrite is an application. kdelibs contains *core* library stuff
> > > > >   Although I agree that a text editor component is indeed a core
> > > > >   component of a desktop environment.
> > > > >
> > > > > - However the fact that kdevelop would depend on kdebase isn't really a
> > > > >   good thing either.
> > > > >
> > > > > How about this:
> > > > >
> > > > > We make kwrite a separate CVS module and "link" it into the kdebase and
> > > > > the kdevelop CVS module via CVSROOT/modules , just like kde-common/admin .
> > > > >
> > > > > What do you think?
> > > > I think that would be a suitable solution. After all, itīs just to keep
> > > > kwrite sourcecode in sync, isnīt it ?
> > >
> > > Yes
> > >
> > >
> > > Now we need the help of the CVS Masters :-) Coolo, David, anyone
> > > listening? :-)
> >
> > Note that the link thing means that everyone will need to recheckout
> > completely kdebase to get the stuff, since cvs update doesn't do it
> > (for some reason).
> 
> OK, that's a very important point. That makes moving kwrite into a
> separate module a nono :-)
> 
> The other option would be to directly link from kdebase/kwrite? This way
> only the kdevelop hackers would have to re-checkout >;-)
> 
I favor that. And note that re-checkout doesn't mean to remove it first

Greetings, Stephan

-- 
... but you ain't had mine

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

Configure | About | News | Add a list | Sponsored by KoreLogic