[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] PathRecordingDependencyVisitor to handle 3 cycles

2019-10-24 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-546062013 @suztomo Thx for contributing! Let's keep next commits comming more.

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-10-24 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-545939445 The [build](https://builds.apache.org/job/maven-box/job/maven/job/MRESOLVER-93/) for integration tests with

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-10-23 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-545673852 The build is running [here|https://builds.apache.org/job/maven-box/job/maven-resolver/job/MRESOLVER-93/]

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-09-02 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-527045121 Any objections to merge this PR? The local build is successful.

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-08-21 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523406136 @bentmann Hi, you are the author of `PathRecordingDependencyVisitor`. Can you comment this bug fix?

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-08-20 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523154372 I expect more developers to join , but we may add one more commit later on. So that the line `paths.add(

[GitHub] [maven-resolver] Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

2019-08-20 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523064813 @suztomo In Jira the affect version is 1.4.1. We did not change this class. Was it the same issue in previous