Re: [PATCH] fix exec/Makefile.am

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/22/2012 3:32 PM, Grüninger, Andreas (LGL Extern) wrote:
> Hi
> 
> sed in Solaris don't know the option -i.
> In build-aux/genman sed is used with this option.
> Of course a soft link in is equivalent e.g. "ln -s /opt/csw/bin/gsed /opt/csw/bin/sed" in my build environment.

Ok, then we can simply fix genman to do the right thing and don´t use
gnu extensions to sed.

we probably need to do some parsing for awk/gawk anyway.. and it´s good
practice to write portable shell code.

Fabio

> 
> Andreas 
> 
> -----Ursprüngliche Nachricht-----
> Von: discuss-bounces@xxxxxxxxxxxx [mailto:discuss-bounces@xxxxxxxxxxxx] Im Auftrag von Fabio M. Di Nitto
> Gesendet: Mittwoch, 22. August 2012 10:11
> An: discuss@xxxxxxxxxxxx
> Betreff: Re:  [PATCH] fix exec/Makefile.am
> 
> On 8/22/2012 10:01 AM, Jan Friesse wrote:
>> Hi,
>> thanks for your patches. Because it looks like most of the problems 
>> are hidden in the fact that we are not using libtool, we decided to 
>> migrate master / needle to libtool (Fabio is working on that), what 
>> should solve all problems you were hitting. It would be really nice if 
>> you could test them.
>>
>> cs_queue.h shouldn't really be in TOTEM_SRC and we will remove that.
>>
>> There is no need to patch generated files and change awk to gawk.
>> configure takes AWK environment variable.Fabio is working on patch to 
>> make same possible with sed (so SED environment variable can be set to 
>> gsed).
> 
> I am not entirely sure I get where the SED requirements come from.
> 
> sed is harcoded all over the place in final Makefile(s) and even if we allow to set SED, it won´t have any effect that I can see right away.
> 
> Anyway I am installing freebsd9 and openindiana151a as we speak.
> 
> My only concern is that I don´t have a darwin architecture available for testing at this point in time.
> 
> Fabio
> 
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss
> _______________________________________________
> discuss mailing list
> discuss@xxxxxxxxxxxx
> http://lists.corosync.org/mailman/listinfo/discuss
> 

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss



[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux