Feature #10384
Tails Installer: Merge master into feature/jessie
100%
Description
In order to be able to rename liveusb-creator to tails-installer, we first need to merge, for a last time, master into feature/jessie.
Subtasks
History
#1 Updated by Anonymous 2015-10-17 09:34:48
- % Done changed from 0 to 30
- Parent task set to
Feature #8552
#2 Updated by Anonymous 2015-10-17 09:38:41
- Assignee set to intrigeri
- QA Check set to Ready for QA
- Feature Branch set to 451f:liveusbcreator/mergepo
Please review:
on branch feature/jessie, commits
commit bc138f1fb5a62495eac7da586ec59e16e4da44ee
commit 54ad97155b8906b5482b9cde1834e9cad5bf900e
commit f4cce772907a543ea2594a6a317bbd857121dcac
commit 9cc2294cb8743750484c5056532a47b29ebe5811
commit 57452e20f34877f048c08f3dc15a1125299cdb3c
and on branch mergepo (this branch also has the previous commits):
commit 781e4c53206f565468974dadbb3721e6d001f6f9
commit 01a1763689e528cfe0d27ad4d2b829dbeacf4038
The po files do contain some more lines than the POT file. But i thought we might as well leave those for now, until we regenerate the PO files. checked with fgrep msgid po/*.po -c
#3 Updated by intrigeri 2015-10-20 02:48:59
- blocks #8538 added
#4 Updated by intrigeri 2015-10-20 02:49:28
- Category set to Installation
- Status changed from Confirmed to In Progress
- Priority changed from Normal to Elevated
- Target version set to Tails_1.7
#5 Updated by intrigeri 2015-10-23 11:24:32
- Assignee deleted (
intrigeri) - % Done changed from 30 to 50
- QA Check changed from Ready for QA to Dev Needed
The diff shows at least 2 self.textEdit.append(message)
that weren’t converted to how we do it with GTK (git log -p append_to_log
would help). Otherwise, looks good!
But the two branches’ content is identical so I’m wondering if I’m looking at the right thing.
I’ll assume that you’ve done basic manual tests, and will postpone uploading an updated package to after the rename, so I won’t run the test suite.
#6 Updated by Anonymous 2015-10-26 08:22:51
intrigeri wrote:
> The diff shows at least 2 self.textEdit.append(message)
that weren’t converted to how we do it with GTK (git log -p append_to_log
would help). Otherwise, looks good!
thanks for reviewing this. I will modify those 2.
> But the two branches’ content is identical so I’m wondering if I’m looking at the right thing.
that was a mistake, I’ve re-separated them, so that feature/jessie has code modifications and the mergepo branch has only a po-related commit.
> I’ll assume that you’ve done basic manual tests, and will postpone uploading an updated package to after the rename, so I won’t run the test suite.
ack.
#7 Updated by Anonymous 2015-10-26 08:58:26
- Assignee set to intrigeri
- QA Check changed from Dev Needed to Ready for QA
- Feature Branch changed from 451f:liveusbcreator/mergepo to 451f:liveusbcreator/feature/jessie
Would you care to reverify my latest commit on feature/jessie please?
I’ve simply replaced
self.textEdit.append(message)
by
self.append_to_log(message)
I am not 100% sure if that is what we wanted.
the 2 branches to be merged are:
451f:liveusbcreator/feature/jessie
451f:liveusbcreator/mergepo
#8 Updated by intrigeri 2015-10-26 12:39:59
- Assignee deleted (
intrigeri) - QA Check changed from Ready for QA to Dev Needed
> Would you care to reverify my latest commit on feature/jessie please?
I’ll be glad to (with some delay, likely), once you’ve tested it, because:
> I am not 100% sure if that is what we wanted.
If you test the exact situation in which this code is triggered, you should see if it works fine, I think.
(If I’ve missed something, then I’m very sorry, please explain me what it is.)
#9 Updated by Anonymous 2015-10-27 07:29:33
- Assignee set to intrigeri
- QA Check changed from Dev Needed to Ready for QA
ok, now it’s ready for review i think.
I’ve manually added intrigeri’s commit from master which deletes the obsolete md5sum verification.
#10 Updated by intrigeri 2015-10-31 07:28:34
- Status changed from In Progress to Resolved
- Assignee deleted (
intrigeri) - % Done changed from 50 to 100
- QA Check changed from Ready for QA to Pass
Merged, thanks! As said earlier, I’m postponing testing to once we’re done with Feature #8552.