Re: [PATCH] random:cipher: handle substitution in sed command

Jussi Kivilinna <[email protected]>
Newsgroups gmane.comp.encryption.gpg.libgcrypt.devel
Message-ID <[email protected]>
Hello,

On 24.6.2024 17.39, Friedrich Haubensak via Gcrypt-devel wrote:
> 
> dear all,
> 
> the patch below did not make it yet into the libgcrypt source tree. Note: cipher/Makefile.am up to now did not have the g flag at the first -e.... -- i can not judge if it would be a Good Thing to add it (probably it is :-).  but the patch to random/Makefile.am seems neccessary:
> 
> g flag at the first -e's....' in random/Makefile.am was dropped in b99952a on 7 May 2024.  unfortunately, the missing g flag now breaks install of libgcrypt-1.11.0 in "gentoo prefix" instance, as some automagic there inserts a double "-O2 -pipe" into the CFLAGS during build.
> 
> thank you for consideration.

Thanks for reminder, this had fallen through the cracks. Patch has been applied to master branch.

-Jussi

> 
> 
> On 6/11/24 03:52, simit.ghane at lge.com (simit.ghane) wrote:
>> It was there earlier and accidentally removed from
>> Makefile.am of cipher and random
>>
>> Signed-off-by: simit.ghane <simit.ghane at lge.com>
>> ---
>>   cipher/Makefile.am | 2 +-
>>   random/Makefile.am | 2 +-
>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/cipher/Makefile.am b/cipher/Makefile.am
>> index a914ed2ba4..956ca345e3 100644
>> --- a/cipher/Makefile.am
>> +++ b/cipher/Makefile.am
>> @@ -168,7 +168,7 @@ gost-s-box$(EXEEXT_FOR_BUILD): gost-s-box.c
>>   if ENABLE_O_FLAG_MUNGING
>> -o_flag_munging = sed -e 's/[[:blank:]]-O\([2-9sgz][2-9sgz]*\)/ -O1 /' -e 's/[[:blank:]]-Ofast/ -O1 /g'
>> +o_flag_munging = sed -e 's/[[:blank:]]-O\([2-9sgz][2-9sgz]*\)/ -O1 /g' -e 's/[[:blank:]]-Ofast/ -O1 /g'
>>   else
>>   o_flag_munging = cat
>>   endif
>> diff --git a/random/Makefile.am b/random/Makefile.am
>> index c7100ef8b8..a42e430649 100644
>> --- a/random/Makefile.am
>> +++ b/random/Makefile.am
>> @@ -56,7 +56,7 @@ jitterentropy-base.c jitterentropy.h jitterentropy-base-user.h
>>   # The rndjent module needs to be compiled without optimization.  */
>>   if ENABLE_O_FLAG_MUNGING
>> -o_flag_munging = sed -e 's/[[:blank:]]-O\([1-9sgz][1-9sgz]*\)/ -O0 /' -e 's/[[:blank:]]-Ofast/ -O0 /g'
>> +o_flag_munging = sed -e 's/[[:blank:]]-O\([1-9sgz][1-9sgz]*\)/ -O0 /g' -e 's/[[:blank:]]-Ofast/ -O0 /g'
>>   else
>>   o_flag_munging = cat
>>   endif
> 
> _______________________________________________
> Gcrypt-devel mailing list
> [email protected]
> https://lists.gnupg.org/mailman/listinfo/gcrypt-devel


_______________________________________________
Gcrypt-devel mailing list
[email protected]
https://lists.gnupg.org/mailman/listinfo/gcrypt-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.