Casper

Merge casper:ubuntu-pro-notification into casper:main

  1. Git
  2. lp:casper
  3. ubuntu-pro-notification
  4. Merge into main
Proposed by Sebastien Bacher
Status: Merged
Approved by: Michael Hudson-Doyle
Approved revision: eff53a5b1ae548d57eb612233371ea3a17e8cebe
Merged at revision: eff53a5b1ae548d57eb612233371ea3a17e8cebe
Proposed branch: casper:ubuntu-pro-notification
Merge into: casper:main
Diff against target: 14 lines (+3/-0)
1 file modified
scripts/casper-bottom/31disable_update_notifier (+3/-0)
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle Approve
Review via email: mp+416098@code.launchpad.net

Description of the change

Currently the "Enabled Ubuntu Pro" notification is displayed on the live session but there is no point attaching that env to Ubuntu Advantage

To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

sorry for the delay in getting to this!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/scripts/casper-bottom/31disable_update_notifier b/scripts/casper-bottom/31disable_update_notifier
2index ab2b47e..51b8e77 100755
3--- a/scripts/casper-bottom/31disable_update_notifier
4+++ b/scripts/casper-bottom/31disable_update_notifier
5@@ -43,6 +43,9 @@ ln -sf /bin/true /root/usr/lib/ubuntu-release-upgrader/check-new-release-gtk
6 #anything it's plugins support
7 rm -f /root/etc/xdg/autostart/gnome-software-service.desktop
8
9+#no need to suggest enabling UA on a live session
10+rm -f /root/etc/xdg/autostart/ubuntu-advantage-notification.desktop
11+
12 GNOME_SOFTWARE_SEARCH_PROVIDER=/root/usr/share/gnome-shell/search-providers/org.gnome.Software-search-provider.ini
13
14 if [ -e "${GNOME_SOFTWARE_SEARCH_PROVIDER}" ]; then

Subscribers

People subscribed via source and target branches