Trying to take a look but doesn't seem to apply cleanly to master for
me -- I may be driving git wrong though.
Not familiar with dm-crypt but looking through the patch, it seems plausible.
Possibly need to add "e" to the parameter list to getopts?
On Mon, 23 Sep 2019 at 21:25, Drew DeVault <sir@cmpwn.com> wrote:
>> Bump. Cc'ing recent committers to this repo.
--
Richard Mortier
mort@cantab.net
On Wed Sep 25, 2019 at 2:55 PM Richard Mortier wrote:
> Trying to take a look but doesn't seem to apply cleanly to master for> me -- I may be driving git wrong though.
Cheers, I'll rebase it.
> Not familiar with dm-crypt but looking through the patch, it seems plausible.
Note, I've used this patch in production on a half a dozen machines
without any issues so far.
> Possibly need to add "e" to the parameter list to getopts?
Ah, you're right.
Huh, notwithstanding git appearing to give me errors, at least one of
the combinations of apply/am/mailinfo that I tried seemed to have in
fact worked. Sigh. One day I'll learn how to use git properly... :)
One other query - I don't understand the #TODO message...?
Don't have a machine handy to try it on but like I said, seems ok to
me, and if you've been using it for a while then SGTM.
Thanks!
On Wed, 25 Sep 2019 at 14:58, Drew DeVault <sir@cmpwn.com> wrote:
>> Actually, it seems to apply cleanly to me. Try this command:>> curl -s https://lists.alpinelinux.org/~alpine/devel/%3C20190424160230.26544-1-sir%40cmpwn.com%3E/raw | git am -3
--
Richard Mortier
mort@cantab.net
On Wed Sep 25, 2019 at 3:11 PM Richard Mortier wrote:
> One other query - I don't understand the #TODO message...?
The if statement it's TODO'ing skips prompting the user for encryption
if they're using LVM or a non-sys disk setup. The task that remains to
be done is tweaking the crypt support to support setting up encrypted
disks for any case other than the basic one.
Ah ok; FWIW I didn't interpret it like that.
I think the only other query I'd have is whether that interactive
block can be entered even if the user has specified -e -- if so, seems
odd to ask when they've already told you.
On Wed, 25 Sep 2019 at 15:13, Drew DeVault <sir@cmpwn.com> wrote:
>> On Wed Sep 25, 2019 at 3:11 PM Richard Mortier wrote:> > One other query - I don't understand the #TODO message...?>> The if statement it's TODO'ing skips prompting the user for encryption> if they're using LVM or a non-sys disk setup. The task that remains to> be done is tweaking the crypt support to support setting up encrypted> disks for any case other than the basic one.
--
Richard Mortier
mort@cantab.net