crashkita / git-repo

Automatically exported from code.google.com/p/git-repo
Apache License 2.0
0 stars 0 forks source link

changing manifest branches fails if previous init was "manifest_branch + manifest_file" #125

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Affected Version:

.repo/repo/repo --version
repo version v1.11.1
       (from https://gerrit.googlesource.com/git-repo)
repo launcher version 1.18
       (from /tmp/toto/.repo/repo/repo)
git version 1.7.11.2
Python 2.7.3 (default, Jul 16 2012, 05:45:18) 
[GCC 4.1.2 20071124 (Red Hat 4.1.2-42)]

Environment:

Imagine two manifest branches:

  branch       files within the branch

 branch_A      default.xml, test_1.xml
 branch_B      default.xml, test_2.xml

What steps will reproduce the problem?
1. repo init -u ${URL} -g all -b branch_A
2. repo init -u ${URL} -g all -b branch_A -m test_1.xml
3. repo init -u ${URL} -g all -b branch_B
fatal: manifest 'default.xml' not available
fatal: cannot link manifest default.xml

What is the expected output? What do you see instead?

Should work.

Please provide any additional information below.

Will provide with a patch to fix the issue

Original issue reported on code.google.com by sf.sigm...@gmail.com on 21 Nov 2012 at 6:15

GoogleCodeExporter commented 9 years ago
patch submitted:

https://gerrit-review.googlesource.com/39722

Original comment by sf.sigm...@gmail.com on 22 Nov 2012 at 9:33

GoogleCodeExporter commented 9 years ago
change is still pending a review+2

Original comment by sf.sigm...@gmail.com on 5 Dec 2012 at 10:56

GoogleCodeExporter commented 9 years ago
change is still pending a review+2

Original comment by sf.sigm...@gmail.com on 2 Jan 2013 at 12:44

GoogleCodeExporter commented 9 years ago
change is still pending a review+2

Original comment by sf.sigm...@gmail.com on 21 Jan 2013 at 2:45

GoogleCodeExporter commented 9 years ago
Issue 129 has been merged into this issue.

Original comment by david.pu...@sonymobile.com on 1 Feb 2013 at 2:06

GoogleCodeExporter commented 9 years ago
Fixed with:

https://gerrit-review.googlesource.com/#/c/39722/

Original comment by david.pu...@sonymobile.com on 1 Feb 2013 at 2:07