[lives] Fix multi_encoder3 python interpreter

[Antonio Trande]

On 10/26/2016 01:02 PM, Dominik 'Rathann' Mierzejewski wrote:
> On Tuesday, 25 October 2016 at 23:44, Antonio wrote:
>> commit 0ead6e97fd35fc2627ecacbc084aa0382dfe2272
>> Author: sagitter <sagitter at fedoraproject.org>
>> Date:   Tue Oct 25 23:43:05 2016 +0200
>>
>>     Fix multi_encoder3 python interpreter
>>
>>  lives.spec | 11 ++++++++---
>>  1 file changed, 8 insertions(+), 3 deletions(-)
>> ---
>> diff --git a/lives.spec b/lives.spec
>> index e216f23..9964dd6 100644
>> --- a/lives.spec
>> +++ b/lives.spec
> [...]
>> @@ -143,8 +144,9 @@ rm -rf %{buildroot}%{_docdir}/lives-%{version}
>>  ##Remove rpaths
>>  chrpath -d %{buildroot}%{_bindir}/lives-exe
>>  
>> -# Fix Python2 interpreter
>> -find %{buildroot} -name 'lives_*' -o -name 'multi_encod*' | xargs sed -i '1s|^#!/usr/bin/env python|#!%{__python2}|'
>> +# Fix Python interpreter
>> +find %{buildroot} -name 'lives_*' -o -name 'multi_encoder' | xargs sed -i '1s|^#!/usr/bin/env python|#!%{__python2}|'
>> +find %{buildroot} -name 'lives_*' -o -name 'multi_encoder3' | xargs sed -i '1s|^#!/usr/bin/env python|#!%{__python3}|'
>>  
>>  ##Set Exec key
>>  desktop-file-edit \
> 
> I hope you do realize that this sets the python hashbang to python3 for
> lives_*, not just multi_encoder3. Is that intended? If yes, then what's
> the point of setting it to python2 in the line before?
> 

This commit should be correct:
https://pkgs.rpmfusion.org/cgit/free/lives.git/commit/?id=5f7db3b749699e678e4bb05b32af1e8d99e6c137

(I had totally ignored first -name option in the 'find' commands).

-- 
---
Antonio Trande
mailto: sagitter 'at' fedoraproject 'dot' org
http://fedoraos.wordpress.com/
https://fedoraproject.org/wiki/User:Sagitter
GPG Key: 0x6CE6D08A
Check on https://keys.fedoraproject.org/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.rpmfusion.org/pipermail/rpmfusion-developers/attachments/20161026/4d4edbbf/attachment.sig>