Merge lp:~tsimonq2/ubiquity/remove-encrypt_home-qt into lp:ubiquity

Proposed by Simon Quigley
Status: Merged
Merged at revision: 6618
Proposed branch: lp:~tsimonq2/ubiquity/remove-encrypt_home-qt
Merge into: lp:ubiquity
Diff against target: 102 lines (+16/-24)
4 files modified
debian/changelog (+10/-0)
debian/ubiquity.templates (+0/-4)
gui/qt/stepUserSetup.ui (+0/-11)
ubiquity/plugins/ubi-usersetup.py (+6/-9)
To merge this branch: bzr merge lp:~tsimonq2/ubiquity/remove-encrypt_home-qt
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Ubuntu Installer Team Pending
Review via email: mp+343759@code.launchpad.net

Commit message

Remove encrypt_home from the Qt frontend.

Description of the change

Following the lead of the GTK frontend, remove the encrypted home directory option from the Qt installer since ecryptfs no longer has support from the Ubuntu Security Team. This removes the frontend but keeps the codebase for the option so it can be added back later using another technology.

Kubuntu considers this release-critical.

To post a comment you must log in.
Revision history for this message
Simon Quigley (tsimonq2) wrote :

Apparently someone was kool and filed this already in bug 1761396 but I didn't link that in the changelog.

6618. By Simon Quigley

Add bug report in the changelog.

Revision history for this message
Steve Langasek (vorlon) wrote :

Uploading this; installer team, please merge this (picking up the resolution to the merge conflicts from the archive).

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2018-04-20 21:11:32 +0000
+++ debian/changelog 2018-04-22 04:39:16 +0000
@@ -1,3 +1,13 @@
1ubiquity (18.04.10) bionic; urgency=medium
2
3 * Following the lead of the GTK frontend, remove the encrypted home
4 directory option from the Qt installer since ecryptfs no longer has
5 support from the Ubuntu Security Team. This removes the frontend but keeps
6 the codebase for the option so it can be added back later using another
7 technology (LP: #1761396).
8
9 -- Simon Quigley <tsimonq2@ubuntu.com> Sat, 21 Apr 2018 21:15:49 -0500
10
1ubiquity (18.04.9) bionic; urgency=medium11ubiquity (18.04.9) bionic; urgency=medium
212
3 * Detect the proper name for zram in /proc/swaps (LP: #1763611).13 * Detect the proper name for zram in /proc/swaps (LP: #1763611).
414
=== modified file 'debian/ubiquity.templates'
--- debian/ubiquity.templates 2018-04-13 17:37:47 +0000
+++ debian/ubiquity.templates 2018-04-22 04:39:16 +0000
@@ -221,10 +221,6 @@
221Type: text221Type: text
222_Description: Require my password to log in222_Description: Require my password to log in
223223
224Template: ubiquity/text/login_encrypt
225Type: text
226_Description: Encrypt my home folder
227
228Template: ubiquity/text/part_auto_heading_label224Template: ubiquity/text/part_auto_heading_label
229Type: text225Type: text
230_Description: Installation type226_Description: Installation type
231227
=== modified file 'gui/qt/stepUserSetup.ui'
--- gui/qt/stepUserSetup.ui 2015-09-08 16:16:32 +0000
+++ gui/qt/stepUserSetup.ui 2018-04-22 04:39:16 +0000
@@ -536,16 +536,6 @@
536 </property>536 </property>
537 </widget>537 </widget>
538 </item>538 </item>
539 <item row="1" column="0">
540 <widget class="QCheckBox" name="login_encrypt">
541 <property name="text">
542 <string>Require Password and decrypt home to login</string>
543 </property>
544 <property name="checked">
545 <bool>false</bool>
546 </property>
547 </widget>
548 </item>
549 </layout>539 </layout>
550 </widget>540 </widget>
551 </item>541 </item>
@@ -572,7 +562,6 @@
572 <tabstop>hostname</tabstop>562 <tabstop>hostname</tabstop>
573 <tabstop>login_pass</tabstop>563 <tabstop>login_pass</tabstop>
574 <tabstop>login_auto</tabstop>564 <tabstop>login_auto</tabstop>
575 <tabstop>login_encrypt</tabstop>
576 </tabstops>565 </tabstops>
577 <resources/>566 <resources/>
578 <connections/>567 <connections/>
579568
=== modified file 'ubiquity/plugins/ubi-usersetup.py'
--- ubiquity/plugins/ubi-usersetup.py 2018-02-28 14:58:37 +0000
+++ ubiquity/plugins/ubi-usersetup.py 2018-04-22 04:39:16 +0000
@@ -456,7 +456,6 @@
456 self.page.username.setEnabled(False)456 self.page.username.setEnabled(False)
457 self.page.login_pass.hide()457 self.page.login_pass.hide()
458 self.page.login_auto.hide()458 self.page.login_auto.hide()
459 self.page.login_encrypt.hide()
460 self.username_edited = True459 self.username_edited = True
461 self.hostname_edited = True460 self.hostname_edited = True
462461
@@ -541,22 +540,20 @@
541 return self.page.login_auto.isChecked()540 return self.page.login_auto.isChecked()
542541
543 def on_login_pass_clicked(self, checked):542 def on_login_pass_clicked(self, checked):
544 self.page.login_encrypt.setEnabled(checked)543 print("Ecryptfs is deprecated")
545544
546 def on_login_auto_clicked(self, checked):545 def on_login_auto_clicked(self, checked):
547 self.page.login_encrypt.setChecked(not(checked))546 print("Ecryptfs is deprecated")
548 self.page.login_encrypt.setEnabled(not(checked))
549547
550 def set_encrypt_home(self, value):548 def set_encrypt_home(self, value):
551 self.page.login_encrypt.setChecked(value)549 print("Ecryptfs is deprecated")
552550
553 def set_force_encrypt_home(self, value):551 def set_force_encrypt_home(self, value):
554 self.page.login_encrypt.setDisabled(value)552 print("Ecryptfs is deprecated")
555 self.page.login_auto.setDisabled(value)
556 self.page.login_pass.setDisabled(value)
557553
554 # Ecryptfs is deprecated
558 def get_encrypt_home(self):555 def get_encrypt_home(self):
559 return self.page.login_encrypt.isChecked()556 return False
560557
561 def username_error(self, msg):558 def username_error(self, msg):
562 self.page.username_error_reason.setText(msg)559 self.page.username_error_reason.setText(msg)

Subscribers

People subscribed via source and target branches

to status/vote changes: