Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Nicholas D Steeves
On Fri, Sep 16, 2016 at 08:55:19PM +0300, Michael Tokarev wrote:
> 16.09.2016 20:51, Ben Hutchings wrote:
> >We should install a minimal NTP client by default.  Not ntp, it's far
> >more complex than needed and (partly as a result of that) has a poor
> >security record.
> 
> Systemd comes with systemd-timesyncd these days, JFYI.

Is the addition systemd-timesyncd documented somewhere?  'just
something along the lines of "Simple ntpdate-like time synchronisation
is now provided by systemd".  I read the NEWS when upgrading and
didn't notice this change.

Does the laptop task support non-systemd inits?  If so,
would there be a benefit to a systemd | ntpdate (or alternative)
dependency for the task?


Cheers,
Nicholas

P.S. Is openntpd the defacto standard these days, for servers?


signature.asc
Description: Digital signature


Re: debootstrap InRelease file support

2016-09-16 Thread Julien Cristau
On Fri, Sep 16, 2016 at 14:56:00 -0400, Lennart Sorensen wrote:

> A bit of a hack but I believe posix compliant would be:
> 
> sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" | tr '\n' 
> '\a' | sed 's/\a$//' | tr '\a' '\n' > "$reldest"
> 
> So simply replace all newlines with bells, then delete the last bell
> and then convert the bells back to newlines.  I sure can't imagine the
> Release file contains any bell characters.  Could use escape instead
> if prefered.
> 
> Pretty awful compared to the head -c -1 option, but maybe not that bad.
> 
Thanks, that looks like it should work :)

Cheers,
Julien



Bug#838031: marked as done (task-laptop: Add 'ntp' to list of recommended packages.)

2016-09-16 Thread Debian Bug Tracking System
Your message dated Fri, 16 Sep 2016 20:22:24 +0100
with message-id <1474053744.2621.81.ca...@decadent.org.uk>
and subject line Re: Bug#838031: task-laptop: Add 'ntp' to list of recommended 
packages.
has caused the Debian Bug report #838031,
regarding task-laptop: Add 'ntp' to list of recommended packages.
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
838031: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=838031
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---

Package: task-laptop
Version: 3.31+deb8u1
Severity: wishlist
Tags: patch

Hi,

When I didn't have 'ntp' installed on my laptop, I found that the clock 
drifted significantly from the true time. (It was almost four minutes 
fast when I noticed it last fall!)


To help other Debian users avoid this, I think adding ntp to the list of 
recommended packages would be helpful. I've attached a patch that I 
think does that.


Thanks,

Nathaniel Beaver

-- System Information:
Debian Release: 8.5
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.6.0-0.bpo.1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8) (ignored: 
LC_ALL set to en_US.UTF-8)

Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages task-laptop depends on:
ii  anacron  2.3-23
ii  tasksel  3.31+deb8u1

Versions of packages task-laptop recommends:
ii  acpi1.7-1
ii  avahi-autoipd   0.6.31-5
ii  bluetooth   5.23-2
ii  iw  3.17-1
ii  powertop2.6.1-1
ii  wireless-tools  30~pre9-8
ii  wpasupplicant   2.3-1+deb8u3

task-laptop suggests no packages.

-- no debconf information
>From 5a38aa3aaddabc453d773232c6e32da7b50c5ee1 Mon Sep 17 00:00:00 2001
From: Nathaniel Beaver 
Date: Fri, 16 Sep 2016 11:23:29 -0500
Subject: [PATCH] Add ntp to prevent clock drift on laptops.

---
 debian/control | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/control b/debian/control
index c87cb53..a5f57ab 100644
--- a/debian/control
+++ b/debian/control
@@ -294,6 +294,7 @@ Depends: ${misc:Depends},
 Recommends:
 	avahi-autoipd,
 	bluetooth,
+	ntp,
 	powertop,
 # wireless networking tools
 	iw,
-- 
2.1.4

--- End Message ---
--- Begin Message ---
On Fri, 2016-09-16 at 20:41 +0200, Vincent Blut wrote:
> > On Fri, 16 Sep 2016 20:55:19 +0300 Michael Tokarev  
> wrote:
>  > 16.09.2016 20:51, Ben Hutchings wrote:
>  > > We should install a minimal NTP client by default.  Not ntp, it's 
> far
>  > > more complex than needed and (partly as a result of that) has a 
> poor
>  > > security record.
>  >
>  > Systemd comes with systemd-timesyncd these days, JFYI.
> 
> Indeed, it is enable by default since systemd 219-1.

Then no change is needed.

Ben.

-- 
Ben Hutchings
All extremists should be taken out and shot.


signature.asc
Description: This is a digitally signed message part
--- End Message ---


Re: debootstrap InRelease file support

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 02:40:06PM -0400, Lennart Sorensen wrote:
> On Fri, Sep 16, 2016 at 02:17:24PM -0400, Lennart Sorensen wrote:
> > On Fri, Sep 16, 2016 at 02:06:51PM -0400, Lennart Sorensen wrote:
> > > On Fri, Sep 16, 2016 at 08:02:10PM +0200, Julien Cristau wrote:
> > > > On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:
> > > > 
> > > > > On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > > > > > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > > > > > 
> > > > > > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > > > > > 
> > > > > > > > If you restore support for `InRelease` and want to use `gpgv`, 
> > > > > > > > please
> > > > > > > > split `InRelease` into two files, i.e. `Release` and 
> > > > > > > > `Release.gpg`, and
> > > > > > > > verify that the signature actually covers all of `Release`.
> > > > > > > > 
> > > > > > > Here's an attempt at doing that.  Only lightly tested.
> > > > > > > 
> > > > > > Ansgar pointed out on IRC that so far nothing in debootstrap 
> > > > > > requires
> > > > > > awk on the host.  I haven't found a way to kill the last newline 
> > > > > > with
> > > > > > sed in a quick attempt, and I don't know how big of a deal 
> > > > > > requiring awk
> > > > > > would be, so help welcome.
> > > > > 
> > > > > How about instead of the awk bit using:
> > > > > 
> > > > > sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > 
> > > > > "$reldest"
> > > > > 
> > > > > At least that works for the InRelease in debian sid since it has a 
> > > > > blank
> > > > > line at the end of the PGP header before the Release file data.
> > > > > 
> > > > My problem is getting something that I can feed to gpgv to verify the
> > > > signature, I don't think your command provides that.
> > > 
> > > Well it makes a Release file that is totally bit for bit identical to
> > > the Release file that goes with Release.gpg
> > > 
> > > diff verified that.
> > > 
> > > So if gpgv wants something different than the original Release file,
> > > then that's weird.
> > 
> > Wow, it does want the last newline removed.  That's just stupid.
> 
> If assuming gnu head or busybox head is available (head is certainly
> assumed to exist), then one could do:
> 
> sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" | head -c 
> -1 > "$reldest"
> 
> But it is not pure posix compliant.  But it would work with both gnu
> coreutils and busybox versions of head.

A bit of a hack but I believe posix compliant would be:

sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" | tr '\n' 
'\a' | sed 's/\a$//' | tr '\a' '\n' > "$reldest"

So simply replace all newlines with bells, then delete the last bell
and then convert the bells back to newlines.  I sure can't imagine the
Release file contains any bell characters.  Could use escape instead
if prefered.

Pretty awful compared to the head -c -1 option, but maybe not that bad.

-- 
Len Sorensen



Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Vincent Blut
On Fri, 16 Sep 2016 20:55:19 +0300 Michael Tokarev  
wrote:

> 16.09.2016 20:51, Ben Hutchings wrote:
> > We should install a minimal NTP client by default.  Not ntp, it's 
far
> > more complex than needed and (partly as a result of that) has a 
poor

> > security record.
>
> Systemd comes with systemd-timesyncd these days, JFYI.

Indeed, it is enable by default since systemd 219-1.

> Thanks,
>
> /mjt
> > Ben.

Cheers,
Vincent



Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 08:55:19PM +0300, Michael Tokarev wrote:
> Systemd comes with systemd-timesyncd these days, JFYI.

