Bug 787517 (dawati-artwork)

Summary: Review Request: dawati-artwork - Artwork for the Dawati UX
Product: [Fedora] Fedora Reporter: Peter Robinson <pbrobinson>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: notting, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-11-25 15:21:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Peter Robinson 2012-02-05 22:36:05 UTC
SPEC: http://pbrobinson.fedorapeople.org/dawati-artwork.spec
SRPM: http://pbrobinson.fedorapeople.org/dawati-artwork-0.90.6-1.fc17.src.rpm

Description:
The artwork, cursors, icons, backgrounds and sounds for the Dawati UX

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3763661

Comment 1 Mario Blättermann 2012-08-05 18:19:49 UTC
Successful scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4361345

Some initial comments:

The source file headers of the gtk engine contain the GPLv2, please add it to the license declaration of your package.

Drop libtool from BuildRequires, because it is needed by gnome-common anyway.

%defattr is only needed if you want to provide your package for EPEL <6.


Wouldn't it be better to provide the gtk engine in a separate package, including the gtk2 themes? The other stuff could also go into a noarch subpackage. Moreover, I'm unsure if dawati-backgrounds.xml needs an extra runtime requirement. For now, the binary package needs the following:

libatk-1.0.so.0()(64bit)  
libc.so.6()(64bit)  
libc.so.6(GLIBC_2.2.5)(64bit)  
libc.so.6(GLIBC_2.3.4)(64bit)  
libcairo.so.2()(64bit)  
libfontconfig.so.1()(64bit)  
libfreetype.so.6()(64bit)  
libgdk-x11-2.0.so.0()(64bit)  
libgdk_pixbuf-2.0.so.0()(64bit)  
libgio-2.0.so.0()(64bit)  
libglib-2.0.so.0()(64bit)  
libgobject-2.0.so.0()(64bit)  
libgtk-x11-2.0.so.0()(64bit)  
libpango-1.0.so.0()(64bit)  
libpangocairo-1.0.so.0()(64bit)  
libpangoft2-1.0.so.0()(64bit)  
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rtld(GNU_HASH)  
rpmlib(PayloadIsXz) <= 5.2-1

I'm missing metacity here. Maybe the package should be split into even more subpackages.

Comment 2 Mario Blättermann 2012-08-29 19:10:58 UTC
Any progress with solving the mentioned problems?

Comment 3 Mario Blättermann 2012-09-20 22:21:37 UTC
Just a reminder...

Comment 4 Mario Blättermann 2012-10-20 19:21:40 UTC
Just a reminder, sent out once a month ... as usual.

If you are no longer interested in to work on this package, please let us know.