Comment 3 for bug 247867

Revision history for this message
Saša Bodiroža (jazzva) wrote :

Hello David,

We're not shipping Iceweasel, so you can safely remove iceweasel dependency. We're also trying not to mention Iceweasel and Icedove in Description fields, in order not to confuse users. Changing Iceweasel to Firefox and Icedove to Thunderbird would be nice.
You don't need to remove a part of description. It's usable, and by removing it we are making diff bigger than needed.
You don't need to create usr/lib/iceweasel/extensions dir in torbutton-extension.dirs file, and link it in torbutton-extension.links file.

Also, it looks like iceweasel-torbutton.* files are still in the debian/ dir. I think they are useless, since that package doesn't exist. I'll ask for another opinion. As far as I can see, you should remove them.

The rest looks good.