Well that's sntp not ntp, so not really the same thing.  I guess for a
laptop you might find it good enough.

-- 
Len Sorensen



Re: debootstrap InRelease file support

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 02:17:24PM -0400, Lennart Sorensen wrote:
> On Fri, Sep 16, 2016 at 02:06:51PM -0400, Lennart Sorensen wrote:
> > On Fri, Sep 16, 2016 at 08:02:10PM +0200, Julien Cristau wrote:
> > > On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:
> > > 
> > > > On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > > > > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > > > > 
> > > > > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > > > > 
> > > > > > > If you restore support for `InRelease` and want to use `gpgv`, 
> > > > > > > please
> > > > > > > split `InRelease` into two files, i.e. `Release` and 
> > > > > > > `Release.gpg`, and
> > > > > > > verify that the signature actually covers all of `Release`.
> > > > > > > 
> > > > > > Here's an attempt at doing that.  Only lightly tested.
> > > > > > 
> > > > > Ansgar pointed out on IRC that so far nothing in debootstrap requires
> > > > > awk on the host.  I haven't found a way to kill the last newline with
> > > > > sed in a quick attempt, and I don't know how big of a deal requiring 
> > > > > awk
> > > > > would be, so help welcome.
> > > > 
> > > > How about instead of the awk bit using:
> > > > 
> > > > sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > 
> > > > "$reldest"
> > > > 
> > > > At least that works for the InRelease in debian sid since it has a blank
> > > > line at the end of the PGP header before the Release file data.
> > > > 
> > > My problem is getting something that I can feed to gpgv to verify the
> > > signature, I don't think your command provides that.
> > 
> > Well it makes a Release file that is totally bit for bit identical to
> > the Release file that goes with Release.gpg
> > 
> > diff verified that.
> > 
> > So if gpgv wants something different than the original Release file,
> > then that's weird.
> 
> Wow, it does want the last newline removed.  That's just stupid.

If assuming gnu head or busybox head is available (head is certainly
assumed to exist), then one could do:

sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" | head -c -1 
> "$reldest"

But it is not pure posix compliant.  But it would work with both gnu
coreutils and busybox versions of head.

-- 
Len Sorensen



Re: debootstrap InRelease file support

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 02:06:51PM -0400, Lennart Sorensen wrote:
> On Fri, Sep 16, 2016 at 08:02:10PM +0200, Julien Cristau wrote:
> > On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:
> > 
> > > On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > > > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > > > 
> > > > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > > > 
> > > > > > If you restore support for `InRelease` and want to use `gpgv`, 
> > > > > > please
> > > > > > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, 
> > > > > > and
> > > > > > verify that the signature actually covers all of `Release`.
> > > > > > 
> > > > > Here's an attempt at doing that.  Only lightly tested.
> > > > > 
> > > > Ansgar pointed out on IRC that so far nothing in debootstrap requires
> > > > awk on the host.  I haven't found a way to kill the last newline with
> > > > sed in a quick attempt, and I don't know how big of a deal requiring awk
> > > > would be, so help welcome.
> > > 
> > > How about instead of the awk bit using:
> > > 
> > > sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > 
> > > "$reldest"
> > > 
> > > At least that works for the InRelease in debian sid since it has a blank
> > > line at the end of the PGP header before the Release file data.
> > > 
> > My problem is getting something that I can feed to gpgv to verify the
> > signature, I don't think your command provides that.
> 
> Well it makes a Release file that is totally bit for bit identical to
> the Release file that goes with Release.gpg
> 
> diff verified that.
> 
> So if gpgv wants something different than the original Release file,
> then that's weird.

Wow, it does want the last newline removed.  That's just stupid.

-- 
Len Sorensen



Re: debootstrap InRelease file support

2016-09-16 Thread Julien Cristau
On Fri, Sep 16, 2016 at 14:06:51 -0400, Lennart Sorensen wrote:

