Re: Re: cygwin 'make install' deleted everything!

From: Christian Holm Christensen <cholm_at_nbi.dk>
Date: Fri, 10 Nov 2006 10:56:56 +0100

Hi Fons et al,

On Fri, 2006-11-10 at 09:26 +0100, Fons Rademakers wrote:
> Hi Nick,
>
> that is still the advised procedure. There should be no difference in
> building for fixed location (i.e. with prefix) or without.

OK, I may have caused some confusion. Sorry about that. However, I was raising this issue because of https://savannah.cern.ch/bugs/?19832 and the discussion we had (offline) on that issue.

So, I guess, that the proper way to do things for an environment driven install, is

        gzip -dc root_v5.14.00.source.tar.gz | tar -cf -   # or co from CVS 
        cd root 
        export ROOTSYS=`pwd`                               # IMPORTANT 
        ./configure                                        # No --prefix
        make 
        make install 

> The problem was
> that we just had a bug in detecting that the build location was also the
> install location. This is now fixed.

I see that you have only changed things like

• rm -rf $(DESTDIR)$(TTFFONTDIR)/CVS; \ + (cd $(DESTDIR)$(TTFFONTDIR); rm -rf CVS); \

How does that fix the problem? In what cases are these two lines not equivalent (assuming that $(DESTDIR)$(TTFFONTDIR) exists)? Also, the commands in the parenthesis should really be and'ed together, like

                   (cd $(DESTDIR)$(TTFFONTDIR) && rm -rf CVS); \
        

In this way, the command will not delete `CVS' in the top-level

directory if the directory `$(DESTDIR)$(TTFFONTDIR)' does not exist
(shouldn't happen, but you never know).  Perhaps you also want to put
quotes around `$(DESTDIR)$(TTFFONTDIR)' in case either `$(DESTDIR)' or `

$(TTFFONTDIR)' contains spaces.

Yours,

-- 
 ___  |  Christian Holm Christensen 
  |_| |  -------------------------------------------------------------
    | |  Address: Sankt Hansgade 23, 1. th.  Phone:  (+45) 35 35 96 91
     _|           DK-2200 Copenhagen N       Cell:   (+45) 24 61 85 91
    _|            Denmark                    Office: (+45) 353  25 404
 ____|   Email:   cholm_at_nbi.dk               Web:    www.nbi.dk/~cholm
 | |

Received on Fri Nov 10 2006 - 10:57:15 MET