Bug 215571 - Review Request: heliodor - Beryl window decorator with Metacity theme support
Summary: Review Request: heliodor - Beryl window decorator with Metacity theme support
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Package Reviews List
Depends On: 209259
TreeView+ depends on / blocked
Reported: 2006-11-14 17:30 UTC by Jarod Wilson
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Clone Of:
Last Closed: 2006-11-21 20:55:30 UTC

Attachments (Terms of Use)
Mock build log of heliodor 0.1.2-1 (61.34 KB, text/plain)
2006-11-16 04:22 UTC, Mamoru TASAKA
no flags Details
Mock build log of heliodor 0.1.2-2 (57.40 KB, text/plain)
2006-11-17 07:05 UTC, Mamoru TASAKA
no flags Details

Description Jarod Wilson 2006-11-14 17:30:26 UTC
Spec URL: http://wilsonet.com/packages/beryl/heliodor.spec
SRPM URL: http://wilsonet.com/packages/beryl/heliodor-0.1.2-1.fc6.src.rpm
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

Heliodor is a gtk-window-decorator for use with beryl
that supports using Metacity themes.

Depends on beryl-core, submitted for FE-review under bug 209259.

Comment 1 Mamoru TASAKA 2006-11-16 04:22:52 UTC
Created attachment 141338 [details]
Mock build log of heliodor 0.1.2-1

I cannot rebuild this by mockbuild on FC-devel i386.

Comment 2 Jarod Wilson 2006-11-16 23:46:51 UTC
Here's an updated build with trimmed down BR:. It builds in FC6/x86_64 mock for
me, will double-check with devel builds tomorrow, if you don't beat me to it
(gotta run for the night).


Comment 3 Mamoru TASAKA 2006-11-17 07:05:32 UTC
Created attachment 141451 [details]
Mock build log of heliodor 0.1.2-2

Umm.. still not okay on FC-devel i386.

Comment 4 Jarod Wilson 2006-11-17 19:37:38 UTC
Okay, this is due to changes in metacity 2.17 in rawhide. The *_corner_rounded
bits were renamed to *_corner_rounded_radius. Easy enough to patch around. New
version builds in devel/x86_64 mock and on FC6/x86_64 for me:


Comment 5 Mamoru TASAKA 2006-11-18 14:28:44 UTC
Well, this time mockbuild for FC-devel i386 is okay.

* Requires: metacity, control-center
  - Not necessary as libraries' dependency automaticall pulls

? /usr/share/gnome/wm-properties/
  - This directory is not owned by any package, however, I don't
    know which package should own this.
    (This package requires metacity, so my thought is that this
     should be metacity, however, do you have any idea?)

    Currently, my system has the following files under
beryl.desktop     (by heliodor-0.1.2-3.fc7)
compiz.desktop    (by compiz-0.3.2-2.fc7)
metacity.desktop  (by metacity-2.17.2-1.fc7)

? For license:
  - lib/beryl-window-manager.c seems to be "free" (not GPL however
  less strict than GPL)
  - src/main.c says this is LGPL (not GPL) (why?)
  - and there is a copy license that this is GPL.....

  Well, however, I trust the document license and I assume that
  this package is licensed under GPL for now.

Other things are okay.

  This packag (beliodor) is APPROVED by me.

Comment 6 Jarod Wilson 2006-11-20 21:51:47 UTC
Thanks much! I've removed the Reqs: for the build I'll import, and the directory
ownership problem has been filed against metacity as bug 216514.

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