> On Fri, Sep 16, 2016 at 08:02:10PM +0200, Julien Cristau wrote:
> > On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:
> > 
> > > On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > > > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > > > 
> > > > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > > > 
> > > > > > If you restore support for `InRelease` and want to use `gpgv`, 
> > > > > > please
> > > > > > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, 
> > > > > > and
> > > > > > verify that the signature actually covers all of `Release`.
> > > > > > 
> > > > > Here's an attempt at doing that.  Only lightly tested.
> > > > > 
> > > > Ansgar pointed out on IRC that so far nothing in debootstrap requires
> > > > awk on the host.  I haven't found a way to kill the last newline with
> > > > sed in a quick attempt, and I don't know how big of a deal requiring awk
> > > > would be, so help welcome.
> > > 
> > > How about instead of the awk bit using:
> > > 
> > > sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > 
> > > "$reldest"
> > > 
> > > At least that works for the InRelease in debian sid since it has a blank
> > > line at the end of the PGP header before the Release file data.
> > > 
> > My problem is getting something that I can feed to gpgv to verify the
> > signature, I don't think your command provides that.
> 
> Well it makes a Release file that is totally bit for bit identical to
> the Release file that goes with Release.gpg
> 
> diff verified that.
> 
> So if gpgv wants something different than the original Release file,
> then that's weird.
> 
As far as I remember, it does.  The final newline needs to be stripped.

Cheers,
Julien



Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Michael Tokarev

16.09.2016 20:51, Ben Hutchings wrote:

We should install a minimal NTP client by default.  Not ntp, it's far
more complex than needed and (partly as a result of that) has a poor
security record.


Systemd comes with systemd-timesyncd these days, JFYI.

Thanks,

/mjt

Ben.





Re: debootstrap InRelease file support

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 08:02:10PM +0200, Julien Cristau wrote:
> On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:
> 
> > On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > > 
> > > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > > 
> > > > > If you restore support for `InRelease` and want to use `gpgv`, please
> > > > > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, 
> > > > > and
> > > > > verify that the signature actually covers all of `Release`.
> > > > > 
> > > > Here's an attempt at doing that.  Only lightly tested.
> > > > 
> > > Ansgar pointed out on IRC that so far nothing in debootstrap requires
> > > awk on the host.  I haven't found a way to kill the last newline with
> > > sed in a quick attempt, and I don't know how big of a deal requiring awk
> > > would be, so help welcome.
> > 
> > How about instead of the awk bit using:
> > 
> > sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > 
> > "$reldest"
> > 
> > At least that works for the InRelease in debian sid since it has a blank
> > line at the end of the PGP header before the Release file data.
> > 
> My problem is getting something that I can feed to gpgv to verify the
> signature, I don't think your command provides that.

Well it makes a Release file that is totally bit for bit identical to
the Release file that goes with Release.gpg

diff verified that.

So if gpgv wants something different than the original Release file,
then that's weird.

-- 
Len Sorensen



Re: debootstrap InRelease file support

2016-09-16 Thread Julien Cristau
On Fri, Sep 16, 2016 at 13:55:53 -0400, Lennart Sorensen wrote:

> On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> > On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> > 
> > > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > > 
> > > > If you restore support for `InRelease` and want to use `gpgv`, please
> > > > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, and
> > > > verify that the signature actually covers all of `Release`.
> > > > 
> > > Here's an attempt at doing that.  Only lightly tested.
> > > 
> > Ansgar pointed out on IRC that so far nothing in debootstrap requires
> > awk on the host.  I haven't found a way to kill the last newline with
> > sed in a quick attempt, and I don't know how big of a deal requiring awk
> > would be, so help welcome.
> 
> How about instead of the awk bit using:
> 
> sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > "$reldest"
> 
> At least that works for the InRelease in debian sid since it has a blank
> line at the end of the PGP header before the Release file data.
> 
My problem is getting something that I can feed to gpgv to verify the
signature, I don't think your command provides that.

Cheers,
Julien



