petscvariables: hardwired build dir instead of install dir

Barry Smith bsmith at mcs.anl.gov
Mon Mar 24 22:14:07 CDT 2008


    Matt,

     The sed is so trivial it is silly to even think about replacing
it with python!  I did not realize until after reading Lisandro's email
that the sed -i option behaved differently on different systems.

    Barry


On Mar 24, 2008, at 10:07 PM, Matthew Knepley wrote:
> 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