Bug 209262 - Review Request: beryl-plugins - Beryl OpenGL window and compositing manager plug-ins
Summary: Review Request: beryl-plugins - Beryl OpenGL window and compositing manager ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michał Bentkowski
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 209259
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-10-04 06:21 UTC by Jarod Wilson
Modified: 2008-07-13 05:35 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-11-21 21:01:05 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jarod Wilson 2006-10-04 06:21:05 UTC
Spec URL: http://wilsonet.com/packages/beryl/beryl-plugins.spec
SRPM URL: http://wilsonet.com/packages/beryl/beryl-plugins-0.1.0-1.fc6.src.rpm
Description:
--
Beryl is a combined window manager and compositing
manager that runs on top of Xgl or AIGLX using OpenGL
to provide effects accelerated by a 3D graphics card
on the desktop. Beryl is a community-driven fork of
Compiz.

Beryl has a flexible plug-in system, which the
contents of this package take advantage of.
--

NOTE: This package depends on beryl-core, under review here: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209259.

Comment 1 Jarod Wilson 2006-10-04 06:38:37 UTC
Oops, this is beryl-plugins, not beryl-settings, fixing Summary...

Comment 2 Jarod Wilson 2006-10-26 15:40:02 UTC
Just pushed out a 0.1.1-1 build.

http://wilsonet.com/packages/beryl/beryl-plugins-0.1.1-1.fc6.src.rpm

Comment 3 Chitlesh GOORAH 2006-11-02 17:04:45 UTC
This is a RFE:
chitlesh(~)[0]$rpm -qf /usr/share/beryl/cubecaps.png
beryl-plugins-0.1.1-2.fc6

from your repo.

could you remplace that file with /usr/share/rhgb/main-logo.png ?

It's more fedora this way :)



Comment 4 Jarod Wilson 2006-11-06 16:54:11 UTC
Rather than replace that file, I've added a Source1 of fedora-cubecaps.png,
which is a copy of rhgb main-logo.png, and edited the default cubecaps setting
to point to the fedora-cubecaps.png. There's a -3 spec and srpm out there now.

Comment 5 Chitlesh GOORAH 2006-11-06 17:12:41 UTC
By replacing I mean, asking beryl to pickup /usr/share/rhgb/main-logo.png.

A better solution might be:
Add fedora-logos-6.0.6-1.fc6 as requires and patch/sed appropriately :)

rpm -qf /usr/share/rhgb/main-logo.png
fedora-logos-6.0.6-1.fc6

Comment 6 Jarod Wilson 2006-11-06 18:30:05 UTC
Good idea, I'll go that route instead.

Comment 7 Jarod Wilson 2006-11-14 19:15:14 UTC
Latest build, using new upstream tarballs:

http://wilsonet.com/packages/beryl/beryl-plugins-0.1.2-2.fc6.src.rpm

Comment 8 Michał Bentkowski 2006-11-15 17:38:15 UTC
MUST items:
 * rpmlint is quiet
 * package is named well
 * spec file name is good
 * package meets Packaging Guidelines
 * package is licensed with a GPL open-source compatible license
 * License field in spec file matches actual license
 * license file is included in %doc
 * md5sums are matching (6721bf8bd5fcbe0cabb0a67caa920fcb)
 * package successfully compiles on x86_64
 * BuildRequires listed well (mock builds successfully)
 * no locales
 * no need to %post and %postun sections
 * not relocatable
 * package owns directories well
 * no duplicates in %files
 * every %files section includes %defattr
 * proper %clean section
 * macros used well

Looks pretty good for me :)
Approved


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