Re: debootstrap InRelease file support

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 06:59:44PM +0200, Julien Cristau wrote:
> On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:
> 
> > On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> > 
> > > If you restore support for `InRelease` and want to use `gpgv`, please
> > > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, and
> > > verify that the signature actually covers all of `Release`.
> > > 
> > Here's an attempt at doing that.  Only lightly tested.
> > 
> Ansgar pointed out on IRC that so far nothing in debootstrap requires
> awk on the host.  I haven't found a way to kill the last newline with
> sed in a quick attempt, and I don't know how big of a deal requiring awk
> would be, so help welcome.

How about instead of the awk bit using:

sed '1,/^$/d;/^-BEGIN PGP SIGNATURE-$/,$d' < "$inreldest" > "$reldest"

At least that works for the InRelease in debian sid since it has a blank
line at the end of the PGP header before the Release file data.

-- 
Len Sorensen



Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Ben Hutchings
On Fri, 2016-09-16 at 11:27 -0500, Nathaniel Beaver wrote:
> Package: task-laptop
> Version: 3.31+deb8u1
> Severity: wishlist
> Tags: patch
> 
> Hi,
> 
> When I didn't have 'ntp' installed on my laptop, I found that the clock 
> drifted significantly from the true time. (It was almost four minutes 
> fast when I noticed it last fall!)
> 
> To help other Debian users avoid this, I think adding ntp to the list of 
> recommended packages would be helpful. I've attached a patch that I 
> think does that.

We should install a minimal NTP client by default.  Not ntp, it's far
more complex than needed and (partly as a result of that) has a poor
security record.

Ben.

-- 
Ben Hutchings
All extremists should be taken out and shot.


signature.asc
Description: This is a digitally signed message part


Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Lennart Sorensen
On Fri, Sep 16, 2016 at 11:27:21AM -0500, Nathaniel Beaver wrote:
> Package: task-laptop
> Version: 3.31+deb8u1
> Severity: wishlist
> Tags: patch
> 
> Hi,
> 
> When I didn't have 'ntp' installed on my laptop, I found that the clock
> drifted significantly from the true time. (It was almost four minutes fast
> when I noticed it last fall!)
> 
> To help other Debian users avoid this, I think adding ntp to the list of
> recommended packages would be helpful. I've attached a patch that I think
> does that.

That's hardly laptop specific.  ntp makes just as much sense on desktops
and servers.  Clearly the laptop task would not be the right place.

-- 
Len Sorensen



Re: debootstrap InRelease file support

2016-09-16 Thread Julien Cristau
On Fri, Sep  2, 2016 at 20:35:12 +0200, Julien Cristau wrote:

> On Mon, Aug 15, 2016 at 12:12:02 +0200, Ansgar Burchardt wrote:
> 
> > If you restore support for `InRelease` and want to use `gpgv`, please
> > split `InRelease` into two files, i.e. `Release` and `Release.gpg`, and
> > verify that the signature actually covers all of `Release`.
> > 
> Here's an attempt at doing that.  Only lightly tested.
> 
Ansgar pointed out on IRC that so far nothing in debootstrap requires
awk on the host.  I haven't found a way to kill the last newline with
sed in a quick attempt, and I don't know how big of a deal requiring awk
would be, so help welcome.

Cheers,
Julien



Bug#838031: task-laptop: Add 'ntp' to list of recommended packages.

2016-09-16 Thread Nathaniel Beaver

Package: task-laptop
Version: 3.31+deb8u1
Severity: wishlist
Tags: patch

Hi,

When I didn't have 'ntp' installed on my laptop, I found that the clock 
drifted significantly from the true time. (It was almost four minutes 
fast when I noticed it last fall!)


To help other Debian users avoid this, I think adding ntp to the list of 
recommended packages would be helpful. I've attached a patch that I 
think does that.


Thanks,

Nathaniel Beaver

-- System Information:
Debian Release: 8.5
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.6.0-0.bpo.1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8) (ignored: 
LC_ALL set to en_US.UTF-8)

Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages task-laptop depends on:
ii  anacron  2.3-23
ii  tasksel  3.31+deb8u1

