Bug #10741

Icedove is started without the wrapper in Jessie

Added by kytv 2015-12-10 05:00:08 . Updated 2016-01-27 13:31:53 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2015-12-10
Due date:
% Done:

100%

Feature Branch:
kytv/bugfix/10741-icedove
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Email Client
Deliverable for:
268

Description


Subtasks


History

#1 Updated by kytv 2015-12-10 05:06:58

  • Assignee changed from kytv to anonym
  • Target version changed from Tails_1.8 to Tails_2.0
  • % Done changed from 0 to 60
  • Feature Branch set to kytv/bugfix/10741-icedove

This branch updates the .desktop file to explicitly start Icedove with the wrapper script from the GNOME menu.

Please review’n’merge for inclusion in Jessie.

#2 Updated by intrigeri 2015-12-13 06:01:01

  • Deliverable for set to 268

#3 Updated by intrigeri 2015-12-19 10:15:18

  • QA Check set to Ready for QA

#4 Updated by intrigeri 2015-12-19 10:26:45

  • Assignee changed from anonym to kytv
  • QA Check changed from Ready for QA to Dev Needed

Looks good to me, except:

  • all our hooks have set -e, please make this one consistent with the others
  • it would be nice to have an echo "What I'm doing" just like our other scripts

I realize the reviewer/tester can do this trivially, and reassigning to kytv is mostly overhead. The only reason I’m doing it is as part of our Great Skill Sharing Plan, so that kytv learns about what’s expected and is given a chance to do better next time :)

#5 Updated by kytv 2015-12-20 10:05:16

  • Assignee changed from kytv to anonym
  • % Done changed from 60 to 70
  • QA Check changed from Dev Needed to Ready for QA

Changes pushed

#6 Updated by anonym 2016-01-03 13:47:16

  • Status changed from In Progress to Fix committed
  • % Done changed from 70 to 100

Applied in changeset commit:d9cdbe47f1050324010e8fb29c6d648ec684be61.

#7 Updated by anonym 2016-01-03 13:56:33

  • Assignee deleted (anonym)
  • QA Check changed from Ready for QA to Pass

We really need Bug #10659.

#8 Updated by anonym 2016-01-27 13:31:53

  • Status changed from Fix committed to Resolved