Merge lp:~paultag/rinput/sasl-config into lp:rinput/debian

[Chase Douglas]

Added SASL default options for the debian directory. Under the hood it runs:

echo $RET | saslpasswd2 -c -u rinput user

where RET is the queried password.

[Chase Douglas (chasedouglas)]

This is about 90% there. Thanks for laying all the ground work! We need to either prompt for both a username and a password, or make it clear that the password is associated with a given default username. I think I prefer the former, and all in one screen (I'm pretty sure this is possible, but maybe not). If we go with the latter, I'd prefer a default username like "rinput" rather than just "user". I think "user" as the username could get confusing.

One other nitpick: why is the template named rinput/passw? If it can be anything, I would choose rinputd/passwd.

Again, thanks a lot for doing this! Not only are you helping me out with rinputd, you're also helping me figure out launchpad :). BTW, if you happen to be swamped with other work I can add the changes myself and merge it in.

review: Needs Fixing

[Paul Tagliamonte (paultag)]

Haha, I'll take care of getting the rest of this accross the finish line. I just whipped this up in a few spare minutes to get the basic idea there.

I'll get it lined up and set up for ya

I'll rename too, I was worried about klobbering the namespace ( I thought password / passwd was reserved, I was wrong ).

More to come ;)

[lp:~paultag/rinput/sasl-config]

80. By Paul Tagliamonte <tag@Loki> on 2010-03-29

small changes to fix open issues regarding a merge against mainline.

[Paul Tagliamonte (paultag)]

Rev 80 pushed. Uses three prompts, one description, one username prompt and one password prompt.

[lp:~paultag/rinput/sasl-config]

81. By Paul Tagliamonte <tag@Loki> on 2010-03-29

fixing error in policy

[Chase Douglas (chasedouglas)]

This looks good. I still have some changes I'm going to make, but I'm merging this in now.

Thanks Paul!

review: Approve

[Paul Tagliamonte (paultag)]

Sure thing!

Thanks!

=== modified file 'debian/changelog'
--- debian/changelog	2010-02-25 02:22:14 +0000
+++ debian/changelog	2010-03-29 13:35:46 +0000
@@ -1,3 +1,9 @@
+rinputd (1.0.3-2) UNRELEASED; urgency=low
+
+  * Added in the config and templates files. 
+
+ -- Paul Tagliamonte <[e-mail]>  Mon, 29 Mar 2010 01:11:00 -0400
+
 rinputd (1.0.3-1) unstable; urgency=low
 
   * Initial package

=== added file 'debian/config'
--- debian/config	1970-01-01 00:00:00 +0000
+++ debian/config	2010-03-29 13:35:46 +0000
@@ -0,0 +1,16 @@
+#!/bin/bash -e
+
+. /usr/share/debconf/confmodule
+
+db_input high rinput/about    || true
+db_input high rinput/username || true
+db_input high rinput/passwd   || true
+
+db_go
+
+db_get rinput/username
+USER="$RET"
+db_get rinput/passwd
+PASS="$RET"
+
+echo $PASS | saslpasswd2 -c -u rinput $USER

=== added file 'debian/templates'
--- debian/templates	1970-01-01 00:00:00 +0000
+++ debian/templates	2010-03-29 13:35:46 +0000
@@ -0,0 +1,19 @@
+Template: rinput/about
+Type: text
+Description: Configure the rinputd sasl username and password
+ The following two prompts will help you set up a new sasl user
+ account and password. This allows for remote access to this
+ computer. 
+
+Template: rinput/username
+Type: string
+Description: This is the user account name for the sasl rinputd login.
+ This has no bearing on the local login, and will not create
+ a local user account, nor conflict with the local user namespace.
+
+Template: rinput/passwd
+Type: password
+Description: Password for the rinputd sasl user account.
+ This password is not related to shell level login, but will allow
+ control of the keyboard and mouse through the rinputd daemon.
+