Versions of packages task-laptop recommends:
ii  acpi1.7-1
ii  avahi-autoipd   0.6.31-5
ii  bluetooth   5.23-2
ii  iw  3.17-1
ii  powertop2.6.1-1
ii  wireless-tools  30~pre9-8
ii  wpasupplicant   2.3-1+deb8u3

task-laptop suggests no packages.

-- no debconf information
>From 5a38aa3aaddabc453d773232c6e32da7b50c5ee1 Mon Sep 17 00:00:00 2001
From: Nathaniel Beaver 
Date: Fri, 16 Sep 2016 11:23:29 -0500
Subject: [PATCH] Add ntp to prevent clock drift on laptops.

---
 debian/control | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/control b/debian/control
index c87cb53..a5f57ab 100644
--- a/debian/control
+++ b/debian/control
@@ -294,6 +294,7 @@ Depends: ${misc:Depends},
 Recommends:
 	avahi-autoipd,
 	bluetooth,
+	ntp,
 	powertop,
 # wireless networking tools
 	iw,
-- 
2.1.4



Re: All D-I's debconf templates in a single HTML page.

2016-09-16 Thread Charles Plessy
Le Tue, Sep 06, 2016 at 09:18:58PM +0900, Charles Plessy a écrit :
> 
> while working on preseeding D-I to make Cloud images, I found it a bit
> difficult to figure out the details of this and that debconf template that is
> listed in the preseed.txt example of the installation guide.  Therefore, I
> collated them in a single HTML page.

Thank you everybody for your positive feedback.

About the location of the page: Brian suggested a wiki, Philip and Cyril
suggested the installer manual, and Cyril reminded of d-i.debian.org.

Actually, before uploading on people.debian.org, I almost went for the wiki, in
order to have a cheap revision tracking, but I was a bit short of time and it
was easier for me to ouptut HTML than MoinMoin wiki syntax.  One argument
against the wiki is that this would suggest that the templates can be improved
by collaborative editing.  In general cases it is great, but here, the people
writing the templates need a good awareness of the constraints of screen
resolution, translation, and style that are very specific to D-I.

I think that outputing DocBook XML should be straightforward, and I like
Cyril's idea to have a curated version in the manual (I guess this means to
cherry-pick the most relevant templates).  Actually, I also considered
integration in the manual before calling for comments, but I could not figure
out how to integrate the translations, as the debconf PO files probably can not
be used directly on the XML-converted text.  Suggestions are welcome.

For the full version, d-i.debian.org is definitely a good idea.  Do I need to
ask to be added to the d-i group ?

Here are point-to-point answers that mostly serve as a kind of to-do list.

Le Tue, Sep 06, 2016 at 07:18:59PM +0100, Brian Potkin a écrit :
> 
> Choices, Choices-C and Default isn't always filled in.

Indeed in some cases, they are empty in the Debconf templates and in that case
I will ignore them instead of leaving them blank.  In at least one case, the
contents were not parsed, for a reason that I did not have time to
troubleshoot.

Le Wed, Sep 07, 2016 at 05:15:50PM +0200, Philip Hands a écrit :
> 
> I guess that adding links to the source of the package that contains
> each template would allow people to explore what difference setting a
> particular thing really makes (by looking at the scripts that use the
> value), which might make this more useful still.

This is a good idea, I will try to do it.

Le Wed, Sep 07, 2016 at 04:42:32PM +0100, Ian Campbell a écrit :
> 
> Might be worth filtering out the non-question stuff, i.e. "text" and
> "error" types which AFAIK are the contents of various dialog boxes and
> not questions.

I will try to produce filtered and unfiltered versions.

Le Thu, Sep 08, 2016 at 12:37:15AM +0200, Cyril Brulebois a écrit :
> 
> There are also '.templates*' files.

Good point, thanks.


Given the lack of speed of my reply to your suggestions, I hope that you will
not be surprised if my further progresses are not lightning-fast.  Of course, 
anybody is welcome to make changes to the script, and if there is interest, I
can put it on collab-maint or any other place that would make sense.

Have a nice day,

Charles

-- 
Charles Plessy
Tsurumi, Kanagawa, Japan