Re: debxo ohmd testers wanted

2009-05-25 Thread Holger Levsen
Hi James,

is this still curent? I suffered some personal fubar, thus my late reply :(

Do you have a source package ready to look at? Is ohmd still the daemon to 
use?

On Montag, 9. März 2009, qu...@laptop.org wrote:
  control:
  Standards-Version should be upgraded to 3.8.0 (after checking)
 I don't know how to check.

see /usr/share/doc/debian-policy/upgrading-checklist.txt.gz

  long description is too short (you wrote more in the mail I'm replying
  to :)
 Fixed.  See patch attached.
  misses homepage: and vcs-pseudoheaders
 Fixed homepage, don't understand vcs-pseudoheaders.

Thanks. Vcs-pseudoheaders are something like this:

Vcs-Git: git://git.debian.org/git/collab-maint/sugar-calculate-activity.git
Vcs-Browser: 
http://git.debian.org/?p=collab-maint/sugar-calculate-activity.git;a=summary

Instead of Git you can use any other vcs...

  copyright:
  misses copyright owner
  misses download location
  misses short GPL blurbs
  (see
  http://lists.debian.org/debian-devel-announce/2006/03/msg00023.html)
  mentions GPL but links to GPL-2
 Fixed.

Thanks.

  README.Debian
  unneeded with current content (see above)
 Replaced with different content.

Thanks.

  changelog:
  version number should rather be something like 0.1.1.git20090304-1
 Left as is, until you join in.

I'm still willing to help, and I should also be able to fullfil this promise 
now.

  we should file an ITP bug, so we can close it here ;) (rather to tell
  others about out intentions to put this software into Debian)
 Don't know how to do that.

Will do, if its still needed. (See the questions at the top of this mail :)

  postrm+postinst:
  unconditionally ending with exit 0 looks wrong
 Okay.  107 of 1051 packages on my system
 /var/lib/dpkg/info/*.postinst ended with exit 0.  Should the package
 fail to install if update-rc.d fails?

Yes, i think so.

  are those files needed at all? I mean, shouldnt debhelper/cdbs create
  them, if they contain autogenerated code only?
 Don't know.

I guess they can go then.

  rules:
  simple-patchsys.mk is unused, and if, I'd prefer patchsys-quilt.mk
 Don't know.

Just remove that line.

  Do you have a vcs where you maintain this?
 Created just now.
 git clone http://dev.laptop.org/~quozl/ohm.git/

Ah, cool.


regards  sorry again,
Holger



signature.asc
Description: This is a digitally signed message part.
___
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel


Re: debxo ohmd testers wanted

2009-03-09 Thread quozl
On Thu, Mar 05, 2009 at 04:49:37PM +0100, Holger Levsen wrote:
 Would you like to (co-)maintain the package in Debian? I'd be happy to
 sponsor uploads and/or co-maintain.

I guess so, but I don't know what that requires.

 Comments on files I looked at so far:
 
 control: 
   Standards-Version should be upgraded to 3.8.0 (after checking)

I don't know how to check.

   long description is too short (you wrote more in the mail I'm replying 
 to :)

Fixed.  See patch attached.

   misses homepage: and vcs-pseudoheaders

Fixed homepage, don't understand vcs-pseudoheaders.

 copyright:
   misses copyright owner
   misses download location
   misses short GPL blurbs
   (see 
 http://lists.debian.org/debian-devel-announce/2006/03/msg00023.html)
   mentions GPL but links to GPL-2

Fixed.

 README.Debian
   unneeded with current content (see above)

Replaced with different content.

 changelog:
   version number should rather be something like 0.1.1.git20090304-1

Left as is, until you join in.

   we should file an ITP bug, so we can close it here ;) (rather to tell 
   others about out intentions to put this software into Debian)

Don't know how to do that.

 postrm+postinst:
   unconditionally ending with exit 0 looks wrong

Okay.  107 of 1051 packages on my system
/var/lib/dpkg/info/*.postinst ended with exit 0.  Should the package
fail to install if update-rc.d fails?

   are those files needed at all? I mean, shouldnt debhelper/cdbs create 
 them,
   if they contain autogenerated code only?

Don't know.

 rules:
   simple-patchsys.mk is unused, and if, I'd prefer patchsys-quilt.mk

Don't know.

 Do you have a vcs where you maintain this?

Created just now.

git clone http://dev.laptop.org/~quozl/ohm.git/

-- 
James Cameronmailto:qu...@us.netrek.org http://quozl.netrek.org/
diff --git a/debian/README.Debian b/debian/README.Debian
index aa31ee0..1614f1f 100644
--- a/debian/README.Debian
+++ b/debian/README.Debian
@@ -1,7 +1,16 @@
-README for Debian-style packaging of ohmd
+README for Debian packaging of ohmd
 by James Cameron qu...@laptop.org
 
-This package was built from source downloaded using
-git clone git+ssh://qu...@dev.laptop.org/git/projects/ohm
+- receives and generates D-Bus events,
 
- -- James Cameron qu...@us.netrek.org, Wed,  4 Mar 2009 15:22:10 +1100
+- maintains a connection to the X server using an .Xauthority file in
+  /home/olpc or /root ... without this the detection of keyboard and
+  mouse idle is not performed,
+
+- directly uses /sys entries specific to the OLPC XO-1,
+
+- with gnome-power-manager installed the user will see a power dialog
+  in response to a power button press, often after the XO has been
+  woken from suspend.
+
+ -- James Cameron qu...@us.netrek.org, Mon,  9 Mar 2009 20:11:51 +1100
diff --git a/debian/changelog b/debian/changelog
index e49f532..5cbfc50 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,8 +1,13 @@
 ohmd (0.1.1-6.21.20090304git.quozl-1) unstable; urgency=low
 
   * start ohmd on install
+  * start ohmd after hald (fixes a segfault)
+  * add homepage
+  * fix copyright
+  * fix build dependencies for Lenny, tested in a pbuilder
+  * add a brief system description to README.Debian
 
- -- James Cameron qu...@laptop.org  Fri, 06 Mar 2009 11:44:22 +1100
+ -- James Cameron qu...@laptop.org  Mon, 09 Mar 2009 20:43:51 +1100
 
 ohmd (0.1.1-6.21.20090304git.quozl) unstable; urgency=low
 
diff --git a/debian/control b/debian/control
index 690f129..ea32096 100644
--- a/debian/control
+++ b/debian/control
@@ -2,11 +2,14 @@ Source: ohmd
 Maintainer: James Cameron qu...@laptop.org
 Section: unknown
 Priority: optional
-Build-Depends: debhelper (= 4.2.32), gtk-doc-tools, libhal-dev
+Build-Depends: debhelper (= 4.2.32), cdbs, gtk-doc-tools, libhal-dev, libglib2.0-dev, libdbus-glib-1-dev, libgtk2.0-dev
 Standards-Version: 3.7.3
 
 Package: ohmd
 Architecture: any
 Depends: ${shlibs:Depends}
 Description: Open Hardware Manager, for OLPC XO-1 Laptops
- Provides suspend on lid close, and suspend on idle.
+ For OLPC XO-1 laptops, provides suspend on power button press,
+ suspend on lid close, display dimming on idle, and suspend on
+ extended idle.
+Homepage: http://ohm.freedesktop.org/
diff --git a/debian/copyright b/debian/copyright
index a9a6903..237b7a3 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -1,4 +1,35 @@
-Copyright: GPL
+This package was debianised by James Cameron qu...@laptop.org on
+2009-03-09.
 
-On Debian and Ubuntu GNU/Linux systems, the complete text of the GNU General
-Public License can be found in `/usr/share/common-licenses/GPL-2'.
+The current Debian maintainer is YOUR NAME y...@email.address
+
+It was downloaded from: 
+
+git clone git+ssh://qu...@dev.laptop.org/git/projects/ohm
+
+/*
+ * Copyright (C) 2007 Richard Hughes rich...@hughsie.com
+ *
+ * Licensed under the GNU Lesser General Public License Version 2
+ *
+ * This program 

Re: debxo ohmd testers wanted

2009-03-09 Thread James Cameron
Released new version of package with following changes:

1.  built for Debian Lenny, no need to swing sources.list to squeeze,

2.  starts ohmd on boot, no need to manually start,

3.  fix a few of the packaging issues reported by Holger.

http://dev.laptop.org/~quozl/debian/ohmd_0.1.1-6.21.20090304git.quozl-1_i386.deb

Tested on an XO for half an hour.  Test reports wanted.  None posted
publically so far.

-- 
James Cameronmailto:qu...@us.netrek.org http://quozl.netrek.org/
___
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel


Re: debxo ohmd testers wanted

2009-03-05 Thread Holger Levsen
Hi James,

On Mittwoch, 4. März 2009, James Cameron wrote:
 I've packaged ohmd for debxo, and done some minimal testing.  I'd like
 other debxo users to have a go at it.

Cool. 

 Packaging improvements welcome.  I'm yet to learn some tricks.

Would you like to (co-)maintain the package in Debian? I'd be happy to sponsor 
uploads and/or co-maintain.

Comments on files I looked at so far:

control: 
Standards-Version should be upgraded to 3.8.0 (after checking)
long description is too short (you wrote more in the mail I'm replying 
to :)
misses homepage: and vcs-pseudoheaders

copyright:
misses copyright owner
misses download location
misses short GPL blurbs
(see 
http://lists.debian.org/debian-devel-announce/2006/03/msg00023.html)
mentions GPL but links to GPL-2

README.Debian
unneeded with current content (see above)

changelog:
version number should rather be something like 0.1.1.git20090304-1
we should file an ITP bug, so we can close it here ;) (rather to tell 
others about out intentions to put this software into Debian)

postrm+postinst:
unconditionally ending with exit 0 looks wrong
are those files needed at all? I mean, shouldnt debhelper/cdbs create 
them,
if they contain autogenerated code only?

rules:
simple-patchsys.mk is unused, and if, I'd prefer patchsys-quilt.mk


Do you have a vcs where you maintain this?


regards,
Holger


signature.asc
Description: This is a digitally signed message part.
___
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel


debxo ohmd testers wanted

2009-03-03 Thread James Cameron
G'day,

I've packaged ohmd for debxo, and done some minimal testing.  I'd like
other debxo users to have a go at it.

ohmd when running provides:

- suspend on power button press,
- suspend on lid close,
- display dimming on idle,
- optional suspend on extended idle.

To test it, turn off any other hardware management scripts you may have,
configure your sources.list for squeeze [1], download the .deb [2],
install it [3], resolve the dependencies [4], and start ohmd [5].  Then
see if the features listed above work.  Let me know the result.

Further details:

[1] edit the file /etc/apt/sources.list and change the word lenny to
squeeze, then run apt-get update,

[2] use wget or some other method to download the file 
http://dev.laptop.org/~quozl/debian/ohmd_0.1.1-6.21.20090304git.quozl_i386.deb

[3] install it using dpkg,

dpkg --install ohmd_0.1.1-6.21.20090304git.quozl_i386.deb

which will generate an error regarding dependencies.

[4] resolve the dependencies using apt,

apt-get -f install

[5] start ohmd either manually by typing ohmd, or by running;

update-rc.d ohmd defaults
invoke-rc.d ohmd start

Packaging improvements welcome.  I'm yet to learn some tricks.

-- 
James Cameronmailto:qu...@us.netrek.org http://quozl.netrek.org/


signature.asc
Description: Digital signature
___
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel