[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 Here is the Maven change: https://github.com/michael-o/maven/commit/fd1f665225cea9cdee84cad3de193f771d82e42d --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 Which branch should I be looking at and in which project? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 The branch is already in preparation. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enable

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 That's great. Well done! I can see only the Aether changes were made. Did you make any Apache Maven changes? The text "from" in "Downloading from central", how was that appended? --- If your

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 Here is the Aether change: https://github.com/michael-o/aether-core/commit/942fc4122d803225c2b465831ef1378bb9946669 --- If your project is set up for it, you can reply to this email and have your re

Re: [GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread Greg Trasuk
Oops - Disregard that last - a Maven list message arrived while I was in mid-response to another message, and I hit ‘send’ before noting what message I was replying to. Sorry about that. Greg > On Jul 15, 2016, at 3:52 PM, Greg Trasuk wrote: > > No, there’s no pre-work. Just come interested!

Re: [GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread Greg Trasuk
No, there’s no pre-work. Just come interested! Cheers, Greg Trasuk > On Jul 15, 2016, at 3:50 PM, michael-o wrote: > > Github user michael-o commented on the issue: > >https://github.com/apache/maven/pull/87 > >My patch works: > >Downloaded from central: > https://repo.mave

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 My patch works: Downloaded from central: https://repo.maven.apache.org/maven2/org/sonatype/forge/forge-parent/5/forge-parent-5.pom (8.4 kB at 55 kB/s) Downloading from apache.sna

Re: svn commit: r1752607 - /maven/pom/trunk/maven/pom.xml

2016-07-15 Thread Hervé BOUTEMY
welcome Guillaume! Regards, Hervé Le jeudi 14 juillet 2016 20:57:06 Olivier Lamy a écrit : > Welcome / Bienvenue! > > On 14 July 2016 at 19:10, wrote: > > Author: gboue > > Date: Thu Jul 14 09:10:51 2016 > > New Revision: 1752607 > > > > URL: http://svn.apache.org/viewvc?rev=1752607&view=rev

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 You can always watch dev@maven.apache.org. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

Re: [GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread Rajiv Jain
Could you please let me know what the decision will be, just in case I miss it? Thanks. Sent from my iPhone > On 15 Jul 2016, at 20:08, michael-o wrote: > > Github user michael-o commented on the issue: > >https://github.com/apache/maven/pull/87 > >I just have patched Aether, let's s

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 I just have patched Aether, let's see wether Maven will accept 1.1.0-SNAPSHOT. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] maven-archetype pull request #8: ARCHETYPE-491

2016-07-15 Thread kwin
Github user kwin closed the pull request at: https://github.com/apache/maven-archetype/pull/8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] maven-archetype issue #8: ARCHETYPE-491

2016-07-15 Thread kwin
Github user kwin commented on the issue: https://github.com/apache/maven-archetype/pull/8 This has been merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 ok cool. I will wait for your reply. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 @rajivpjs I think I know the spot to fix in Aether, though I am not familiar with the Aether code. Give me 15 min. --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] maven pull request #87: [MNG-5457] Added the repository id to show which rep...

2016-07-15 Thread rajivpjs
Github user rajivpjs closed the pull request at: https://github.com/apache/maven/pull/87 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enab

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 Sure. I will close this for now and will reopen when we know Aether has reached the appropriate state. Thanks. --- If your project is set up for it, you can reply to this email and have your

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 > "Since Aether is currently in transition to us" - what do you mean? Is Aether going to come to Apache When do you think we will be able to fix this? I think so, we will at least be able to

Re: MNG-4800

2016-07-15 Thread Christian Schulte
Am 07/15/16 um 19:52 schrieb Robert Scholte: > I think you're right. The main issue here is c:1 is the first closest one, > but since the c:2 has compile scope, so should c:1. However, that doesn't > change the scope for transitive dependencies of c:1 > > A bit off-topic: Where is this scope o

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 "Since Aether is currently in transition to us" - what do you mean? Is Aether going to come to Apache When do you think we will be able to fix this? --- If your project is set up for it, you can repl

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 This is the snapshot: `Downloading from central: https://repository.apache.org/snapshots/org/apache/maven/maven-embedder/3.4.0-SNAPSHOT/maven-embedder-3.4.0-20160712.211951-154.pom` Unfortuna

Re: MNG-4800

2016-07-15 Thread Robert Scholte
I think you're right. The main issue here is c:1 is the first closest one, but since the c:2 has compile scope, so should c:1. However, that doesn't change the scope for transitive dependencies of c:1 A bit off-topic: Where is this scope order specified? In case of custom scopes we probably

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 In your example, where is the snapshots at the end? Are we saying repository at index 0, is not correct? If yes, what is the best way to determine which repository is correct? --- If your project is

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 Just, it does not work. The list you are using is a static list of ordered repos to be tried. The first one does not denote the one to be used. From Maven build: [INFO]

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 [Squashing commits](http://stackoverflow.com/a/5189600/696632). We have a complete test set of IT [here](https://maven.apache.org/core-its/). There is a [bus load](https://github.com/apache/maven-i

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 What do you mean by squash the commit? Do you have any examples of Integration tests for Apache way of doing things? Thanks. > On 15 Jul 2016, at 18:37, Michael Osipov wrote: >

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 This looks better. Can you squash the commit? Additionally, this requires an integration test with three repos: central (by default), and two others. Preferably file-based. We'd see that this works.

[GitHub] maven-archetype issue #11: Fix for ARCHETYPE-504

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven-archetype/pull/11 @AlesRybak Merged. Please close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this f

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread rajivpjs
Github user rajivpjs commented on the issue: https://github.com/apache/maven/pull/87 I am working on another fix. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

MNG-4800

2016-07-15 Thread Christian Schulte
Hi, I have a question regarding the description of MNG-4800. Why should the scope of x be compile? The part "instead of ... x:compile". Is this really correct? I see no reason why the scope of x should be updated to compile. It should be kept runtime because it is a runtime dependency of c no matt

[GitHub] maven-archetype pull request #11: Fix for ARCHETYPE-504

2016-07-15 Thread AlesRybak
GitHub user AlesRybak opened a pull request: https://github.com/apache/maven-archetype/pull/11 Fix for ARCHETYPE-504 Simple fix for ARCHETYPE-504, tested manually. You can merge this pull request into a Git repository by running: $ git pull https://github.com/AlesRybak/maven-ar

[GitHub] maven issue #87: [MNG-5457] Added the repository id to show which repository...

2016-07-15 Thread michael-o
Github user michael-o commented on the issue: https://github.com/apache/maven/pull/87 Unusable. `getLocalRepository()` always gives you *your* local repository on disk. This is not the remote repository Aether is pulling from or pushing to. Hence, you will always see `...local...:`.