Bug #10741
Icedove is started without the wrapper in Jessie
Start date:
2015-12-10
Due date:
% Done:
100%
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