[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: kcontrol/infocenter patch
From: Helge Deller <deller () gmx ! de>
Date: 2002-10-30 22:58:14
[Download RAW message or body]
On Wednesday 30 October 2002 23:51, David Faure wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On Wednesday 30 October 2002 23:42, Helge Deller wrote:
> > On Wednesday 30 October 2002 23:30, David Faure wrote:
> > > -----BEGIN PGP SIGNED MESSAGE-----
> > > Hash: SHA1
> > >
> > > On Wednesday 30 October 2002 23:20, Helge Deller wrote:
> > > > Running "make install" in kdebase/kcontrol/kcontrol produces
> > > > currently the following "Error" if KDE hasn't been installed yet:
> > > >
> > > > rm: cannot remove `/opt/kde/bin/kinfocenter': No such file or
> > > > directory make[4]: [install-data-local] Error 1 (ignored)
> > > >
> > > > The attached patch fixes this for me.
> > > > Ok to commit ?
> > >
> > > A simple -f after rm would do the job too ;)
> > > (No need to create the symlink in the build dir)
> >
> > No, I think we had that once in the Makefile and it broke on
> > some other arches than i386.
>
> AFAIK nobody complained about e.g. kdebase/kcontrol/fileshare/Makefile.am
> since it uses "-rm -f".
>
> Or to take something older, kdelibs/kdoctools/Makefile.am:
> xslt_pure.o: xslt.cpp
> -rm -f xslt_pure.cpp
> $(LN_S) $(srcdir)/xslt.cpp xslt_pure.cpp
>
> I don't think it breaks anywhere.
Ok, then I'll commit your "-f" solution, but I still think my second patch is a nicer solution :-)
Thanks,
Helge
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic