[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#384532: partman-crypto: implement on-demand loading



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



Reply to: