Hey David,
thanks for working on this.
Just a quick comment, as I leave for the weekend in 10 mins:
On Fri, Aug 25, 2006 at 12:04:24AM +0200, David Härdeman wrote:
> Index: debian/control
> ===================================================================
> --- debian/control (revision 40192)
> +++ debian/control (working copy)
> @@ -8,17 +8,18 @@
> Package: partman-crypto
> XC-Package-Type: udeb
> Architecture: any
> -Depends: partman-base (>= 87), partman-crypto-dm, partman-crypto-loop,
> cdebconf-newt-entropy (>= 0.3), ${shlibs:Depends}, ${misc:Depends}
> +Priority: standard
> +Depends: partman-base (>= 87), ${shlibs:Depends}, ${misc:Depends}
> Description: Add to partman support for block device encryption
>
> Package: partman-crypto-dm
> XC-Package-Type: udeb
> Architecture: all
> -Depends: partman-crypto, crypto-modules, cryptsetup-udeb
> +Depends: partman-crypto, crypto-modules, cryptsetup-udeb,
> cdebconf-newt-entropy (>= 0.3)
> Description: Add to partman support for dm-crypt encryption
To prevent -dm and -loop from being included automatically, I
think we'll need to change their Priority to optional.
Looks good to me apart from that -- in an early morning hurried
reading. I hope to take a closer look on Sat/Sun, however feel
free to commit already.
cheers,
Max
