Bugzilla Bug 163497

Hi!

Please find attached app-laptop/pommed-1.1-r1.ebuild.

Pommed is a daemon managing special features such as screen and keyboard
backlight on Apple MacBook Pro/PowerBook laptops. The ebuild includes gpomme,
which shows gui popups when pommed settings are changed, and wmpomme, a dockapp
showing current settings.

pommed lives here: http://technologeek.org/projects/pommed/index.html

The ebuild seems to work fine. I suggest app-laptop/pommed.

Created an attachment (id=107963) [details]
ebuild

The ebuild posted above will not work, because it relies on a dbus.eclass I
wrote specially for it (I am the original ebuild author BTW). pommed also
require libsmbios, which is not yet in the tree.

I'll attach a tar.bz2 containing ebuilds for pommed-1.1, pommed-9999,
libsmbios-0.12.4 (latest) and the appropriate dbus.eclass.

The ebuild for libsmbios is largelly perfectible (doesn't install doc despite
USE=doc for example). Feel free to enhance it.

dbus.eclass contains a single function 'newdbussystemd' which installs a dbus
configuration in the appropriate location (/etc/dbus-1/system.d). Again, if you
don't like an eclass, feel free to merge it into pommed ebuilds which use it.

Please find the tarball below. It can also be downloaded here:
http://pico.is.free.fr/pommed-overlay-20070205.tar.bz2

Created an attachment (id=110041) [details]
ebuilds for pommed, libsmbios

Created an attachment (id=110071) [details]
ebuilds for pommed, libsmbios with updated init script

This one contains an updated pommed init.d script with a depend on alsasound.
This is needed to allow pommed to adjust sound levels.

Regards,

where to get pommed-1.1.tar.gz ? it's not on alioth.debian.org (any more?)

https://alioth.debian.org/frs/?group_id=31066

Created an attachment (id=110800) [details]
pommed-1.1 source file

Created an attachment (id=110802) [details]
pommed-1.1.tar.gz

Sorry for my previous failed attempt :P

Created an attachment (id=111672) [details]
updated ebuilds

Updated ebuilds:
- 1.1 is clean and commented
- 9999 is updated against latest revisions in SVN

In cvs now
Thanks

(In reply to comment #9)
> In cvs now
> Thanks

Thanks to you, your ebuild is much cleaner than mine :)

There are a few details that could be enhanced, however:
- KEYWORDS: pommed is supposed to work on amd64 (new MacBook and MacBook Pro)
and ppc (iBook, PowerBook) as well
- DEPEND: pommed depends on libsmbios only on x86 and amd64
- config file: the config file is supposed to be "/etc/pommed.conf". You should
install either pommed.conf.mactel or pommed.conf.pmac depending on x86|amd64 or
ppc

I'll attach a patch below to fix these issues.

Anyways thanks for getting pommed in the tree :)

Created an attachment (id=112157) [details]
pommed-1.2.ebuild.patch

Here's the patch

Regards,

(In reply to comment #10)
> (In reply to comment #9)
> > In cvs now
> > Thanks
> 
> Thanks to you, your ebuild is much cleaner than mine :)
> 
> There are a few details that could be enhanced, however:
> - KEYWORDS: pommed is supposed to work on amd64 (new MacBook and MacBook Pro)
> and ppc (iBook, PowerBook) as well
> - DEPEND: pommed depends on libsmbios only on x86 and amd64

Only arch teams that can actually test on that architecture can add keywords so
you need to file keywording request bugs to them if you want stuff keyworded
for those arches.

> - config file: the config file is supposed to be "/etc/pommed.conf". You should
> install either pommed.conf.mactel or pommed.conf.pmac depending on x86|amd64 or
> ppc
> 
> I'll attach a patch below to fix these issues.
> 
> Anyways thanks for getting pommed in the tree :)
> 

Reopening for the patch. Sometimes it's better to just open a new bug for those
issues so that you can be sure it does not get ignored if for example the
notification mail is not acted upon.

Reassigning to cedk or he won't see the changes (if I still have my privs here,
that is).

Also, the gtk+ dependency sounds broken, it doesn't depend on a proper version,
which means that it would be satisfied by 1.2 (this supposing it uses 2.0
instead, haven't had time to look at it yet).

Fix in cvs
Thanks