Tails

#1 Updated by BitingBird 2014-03-24 04:45:51

• Category set to 176

#2 Updated by intrigeri 2014-03-24 08:58:29

• Category deleted (176)

Icedove is a mail client, not a browser.

#3 Updated by BitingBird 2014-05-12 11:54:59

• Category set to 176

#4 Updated by BitingBird 2014-05-12 12:15:31

• Category deleted (176)

#5 Updated by sajolida 2014-07-10 20:23:52

• Priority changed from High to Normal

#6 Updated by intrigeri 2014-08-05 08:40:21

• blocked by Feature #7746: Rebase our patches on top of Icedove 38 added

#7 Updated by intrigeri 2014-08-05 08:40:39

• Subject changed from Build Debian packages of Icedove 24 with our patches to Build Debian packages of Icedove 31 with our patches

#8 Updated by intrigeri 2014-08-05 08:42:12

• blocks deleted (Feature #7746: Rebase our patches on top of Icedove 38)

#9 Updated by intrigeri 2014-08-05 08:43:16

• blocked by Feature #7746: Rebase our patches on top of Icedove 38 added

#10 Updated by intrigeri 2014-08-12 13:44:09

• Category set to 212

#11 Updated by intrigeri 2015-05-29 12:31:53

• Subject changed from Build Debian packages of Icedove 31 with our patches to Build Debian packages of Icedove 38 with our patches
• Assignee deleted (None)
• Target version set to 246

#12 Updated by intrigeri 2015-05-29 12:32:07

• blocks #8668 added

#13 Updated by sajolida 2015-11-27 04:45:06

• Target version changed from 246 to Tails_2.0

#14 Updated by Anonymous 2015-12-30 04:39:02

• related to Feature #6158: Fix secure Icedove autoconfig wizard in Tails added

#15 Updated by Anonymous 2015-12-30 04:40:17

• related to Feature #6157: Fix re-test in secure Icedove autoconfig wizard added

#16 Updated by Anonymous 2016-02-04 18:29:59

• Feature Branch set to icedove:tails-secure_account_creation-38.5.0

#17 Updated by Anonymous 2016-02-05 16:48:46

• Priority changed from Normal to Elevated
• Target version changed from Tails_2.0 to Tails_2.2

#18 Updated by Anonymous 2016-02-24 11:21:12

• Status changed from Confirmed to In Progress
• % Done changed from 0 to 10

I’ve successfully built packages. Still need to upload my branch though.

#21 Updated by Anonymous 2016-03-10 14:44:47

• blocked by deleted (Feature #6368: Test our rebased patches on Icedove 38)

#22 Updated by Anonymous 2016-03-10 14:54:19

• Subject changed from Build Debian packages of Icedove 38 with our patches to Build Debian packages of Icedove 38 with our patches / create proper branch situation
• Feature Branch changed from icedove:tails-secure_account_creation-38.5.0 to icedove:tails-secure_account_creation-38.6.0

We actually need to rebase the Debian patches on 2 branches,the unstable (master) branch and jessie-security, both with our patches. These branches have very diverged debian/ files (control & patches).

#23 Updated by Anonymous 2016-03-10 14:56:14

• Target version changed from Tails_2.2 to Tails_2.3

#24 Updated by anonym 2016-05-08 05:10:23

• Target version changed from Tails_2.3 to Tails_2.4

#25 Updated by Anonymous 2016-05-10 04:42:11

• Assignee set to anonym
• % Done changed from 10 to 20

Hi,

I’ve created and pushed to my repo:

* tails/38.7.2-1_secure_account_creation - that’s currently in Debian unstable

Then we have your tag using the patches already, now pushed in this same repo:

• tails/38.6.0-1_deb8u1+tails0_secure_account_creation - that’s based on what is currently in Debian security.

Please check if that works for you.

#26 Updated by anonym 2016-05-14 10:13:06

• Assignee deleted (anonym)
• % Done changed from 20 to 30
• QA Check set to Info Needed

u wrote:
> * tails/38.6.0-1_deb8u1+tails0_secure_account_creation - that’s based on what is currently in Debian security.

I built packages from this branch, and now have 38.6.0-1~deb8u1+tails0 packages. But that is the version of packages already present in the feature-6154-secure-autoconfig-in-icedove APT suite, which are the packages I built two months ago from my tails_secure_account_creation-38.6.0-deb8u1 branch. I dupload:ed with --force, so I’ve now replaced the old packages with the new (despite the identical version), but haven’t tested them yet.

So, what are the next steps for comparing our builds, which you wanted? TBH, I don’t really care about that as I’m confident all is fine. :)

#28 Updated by Anonymous 2016-05-16 07:05:37

• Assignee set to anonym
• QA Check deleted (Info Needed)
• Feature Branch changed from icedove:tails-secure_account_creation-38.6.0 to 451f:tails/38.8.0-1_deb8u1_secure_account_creation

Hi,

I’ve pushed tails/38.8.0-1_deb8u1_secure_account_creation and pristine-tar to my repository. This is the latest security update of Icedove in Debian, it came out yesterday.
I’ve modified a bit the naming of the tails branch, but did not dare to change it on the last version in the main repository.

Trying to build now from my side just to see if all is fine.
Reassigning to you in the meantime.

Cheers!

#29 Updated by anonym 2016-05-20 14:11:41

• Assignee deleted (anonym)
• QA Check set to Dev Needed

I just found a blocker for inclusion in Tails 2.4: it seems that you have imported the wrong patches! Notice how in your patches the new pref is called mailnews.auto_config_ssl_only but I changed the name quite some time ago to mailnews.auto_config.ssl_only (notice the dot instead of underscore!). Also, mailnews.auto_config.dns_mx_lookup.enabled is not present in the patches you imported.

Can you please import the patches from the new branch secure_account_creation-38.8.0-1 that I just pushed? Unfortunately the wrong patches were submitted to the upstream Mozilla ticket, so you need to push new ones there too and explain the situation. Just to be sure I have meticulously and painstakingly compared the two sets of patches to make sure nothing strange is going on and that nothing was lost, so you can make it clear that the only things that changed are:

• mailnews.auto_config_ssl_only => mailnews.auto_config.ssl_only
• The above pref has the default True
• DNS MX lookup is controller by its own pref mailnews.auto_config.dns_mx_lookup.enabled
• The above pref has the default True
• Improved commit messages

When the new packages land in Tails, I have to do this in the feature/6154-secure-autoconfig-in-icedove branch:

• I can drop setting mailnews.auto_config.ssl_only since Icedove now sets it by default.
• But I must set mailnews.auto_config.dns_mx_lookup.enabled to False. We should alert the TorBirdy devs about this!

I cannot believe none of this was noticed by any of us! :)

