petscvariables: hardwired build dir instead of install dir

Matthew Knepley knepley at gmail.com
Mon Mar 24 22:07:43 CDT 2008


On Mon, Mar 24, 2008 at 9:50 PM, Barry Smith <bsmith at mcs.anl.gov> wrote:
>
>  On Mar 24, 2008, at 10:57 AM, Lisandro Dalcin wrote:
>  > Barry, things are still broken. I think that at some point we have to
>  > review the 'install:' target  more carefully.
>  >
>  > First, the 'sed' command i being called in a wrong way.
>
>     This is not true; the sed is being called correctly. The problem
>  is that -i
>  is not a standard sed option and different systems gnu and freebsd treat
>  it differently. freebsd requires a space between the -i and the suffix;
>  gnu has no space; gnu also allows the use of -i to indicate no backup
>  while freebsd expects -i ""
>
>    Your patch works on POS gnu systems, but is broken on far superior
>  Apple MacOS X systems! :-)
>
>     Matt you need to add a config/configure.py test to detect the
>  type of sed -i it is.

I totally disagree. We should ditch all this crap, and just write nice, PORTABLE
Python code. I will do it. I just need someone to explain what this
sed is doing.

   Matt

-- 
What most experimenters take for granted before they begin their
experiments is infinitely more interesting than any results to which
their experiments lead.
-- Norbert Wiener




More information about the petsc-dev mailing list