Hi,
>From a git-p4 point of view, I see no problems with this change.
Thanks and regards,
Vitor
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On April 22, 2015 9:47:42 PM GMT+01:00, Luke Diamand l...@diamand.org wrote:
On 22/04/15 18:11, Junio C Hamano wrote:
Vitor Antunes vitor@gmail.com writes:
The updates introduced in the third revision of these two patches
consist only
on updates to the commit messages to better clarify
On April 20, 2015 11:45:15 PM GMT+01:00, Junio C Hamano gits...@pobox.com
wrote:
Vitor Antunes vitor@gmail.com writes:
On April 20, 2015 6:43:49 AM GMT+01:00, Junio C Hamano
gits...@pobox.com wrote:
Vitor Antunes vitor@gmail.com writes:
Add failing scenario where branch detection
/branch1/...
The test case also includes an extra sub-file mapping to enforce
robustness check of git-p4's client view support:
//depot/branch1/base/dir/sub_file1 //client/branch1/sub_file1
Signed-off-by: Vitor Antunes vitor@gmail.com
Signed-off-by: Junio C Hamano gits...@pobox.com
---
t
The updates introduced in the third revision of these two patches consist only
on updates to the commit messages to better clarify what they implement.
Vitor Antunes (2):
t9801: check git-p4's branch detection with client spec enabled
git-p4: improve client path detection when branches
in /
Signed-off-by: Vitor Antunes vitor@gmail.com
Signed-off-by: Junio C Hamano gits...@pobox.com
---
git-p4.py| 13 +
t/t9801-git-p4-branch.sh |2 +-
2 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/git-p4.py b/git-p4.py
index 549022e..34e4fdd
On April 20, 2015 6:43:49 AM GMT+01:00, Junio C Hamano gits...@pobox.com
wrote:
Vitor Antunes vitor@gmail.com writes:
Add failing scenario where branch detection is enabled together with
use client view. In this specific scenario git-p4 will break when the
perforce client view removes
Junio C Hamano gits...@pobox.com wrote on Sat, 18 Apr 2015 17:58:11 -0700
Vitor Antunes vitor@gmail.com writes:
This patch makes the client path detection more robust by limiting the valid
results from p4 where. The test case is also made more complex, to guarantee
that such client views
implementation.
Signed-off-by: Vitor Antunes vitor@gmail.com
Signed-off-by: Junio C Hamano gits...@pobox.com
---
t/t9801-git-p4-branch.sh | 106 ++
1 file changed, 106 insertions(+)
diff --git a/t/t9801-git-p4-branch.sh b/t/t9801-git-p4-branch.sh
index 2bf142d
repository.
2. Search for mappings that contain the depot path, instead
of requiring an exact match. To guarantee robustness,
paths only match if ending in /
Signed-off-by: Vitor Antunes vitor@gmail.com
Signed-off-by: Junio C Hamano gits...@pobox.com
---
git-p4.py| 13
with /... to guarantee that the correct client side path is
identified.
Vitor Antunes (2):
t9801: check git-p4's branch detection and client view together
git-p4: improve client path detection when branches are used
git-p4.py| 13 --
t/t9801-git-p4-branch.sh | 106
Hi Junio,
Junio C Hamano gits...@pobox.com wrote on Sun, 12 Apr 2015 20:40:58 -0700
Vitor Antunes vitor@gmail.com writes:
Luke Diamand l...@diamand.org wrote on Sun, 05 Apr 2015 20:27:11 +0100
Vitor, one thing I wondered about with this part of the change:
-if entry[depotFile
This patch makes the client path detection more robust by limiting the valid
results from p4 where. The test case is also made more complex, to guarantee
that such client views are supported.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
git-p4.py|4 +++-
t/t9801-git
.
* Remove all case statements and replace them with simple tests to check
that source is file2.
Signed-off-by: Vitor Antunes vitor@gmail.com
Acked-by: Luke Diamand l...@diamand.org
---
t/t9814-git-p4-rename.sh | 46 +++---
1 file changed, 31 insertions
Luke Diamand l...@diamand.org wrote on Sun, 05 Apr 2015 20:27:11 +0100
On 28/03/15 12:28, Vitor Antunes wrote:
I'm adding a test case for a scenario I was confronted with when using
branch
detection and a client view specification. It is possible that the
implemented
fix may not cover
* Modify source file (file2) before copying the file.
* Check that only file2 is the source in the output of p4 filelog.
* Remove all case statements and replace them with simple tests to check
that source is file2.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh
* Modify source file (file2) before copying the file.
* Check that only file2 is the source in the output of p4 filelog.
* Remove all case statements and replace them simple tests to check that
source is file2.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh
Add failing scenario where using branch detection and a client view will break
git p4 submit functionality.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9801-git-p4-branch.sh | 98 ++
1 file changed, 98 insertions(+)
diff --git a/t/t9801
I'm adding a test case for a scenario I was confronted with when using branch
detection and a client view specification. It is possible that the implemented
fix may not cover all possible scenarios, but there is no regression in the
available tests.
Vitor Antunes (2):
git-p4: Check branch
Junio C Hamano gits...@pobox.com wrote:
Vitor Antunes vitor@gmail.com writes:
File file11 is copied from file2 and diff-tree correctly reports this file as
its the source, but the test expression was checking for file10 instead
(which
was a file that also originated from file2
of file11, as
was done in other tests in this file.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh |5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/t/t9814-git-p4-rename.sh b/t/t9814-git-p4-rename.sh
index efae143..8b9c295 100755
--- a/t/t9814-git
makes much sense to me.
Vitor Antunes (2):
git-p4: Make rename test case runnable under dash
git-p4: Fix copy detection test
t/t9814-git-p4-rename.sh |9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
--
1.7.10.4
--
To unsubscribe from this list: send the line unsubscribe git
Vitor Antunes vitor@gmail.com wrote:
Junio C Hamano gits...@pobox.com wrote:
Pete, these tests blame to your 9b6513ac (git p4 test: split up big
t9800 test, 2012-06-27). I presume that you tested the result of
this splitting, but do you happen to know if we did something to
cause the test
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/t/t9814-git-p4-rename.sh b/t/t9814-git-p4-rename.sh
index 95f4421..efae143 100755
--- a/t/t9814-git-p4-rename.sh
+++ b/t/t9814-git-p4-rename.sh
in this mismatch in
behavior.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/t/t9814-git-p4-rename.sh b/t/t9814-git-p4-rename.sh
index 24008ff..018f01d 100755
--- a/t/t9814-git-p4-rename.sh
+++ b/t
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9814-git-p4-rename.sh |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/t/t9814-git-p4-rename.sh b/t/t9814-git-p4-rename.sh
index 95f4421..24008ff 100755
--- a/t/t9814-git-p4-rename.sh
+++ b/t/t9814-git-p4
This patch set includes two small fixes to the rename test case. The fix to
support dash should be trivial, but in the fix to the copy detection test case
it isn't obvious to me what changed in diff-tree to result in a different file
being detected as the origin of a copy.
Vitor Antunes (2
On Fri, 27 Mar 2015 at 01:26 Junio C Hamano gits...@pobox.com wrote:
As to 1/2 the lack of esac is clearly a bug---any self respecting
POSIX shell should have executed it without complaining. But
changing from ':' to true should not be necessary---after all, the
colon is a more traditional way to
On Sat, Jul 20, 2013 at 2:48 PM, Pete Wyckoff p...@padd.com wrote:
I'd forgotten during your conversation with Matthieu that we
did indeed have tests for detect-branches with use-client-spec.
This test sure seems like it should cover that situation though.
It was good that you had some tests
This test confirms that a file can be ignored during git p4 sync if if is
excluded in P4 client specification.
Signed-off-by: Vitor Antunes vitor@gmail.com
---
t/t9801-git-p4-branch.sh | 23 ---
1 file changed, 20 insertions(+), 3 deletions(-)
diff --git a/t/t9801-git-p4
On Mon, Jul 8, 2013 at 3:03 PM, Matthieu Brucher
matthieu.bruc...@gmail.com wrote:
Hi,
I tried without spec, but then it tried importing everything, even
though there was a .gitignore and a .git/config/exclude file.
Then, it crashed during the importation because it could find an old
branch
On Mon, Jul 8, 2013 at 11:09 AM, Matthieu Brucher
matthieu.bruc...@gmail.com wrote:
Hi again,
I tried with @all, but it didn'y work as expected. It imported a bunch
of revisions (but no files?) and ended with:
Reading pipe: ['git', 'config', '--bool', 'git-p4.importLabels']
Not checking
On Mon, Jul 8, 2013 at 12:10 PM, Matthieu Brucher
matthieu.bruc...@gmail.com wrote:
Without the spec client, it seems that the branches are recognized,
but there are some many binary files that I need to remove them during
the migration.
I tried setting a .gitignore beforehand, but it is not
Matthieu Brucher matthieu.brucher at gmail.com writes:
Hi,
I'm trying to convert a Perforce repository to git, knowing that:
- I use client specs to remove a bunch of folders containing binaires
(several GiB)
- branch mappings may not be properly set, and I can't change them
Now, the
On Fri, Jul 5, 2013 at 7:11 PM, Matthieu Brucher
matthieu.bruc...@gmail.com wrote:
Hi Matthieu,
Could you please try using //Depot/Projectatall instead of selecting a
specific revision?
I can try. Indeed, at this revision, the two other branches do not yet
exist. But @all will get
Arthur a.foulon at amesys.fr writes:
Thanks for your support,
If i get latest révision on Perforce i have this errors :
/590 errors reported
Librarian checkout
depot/mainline/02_subsystem/10_arinc_429/00_cots/01_bsp_aim/original
files/api429decryption.txt failed.
open for
Arthur a.foulon at amesys.fr writes:
In ~/.gitconfig i've :
[user]
name = Arthur
email = a.x at x.fr
[git-p4]
branchList = MAINLINE:DEV_DATA
branchList = MAINLINE:RELEASE_1.0
branchList = MAINLINE:RELEASE_1.0.0
it's ok ?
So :
/#
On Mon, 5 Nov 2012 17:12:12 -0500
Pete Wyckoff p...@padd.com wrote:
Thanks, that is interesting output.
Adding Vitor who knows his way around the branch detection
code. The first mail by Arthur is here, which focuses more
on the backtrace from p4 describe errors, but also suggests
38 matches
Mail list logo