Lastly, that branch now includes three additional commits that resolves the points made during the review of the upstream ticket. Oh, ans also note that the other upstream ticket is completely unaffected by all this.

#32 Updated by Anonymous 2016-05-20 18:23:28

• Assignee set to anonym
• QA Check changed from Dev Needed to Info Needed

anonym wrote:
> I just found a blocker for inclusion in Tails 2.4: it seems that you have imported the wrong patches! Notice how in your patches the new pref is called mailnews.auto_config_ssl_only but I changed the name quite some time ago to mailnews.auto_config.ssl_only (notice the dot instead of underscore!). Also, mailnews.auto_config.dns_mx_lookup.enabled is not present in the patches you imported.

I’m very happy you saw that and I’m very sorry about this. I must have used an older export or branch for these patches.

> Can you please import the patches from the new branch secure_account_creation-38.8.0-1 that I just pushed? Unfortunately the wrong patches were submitted to the “upstream Mozilla ticket”: https://bugzilla.mozilla.org/show_bug.cgi?id=971347, so you need to push new ones there too and explain the situation. Just to be sure I have meticulously and painstakingly compared the two sets of patches to make sure nothing strange is going on and that nothing was lost, so you can make it clear that the only things that changed are:
> * mailnews.auto_config_ssl_only => mailnews.auto_config.ssl_only
> * The above pref has the default True
> * DNS MX lookup is controller by its own pref mailnews.auto_config.dns_mx_lookup.enabled
> * The above pref has the default True
> * Improved commit messages
>
> When the new packages land in Tails, I have to do this in the feature/6154-secure-autoconfig-in-icedove branch:
> * I can drop setting mailnews.auto_config.ssl_only since Icedove now sets it by default.
> * But I must set mailnews.auto_config.dns_mx_lookup.enabled to False. We should alert the TorBirdy devs about this!

Should I prepare a TorBirdy patch? That might be easier to get this upstreamed.

> I cannot believe none of this was noticed by any of us! :)

Well, it was. By you. Now :)

> Lastly, that branch now includes three additional commits that resolves the points made during the review of the upstream ticket. Oh, ans also note that the other upstream ticket is completely unaffected by all this.

I’ve explained this on https://bugzilla.mozilla.org/show_bug.cgi?id=971347 and uploaded the new patch set.

Previously I had also imported this one: https://bugzilla.mozilla.org/attachment.cgi?id=8724400 but I think we decided that this is unlikely to get merged. And it’s not present in the current patchset anymore. (That’s patch 0007.) Can you please confirm this?

I will add these patches to the packaging branch now.

#33 Updated by Anonymous 2016-05-20 18:37:33

• Feature Branch changed from 451f:tails/38.8.0-1_deb8u1_secure_account_creation to 451f:icedove/tails/38.8.0-1_deb8u1_sac

Pushed a debian branch to 451f:/icedove/tails/38.8.0-1_deb8u1_sac
But I’ve not tested building yet.

#34 Updated by anonym 2016-05-21 00:24:54

