Bug 531773 - Review Request: synfig - Vector-based 2D animation rendering backend
Summary: Review Request: synfig - Vector-based 2D animation rendering backend
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Luya Tshimbalanga
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 428568 (view as bug list)
Depends On:
Blocks: synfigstudio
TreeView+ depends on / blocked
 
Reported: 2009-10-29 13:27 UTC by Lubomir Rintel
Modified: 2009-12-17 22:29 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-12-17 22:29:48 UTC
Type: ---
Embargoed:
luya: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Rintel 2009-10-29 13:27:05 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/synfig.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/synfig-0.61.09-4.fc11.src.rpm  

Description:

Synfig is a powerful, industrial-strength vector-based 2D animation
software, designed from the ground-up for producing feature-film quality
animation with fewer people and resources.  It is designed to be capable of
producing feature-film quality animation. It eliminates the need for
tweening, preventing the need to hand-draw each frame. Synfig features
spatial and temporal resolution independence (sharp and smoothat any
resolution or framerate), high dynamic range images, and a flexible plugin
system.

This package contains the command-line-based rendering backend.
Install synfigstudio package for GUI-based animation studio.

Comment 1 Susi Lehtola 2009-10-29 13:32:04 UTC
*** Bug 428568 has been marked as a duplicate of this bug. ***

Comment 2 Lubomir Rintel 2009-10-29 14:26:25 UTC
"kouzi skrec bild"
http://koji.fedoraproject.org/koji/taskinfo?taskID=1776478

Comment 3 Luya Tshimbalanga 2009-11-12 22:10:51 UTC
- $ rpmlint rpmbuild/SRPMS/synfig-0.61.09-4.fc11.src.rpm synfig.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 64)
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

+ Package complies to Package Naming guideline

+ Spec file name matches the base package %{name}, synfig in this case, in the format %{name}.spec

+ Package meets the Packaging guideline with the removal of rpath listed in spec file

+ License filed in package spec match the factual license which is GPLv2+

+ Text of licnse included in %doc

+ Spec file is legible and written in American English

+ Succesfully build in all architectures http://koji.fedoraproject.org/koji/taskinfo?taskID=1776478

+ Locale handled using %find_lang

+ %post and %postun called

+ No bundle copies of system libraries.

+ All created directories are owned by the package

+ Permission on files is properly set

+ Header files are in -devel package

+ No .la libtool archived

- Missing %{name}.desktop file because synfig is a GuI application

+ All filename in rpm package are valid UTF-8

I am unable to match md5sum because sourceforge seems to be down. Please correct those comments listed in "-" line. Also I had trouble to locally rebuild package with this result:

checking how to run the C++ preprocessor... /lib/cpp
configure: error: in `/home/luya/rpmbuild/BUILD/synfig-0.61.09':
configure: error: C++ preprocessor "/lib/cpp" fails sanity check
See `config.log' for more details.
error: Bad exit status from /var/tmp/rpm-tmp.WcYKNa (%build)

Comment 4 Lubomir Rintel 2009-11-12 23:24:16 UTC
Thanks for the review:

(In reply to comment #3)
> - $ rpmlint rpmbuild/SRPMS/synfig-0.61.09-4.fc11.src.rpm synfig.src: W:
> mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 64)

Fixed. (This tiny change being the only one, I did not bump a release number.)

> - Missing %{name}.desktop file because synfig is a GuI application

It's not, please read the description. synfigstudio is, see bug #479527

> checking how to run the C++ preprocessor... /lib/cpp
> configure: error: in `/home/luya/rpmbuild/BUILD/synfig-0.61.09':
> configure: error: C++ preprocessor "/lib/cpp" fails sanity check
> See `config.log' for more details.
> error: Bad exit status from /var/tmp/rpm-tmp.WcYKNa (%build)  

That's probably you don't have the build group installed [1] (presumably the cpp package). Either install it, or use mock to do the build.

[1] https://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

New package:

SPEC: http://v3.sk/~lkundrak/SPECS/synfig.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/synfig-0.61.09-4.fc11.src.rpm

Comment 5 Luya Tshimbalanga 2009-11-13 02:45:50 UTC
(In reply to comment #4)
> Thanks for the review:
> 
> (In reply to comment #3)
> > - $ rpmlint rpmbuild/SRPMS/synfig-0.61.09-4.fc11.src.rpm synfig.src: W:
> > mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 64)
> 
> Fixed. (This tiny change being the only one, I did not bump a release number.)


> > - Missing %{name}.desktop file because synfig is a GuI application
> 
> It's not, please read the description. synfigstudio is, see bug #479527

Ok. I am also talikg synfigstudio for review as well.
 
> > checking how to run the C++ preprocessor... /lib/cpp
> > configure: error: in `/home/luya/rpmbuild/BUILD/synfig-0.61.09':
> > configure: error: C++ preprocessor "/lib/cpp" fails sanity check
> > See `config.log' for more details.
> > error: Bad exit status from /var/tmp/rpm-tmp.WcYKNa (%build)  
> 
> That's probably you don't have the build group installed [1] (presumably the
> cpp package). Either install it, or use mock to do the build.
> 

It was gcc-c++ package that was not installed although it should with the use of yum-builddep command with src package (might be a bug but it is another story).


Back to the review:

+ $ rpmlint Downloads/synfig-0.61.09-4.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

+  md5sum rpmbuild/SOURCES/synfig-0.61.09.tar.gz Downloads/synfig-0.61.09.tar.gz 
0330521fe46b9e8a39df7081ec64c5d5  rpmbuild/SOURCES/synfig-0.61.09.tar.gz
0330521fe46b9e8a39df7081ec64c5d5  Downloads/synfig-0.61.09.tar.gz

+ sha512sum rpmbuild/SOURCES/synfig-0.61.09.tar.gz Downloads/synfig-0.61.09.tar.gz 
799c076a2ba8d00b02273ee191421ffb2379081107b1d5e101db010bb0c347ea75e89ba17891fe367f0fe2d7bb979e7660317240afb402fe11147c268c0592ea  rpmbuild/SOURCES/synfig-0.61.09.tar.gz
799c076a2ba8d00b02273ee191421ffb2379081107b1d5e101db010bb0c347ea75e89ba17891fe367f0fe2d7bb979e7660317240afb402fe11147c268c0592ea  Downloads/synfig-0.61.09.tar.gz

+ Source package succesfully built on all architecture using Koji and also built locally

In conclusion, the package fully complies with packaging guideline.

Comment 6 Luya Tshimbalanga 2009-11-13 02:47:25 UTC
BTW, once you have created cvs. Make sure to update to latest upstream version (0.62).

Comment 7 Lubomir Rintel 2009-11-13 08:21:38 UTC
Thanks a lot for review Luya!

(In reply to comment #5)
> It was gcc-c++ package that was not installed although it should with the use
> of yum-builddep command with src package (might be a bug but it is another
> story).

That might be a valid point. You could probably try opening a ticket against yum-utils.

(In reply to comment #6)
> BTW, once you have created cvs. Make sure to update to latest upstream version
> (0.62).  

Surely, I'll do.

New Package CVS Request
=======================
Package Name: synfig
Short Description: Vector-based 2D animation rendering backend
Owners: lkundrak
Branches: F-11 F-12 EL-5

Comment 8 Jason Tibbitts 2009-11-13 19:34:16 UTC
CVS done.

Comment 9 Luya Tshimbalanga 2009-12-17 22:29:48 UTC
Closing this report because synfig is available on repository.


Note You need to log in before you can comment on or make changes to this bug.