Bug #10076

Fix the documentation for translating with Git

Added by elouann 2015-08-23 08:37:18 . Updated 2015-09-21 08:05:28 .

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

20%

Feature Branch:
doc/10076-fix-translating-with-git
Type of work:
Contributors documentation
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

- The documented command line `git clone git@gitlab.com:Tails/tails.git tails` returns an error: “Permission denied (publickey)”.
- The chapter “Configure your credentials” is outdated


Subtasks


History

#2 Updated by intrigeri 2015-08-25 00:13:49

  • Status changed from New to In Progress
  • Assignee set to sajolida
  • % Done changed from 0 to 20
  • QA Check set to Ready for QA
  • Feature Branch set to elouann:doc/10076-fix-translating-with-git

#3 Updated by intrigeri 2015-08-25 00:14:37

  • Subject changed from Fix the documentation for translating with git to Fix the documentation for translating with Git

#4 Updated by sajolida 2015-08-27 08:12:02

  • Assignee changed from sajolida to elouann
  • QA Check changed from Ready for QA to Dev Needed

Thanks for working on this. I’m glad to see more people working on documentation lately.

But your branch mixes up many unrelated commits and this time the work seems quite important, so I won’t do fancy Git operations myself and I ask you to clean your Git history first. Here are some tips.

You must have a clear difference of history between origin/master and your branch. You can visualize this with git log -p origin/master...doc/10076-fix-translating-with-git. This must show only the commits that are specific to your branch. Here it’s not the case and I see at least 114ec4f, a078471, b3a98c2, 4d05f2d, etc. which are unrelated.

To fix this, the technique I find the easiest to understand is to:

  • Checkout your branch.
  • Visualize the difference between your branch and origin/master with git log -p as explained earlier.
  • Write down a list of commits that you want to save from this branch. For example, ecc59fe, 9d46194, c686f44, etc.
  • Reset your branch to the state of master: git reset --hard origin/master.
  • Cherry-pick in chronological order the relevant commits that you wrote down. For example git cherry-pick c686f44, then git cherry-pick 9d46194, etc.
  • Make sure that your history is clean with git log -p as explained earlier.
  • Push your branch to your remote repo with the --force option to overwrite it. git push --force elouann doc/10076-fix-translating-with-git.

#5 Updated by intrigeri 2015-08-28 11:28:46

> * Reset your branch to the state of master: git reset --hard origin/master.

You may want to save a copy of your original branch (e.g. by checking it out under another name) before you do that: the next steps will make the unrelated commits hard to find (since they won’t be referrenced by the rewritten branch anymore).

#6 Updated by elouann 2015-09-14 15:55:51

I finally cleaned up my branch. Please review, and thanks a lot for the guidance!


FTR, I used the following command lines:

Make a backup
git checkout -b doc/10076-fix-translating-with-git-BACKUP
git checkout doc/10076-fix-translating-with-git

Save the list of commits
git log origin/master…doc/10076-fix-translating-with-git —author=elouann > git-commits-list

Reset
git reset —hard origin/master

Cheery-pick using the git-commits-list

Check
git log origin/master…doc/10076-fix-translating-with-git

Push
git push —force elouann doc/10076-fix-translating-with-git

#7 Updated by intrigeri 2015-09-15 02:06:49

  • Assignee changed from elouann to sajolida
  • QA Check changed from Dev Needed to Ready for QA

#8 Updated by sajolida 2015-09-21 07:45:04

  • Status changed from In Progress to Resolved
  • % Done changed from 20 to 100

Applied in changeset commit:5bf41ac3a4cebc6f1e3363b08e1e137b05927ac5.

#9 Updated by sajolida 2015-09-21 08:05:28

  • Assignee deleted (sajolida)
  • % Done changed from 100 to 20
  • QA Check deleted (Ready for QA)
  • Feature Branch changed from elouann:doc/10076-fix-translating-with-git to doc/10076-fix-translating-with-git

Super cool! I added a bunch of minor improvements with c7f8aa6..effc9de and merged it.