1) The patch in that link looks like it's modifying postinst, not preinst. Dunno whether that makes a difference for purposes of the discussion; for that matter, I might also be misunderstanding how the patches are applied.
More importantly,
2) In that patch, the removal is only triggered if the hibernation-enabled file is present. On the systems where I've encountered the bug, the *file* never gets created because we're not using hibernation, but the misplaced *directory* is still created. Shouldn't the condition instead be to test for the existence of the *directory*? The current body of the 'if' would still be the correct action.
Mitchell,
1) The patch in that link looks like it's modifying postinst, not preinst. Dunno whether that makes a difference for purposes of the discussion; for that matter, I might also be misunderstanding how the patches are applied.
More importantly,
2) In that patch, the removal is only triggered if the hibernation-enabled file is present. On the systems where I've encountered the bug, the *file* never gets created because we're not using hibernation, but the misplaced *directory* is still created. Shouldn't the condition instead be to test for the existence of the *directory*? The current body of the 'if' would still be the correct action.
looking forward to this hitting the 22.04 builds!