[Bug 3694] Review request: flowblade - A fast, efficient video editor

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3694

--- Comment #8 from MartinKG <mgansser at alice.de> 2015-06-21 11:09:03 CEST ---
(In reply to comment #6)
> Created attachment 1449 [details]
> fix to  /usr/bin/flowblade
> 
> Hi, 
> 1- Now to run flowblade,  we need this patch, if no better alternative ,
> please add patch in flowblade.spec 

done
> 
> 2- Please build on %build and install on %install with:
> 
>   %build 
>   python setup.py build
> 
>   %install 
>   python setup.py install --skip-build --root %{buildroot} 
>

done
> 3- Still need improve locales, why you use : 
>   %exclude %{python_sitelib}/Flowblade/locale ? , they aren't in package ! 
> 

because the files are listed twice
warning: File listed twice:
/usr/lib/python2.7/site-packages/Flowblade/locale/cs/LC_MESSAGES/Flowblade.mo

> 4- About /usr/libexec/flowblade file [1] , sorry, after analyze better it is a
> mimetype with action !?! So maybe better is delete after the do setup.py
> install , I don't know , for me still a mystery where put this file . 
> 
> I think it all to accept the package 
> Thanks,
> 
> [1]
> cat /usr/libexec/flowblade 
> application/vnd.flowblade-project; /usr/bin/flowblade '%s'; test=test -n
> "$DISPLAY"; priority=5

Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/flowblade.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/flowblade-1.0.0-3.git94f69ce.fc22.src.rpm

%changelog
* Sun Jun 21 2015 Martin Gansser <martinkg at fedoraproject.org> -
1.0.0-3.git94f69ce
- added flowblade.patch
- put setup.py into %%build section

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.