• Assignee deleted (anonym)
• QA Check changed from Info Needed to Dev Needed

u wrote:
> anonym wrote:
> > I just found a blocker for inclusion in Tails 2.4: it seems that you have imported the wrong patches! Notice how in your patches the new pref is called mailnews.auto_config_ssl_only but I changed the name quite some time ago to mailnews.auto_config.ssl_only (notice the dot instead of underscore!). Also, mailnews.auto_config.dns_mx_lookup.enabled is not present in the patches you imported.
>
> I’m very happy you saw that and I’m very sorry about this. I must have used an older export or branch for these patches.

Don’t worry about this — things like these happens. :) I’m equally much the cause for failing to communicate clearly which branch was the newest (and for not noticing this for months!).

> > Can you please import the patches from the new branch secure_account_creation-38.8.0-1 that I just pushed? Unfortunately the wrong patches were submitted to the “upstream Mozilla ticket”: https://bugzilla.mozilla.org/show_bug.cgi?id=971347, so you need to push new ones there too and explain the situation. Just to be sure I have meticulously and painstakingly compared the two sets of patches to make sure nothing strange is going on and that nothing was lost, so you can make it clear that the only things that changed are:
> > * mailnews.auto_config_ssl_only => mailnews.auto_config.ssl_only
> > * The above pref has the default True
> > * DNS MX lookup is controller by its own pref mailnews.auto_config.dns_mx_lookup.enabled
> > * The above pref has the default True
> > * Improved commit messages
> >
> > When the new packages land in Tails, I have to do this in the feature/6154-secure-autoconfig-in-icedove branch:
> > * I can drop setting mailnews.auto_config.ssl_only since Icedove now sets it by default.
> > * But I must set mailnews.auto_config.dns_mx_lookup.enabled to False. We should alert the TorBirdy devs about this!
>
> Should I prepare a TorBirdy patch? That might be easier to get this upstreamed.

Don’t worry about this. This last part is not a blocker for you, and the upstreaming can wait until post-2.4.

> > I cannot believe none of this was noticed by any of us! :)
>
> Well, it was. By you. Now :)

Can we please just forget about it! :P

> > Lastly, that branch now includes three additional commits that resolves the points made during the review of the upstream ticket. Oh, ans also note that the other upstream ticket is completely unaffected by all this.
>
> I’ve explained this on https://bugzilla.mozilla.org/show_bug.cgi?id=971347 and uploaded the new patch set.

Great!

> Previously I had also imported this one: https://bugzilla.mozilla.org/attachment.cgi?id=8724400 but I think we decided that this is unlikely to get merged. And it’s not present in the current patchset anymore. (That’s patch 0007.) Can you please confirm this?

Kill it with fire!

> I will add these patches to the packaging branch now.

Excellent, but…

> Pushed a debian branch to 451f:/icedove/tails/38.8.0-1_deb8u1_sac
> But I’ve not tested building yet.

… it doesn’t seem like you did it based on our previous Git branch, so the current release (38.8.0-1~deb8u1+tails0) and other history would be lost from debian/changelog. To not lose history, let’s do this instead: take your previous branch (tails/38.8.0-1_deb8u1_secure_account_creation) rename it to tails/jessie and then refresh the old patches to the new/fixed ones (as new commits! no history rewriting!), and bump the tails* counter for a new release (38.8.0-1~deb8u1+tails1). From now on, let’s base all our release work on this branch.

#36 Updated by Anonymous 2016-05-21 10:11:05

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

Pushing there right now.

#37 Updated by Anonymous 2016-05-21 10:11:16

• Feature Branch changed from 451f:icedove/tails/38.8.0-1_deb8u1_sac to 451f:icedove/tails/jessie

#38 Updated by anonym 2016-05-22 20:46:17

• % Done changed from 30 to 40

I pushed your tails/jessie branch to Tails’ icedove repo, and a fix on top of it: you had forgotten to remove the old patches (with their different names) from debian/patches/series. Debian packagin is fun! :)

Building 38.8.0-1~deb8u1+tails2…

#39 Updated by anonym 2016-05-22 22:53:14

• % Done changed from 40 to 50

anonym wrote:
> Building 38.8.0-1~deb8u1+tails2…

Finished, and uploaded to the feature-6154-secure-autoconfig-in-icedove APT suite.

Some testing would be excellent! Images will be available here: http://nightly.tails.boum.org/build_Tails_ISO_feature-6154-secure-autoconfig-in-icedove/lastSuccessful/archive/build-artifacts/

#41 Updated by anonym 2016-05-24 20:15:16

• Status changed from In Progress to Resolved
• Assignee deleted (anonym)
• % Done changed from 50 to 100
• QA Check changed from Ready for QA to Pass

I have uploaded 38.8.0-1~deb8u1+tails3 which deals with the locale issue, and the needed pref changes are in the feature/6154-secure-autoconfig-in-icedove branch.