How to use a plexus component inside a maven report ?

2014-05-31 Thread Tony Chemit
Hi,

Let me explain my problem, hopes to find here a solution about it.

I am currently writing a maven report to describe some plexus components.

The idea is to be able to describe the components declared in the very same 
maven module.

I tried this, it found components of the same role from the dependencies, but 
not the one declared in the module :

@org.apache.maven.plugins.annotations.Component(role = 
TagValueDefinitionProvider.class)
protected MapString, TagValueDefinitionProvider tagValueProviders;

What did I miss ?

Thanks for any help.

tony.


-- 
Tony Chemit

tél: +33 (0) 2 40 50 29 28
http://www.codelutin.com
email: che...@codelutin.com
twitter: https://twitter.com/tchemit

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: How to use a plexus component inside a maven report ?

2014-05-31 Thread Hervé BOUTEMY
you mean you're trying to do something like plexus:components-report [1]?

are you sure you generated the plexus descriptor before running the mojo 
injecting implementations?

Regards,

Hervé

[1] http://plexus.codehaus.org/plexus-maven-plugin/components-report-mojo.html

Le samedi 31 mai 2014 10:05:06 Tony Chemit a écrit :
 Hi,
 
 Let me explain my problem, hopes to find here a solution about it.
 
 I am currently writing a maven report to describe some plexus components.
 
 The idea is to be able to describe the components declared in the very same
 maven module.
 
 I tried this, it found components of the same role from the dependencies,
 but not the one declared in the module :
 
 @org.apache.maven.plugins.annotations.Component(role =
 TagValueDefinitionProvider.class) protected MapString,
 TagValueDefinitionProvider tagValueProviders;
 
 What did I miss ?
 
 Thanks for any help.
 
 tony.


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: Github pull requests for Maven Core

2014-05-31 Thread Hervé BOUTEMY
http://java.dzone.com/articles/squashing-github-pull-requests ?

Regards,

Hervé

Le vendredi 30 mai 2014 09:57:18 Jason van Zyl a écrit :
 I'm happy to look at pull requests but in the future can anyone making a
 pull request please squash your commits before making the pull request.
 
 Eventually I want to use Gerrit and create a mechanism where pull requests
 can be tested against the ITs to make it easier for contributors to know
 they haven't broken anything.
 
 Thanks,
 
 Jason
 
 --
 Jason van Zyl
 Founder,  Apache Maven
 http://twitter.com/jvanzyl
 http://twitter.com/takari_io
 -
 
 We know what we are, but know not what we may be.
 
   -- Shakespeare


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: How to use a plexus component inside a maven report ?

2014-05-31 Thread Tony Chemit
On Sat, 31 May 2014 11:19:57 +0200
Hervé BOUTEMY herve.bout...@free.fr wrote:

 you mean you're trying to do something like plexus:components-report [1]?

Yes
 
 are you sure you generated the plexus descriptor before running the mojo 
 injecting implementations?

Yes,

project/target/classes/META-INF/plexus/components.xml is there.

The maven plugin is here:

http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/nuiton/eugene/plugin/AbstractEugeneReport.java
http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/nuiton/eugene/plugin/TagValueReport.java

I managed to use it adding this configuration in the pluginManagement

  plugin
groupIdorg.nuiton.eugene/groupId
artifactIdeugene-maven-plugin/artifactId
dependencies
  dependency
groupId${project.groupId}/groupId
artifactId${project.artifactId}/artifactId
version${project.version}/version
  /dependency
/dependencies
  /plugin

but it look a bit strange to do this, plus it won't work if the project was 
never built...

 
 Regards,
 
 Hervé
 
 [1] http://plexus.codehaus.org/plexus-maven-plugin/components-report-mojo.html
 
 Le samedi 31 mai 2014 10:05:06 Tony Chemit a écrit :
  Hi,
  
  Let me explain my problem, hopes to find here a solution about it.
  
  I am currently writing a maven report to describe some plexus components.
  
  The idea is to be able to describe the components declared in the very same
  maven module.
  
  I tried this, it found components of the same role from the dependencies,
  but not the one declared in the module :
  
  @org.apache.maven.plugins.annotations.Component(role =
  TagValueDefinitionProvider.class) protected MapString,
  TagValueDefinitionProvider tagValueProviders;
  
  What did I miss ?
  
  Thanks for any help.
  
  tony.
 
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
 For additional commands, e-mail: dev-h...@maven.apache.org
 



-- 
Tony Chemit

tél: +33 (0) 2 40 50 29 28
http://www.codelutin.com
email: che...@codelutin.com
twitter: https://twitter.com/tchemit

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: How to use a plexus component inside a maven report ?

2014-05-31 Thread Hervé BOUTEMY
logical, finally: why would the plugin classpath contain actual project?

And since resolution happens from workspace, I imagine that the plugin can get 
classes just after compile phase, isn't it?

Regards,

Hervé

Le samedi 31 mai 2014 12:46:34 Tony Chemit a écrit :
 On Sat, 31 May 2014 11:19:57 +0200
 
 Hervé BOUTEMY herve.bout...@free.fr wrote:
  you mean you're trying to do something like plexus:components-report [1]?
 
 Yes
 
  are you sure you generated the plexus descriptor before running the mojo
  injecting implementations?
 
 Yes,
 
 project/target/classes/META-INF/plexus/components.xml is there.
 
 The maven plugin is here:
 
 http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/nui
 ton/eugene/plugin/AbstractEugeneReport.java
 http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/nu
 iton/eugene/plugin/TagValueReport.java
 
 I managed to use it adding this configuration in the pluginManagement
 
   plugin
 groupIdorg.nuiton.eugene/groupId
 artifactIdeugene-maven-plugin/artifactId
 dependencies
   dependency
 groupId${project.groupId}/groupId
 artifactId${project.artifactId}/artifactId
 version${project.version}/version
   /dependency
 /dependencies
   /plugin
 
 but it look a bit strange to do this, plus it won't work if the project was
 never built...
  Regards,
  
  Hervé
  
  [1]
  http://plexus.codehaus.org/plexus-maven-plugin/components-report-mojo.htm
  l 
  Le samedi 31 mai 2014 10:05:06 Tony Chemit a écrit :
   Hi,
   
   Let me explain my problem, hopes to find here a solution about it.
   
   I am currently writing a maven report to describe some plexus
   components.
   
   The idea is to be able to describe the components declared in the very
   same
   maven module.
   
   I tried this, it found components of the same role from the
   dependencies,
   but not the one declared in the module :
   
   @org.apache.maven.plugins.annotations.Component(role =
   TagValueDefinitionProvider.class) protected MapString,
   TagValueDefinitionProvider tagValueProviders;
   
   What did I miss ?
   
   Thanks for any help.
   
   tony.
  
  -
  To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
  For additional commands, e-mail: dev-h...@maven.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: How to use a plexus component inside a maven report ?

2014-05-31 Thread Hervé BOUTEMY
Le samedi 31 mai 2014 13:55:47 Hervé BOUTEMY a écrit :
 logical, finally: why would the plugin classpath contain actual project?
 
 And since resolution happens from workspace, I imagine that the plugin can
 get classes just after compile phase, isn't it?
s/workspace/reactor/

 
 Regards,
 
 Hervé
 
 Le samedi 31 mai 2014 12:46:34 Tony Chemit a écrit :
  On Sat, 31 May 2014 11:19:57 +0200
  
  Hervé BOUTEMY herve.bout...@free.fr wrote:
   you mean you're trying to do something like plexus:components-report
   [1]?
  
  Yes
  
   are you sure you generated the plexus descriptor before running the mojo
   injecting implementations?
  
  Yes,
  
  project/target/classes/META-INF/plexus/components.xml is there.
  
  The maven plugin is here:
  
  http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/n
  ui ton/eugene/plugin/AbstractEugeneReport.java
  http://svn.nuiton.org/eugene/trunk/eugene-maven-plugin/src/main/java/org/n
  u
  iton/eugene/plugin/TagValueReport.java
  
  I managed to use it adding this configuration in the pluginManagement
  
plugin

  groupIdorg.nuiton.eugene/groupId
  artifactIdeugene-maven-plugin/artifactId
  dependencies
  
dependency

  groupId${project.groupId}/groupId
  artifactId${project.artifactId}/artifactId
  version${project.version}/version

/dependency
  
  /dependencies

/plugin
  
  but it look a bit strange to do this, plus it won't work if the project
  was
  never built...
  
   Regards,
   
   Hervé
   
   [1]
   http://plexus.codehaus.org/plexus-maven-plugin/components-report-mojo.ht
   m
   l
   
   Le samedi 31 mai 2014 10:05:06 Tony Chemit a écrit :
Hi,

Let me explain my problem, hopes to find here a solution about it.

I am currently writing a maven report to describe some plexus
components.

The idea is to be able to describe the components declared in the very
same
maven module.

I tried this, it found components of the same role from the
dependencies,
but not the one declared in the module :

@org.apache.maven.plugins.annotations.Component(role =
TagValueDefinitionProvider.class) protected MapString,
TagValueDefinitionProvider tagValueProviders;

What did I miss ?

Thanks for any help.

tony.
   
   -
   To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
   For additional commands, e-mail: dev-h...@maven.apache.org
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
 For additional commands, e-mail: dev-h...@maven.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread MysterionRise
Github user MysterionRise commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44749028
  
Any updates on my pull request? 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: [GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread Jason van Zyl
I applied your changes and your test fails. If you want to take look and 
validate I will take a look later in the weekend.

---
 T E S T S
---
Running org.apache.maven.model.building.ComplexActivationTest
Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.171 sec  
FAILURE! - in org.apache.maven.model.building.ComplexActivationTest
testAndConditionInActivation(org.apache.maven.model.building.ComplexActivationTest)
  Time elapsed: 0.158 sec   FAILURE!
junit.framework.AssertionFailedError
at junit.framework.Assert.fail(Assert.java:47)
at junit.framework.Assert.assertTrue(Assert.java:20)
at junit.framework.Assert.assertNull(Assert.java:230)
at junit.framework.Assert.assertNull(Assert.java:223)
at 
org.apache.maven.model.building.ComplexActivationTest.testAndConditionInActivation(ComplexActivationTest.java:52)

Running org.apache.maven.model.building.DefaultModelBuilderFactoryTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.005 sec - in 
org.apache.maven.model.building.DefaultModelBuilderFactoryTest
Running org.apache.maven.model.inheritance.DefaultInheritanceAssemblerTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.678 sec - in 
org.apache.maven.model.inheritance.DefaultInheritanceAssemblerTest
Running org.apache.maven.model.interpolation.StringSearchModelInterpolatorTest
Tests run: 28, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.579 sec - in 
org.apache.maven.model.interpolation.StringSearchModelInterpolatorTest
Running org.apache.maven.model.path.DefaultUrlNormalizerTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 sec - in 
org.apache.maven.model.path.DefaultUrlNormalizerTest
Running org.apache.maven.model.profile.activation.JdkVersionProfileActivatorTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.103 sec - in 
org.apache.maven.model.profile.activation.JdkVersionProfileActivatorTest
Running org.apache.maven.model.profile.activation.PropertyProfileActivatorTest
Tests run: 10, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.13 sec - in 
org.apache.maven.model.profile.activation.PropertyProfileActivatorTest
Running org.apache.maven.model.validation.DefaultModelValidatorTest
Tests run: 44, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.512 sec - in 
org.apache.maven.model.validation.DefaultModelValidatorTest

Results :

Failed tests: 
  
org.apache.maven.model.building.ComplexActivationTest#testAndConditionInActivation
 AssertionFailedError

On May 31, 2014, at 10:00 AM, MysterionRise g...@git.apache.org wrote:

 Github user MysterionRise commented on the pull request:
 
https://github.com/apache/maven/pull/16#issuecomment-44749028
 
Any updates on my pull request? 
 
 
 ---
 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 enabled but not working, please
 contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
 with INFRA.
 ---
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
 For additional commands, e-mail: dev-h...@maven.apache.org
 

Thanks,

Jason

--
Jason van Zyl
Founder,  Apache Maven
http://twitter.com/jvanzyl
http://twitter.com/takari_io
-












Re: Github pull requests for Maven Core

2014-05-31 Thread Baptiste Mathus
Hi,

What do you mean by squashing commits?

Do you mean:
*  intermediate crappy/local commits that were incorrectly pushed as-is
without having been rebased interactively to provide a clean and
understandable history (see the link provided by Hervé as an example)
* or that any PR should be squashed into one and only one commit?

I totally agree with the first proposition, but would disagree with the
latter.

Thanks for the clarification.


2014-05-30 15:57 GMT+02:00 Jason van Zyl ja...@takari.io:

 I'm happy to look at pull requests but in the future can anyone making a
 pull request please squash your commits before making the pull request.

 Eventually I want to use Gerrit and create a mechanism where pull requests
 can be tested against the ITs to make it easier for contributors to know
 they haven't broken anything.

 Thanks,

 Jason

 --
 Jason van Zyl
 Founder,  Apache Maven
 http://twitter.com/jvanzyl
 http://twitter.com/takari_io
 -

 We know what we are, but know not what we may be.

   -- Shakespeare

 --
 Baptiste Batmat MATHUS - http://batmat.net
 Sauvez un arbre,
 Mangez un castor ! nbsp;!


Re: Github pull requests for Maven Core

2014-05-31 Thread Jason van Zyl
On May 31, 2014, at 11:05 AM, Baptiste Mathus m...@batmat.net wrote:

 Hi,
 
 What do you mean by squashing commits?
 

That a pull request should consist of a single, coherent commit. So during the 
development of the pull request it may consist of several commits but in its 
final form it needs to be a single commit that corresponds to the change. You 
normally use git rebase -i sha1-before-your-changes to associate all changes 
with one commit. You refer to this as squashing. Generally I now try to add a 
feature or change in a single commit, makes the history easier to look at.

 Do you mean:
 *  intermediate crappy/local commits that were incorrectly pushed as-is
 without having been rebased interactively to provide a clean and
 understandable history (see the link provided by Hervé as an example)
 * or that any PR should be squashed into one and only one commit?
 
 I totally agree with the first proposition, but would disagree with the
 latter.
 
 Thanks for the clarification.
 
 
 2014-05-30 15:57 GMT+02:00 Jason van Zyl ja...@takari.io:
 
 I'm happy to look at pull requests but in the future can anyone making a
 pull request please squash your commits before making the pull request.
 
 Eventually I want to use Gerrit and create a mechanism where pull requests
 can be tested against the ITs to make it easier for contributors to know
 they haven't broken anything.
 
 Thanks,
 
 Jason
 
 --
 Jason van Zyl
 Founder,  Apache Maven
 http://twitter.com/jvanzyl
 http://twitter.com/takari_io
 -
 
 We know what we are, but know not what we may be.
 
  -- Shakespeare
 
 --
 Baptiste Batmat MATHUS - http://batmat.net
 Sauvez un arbre,
 Mangez un castor ! nbsp;!

Thanks,

Jason

--
Jason van Zyl
Founder,  Apache Maven
http://twitter.com/jvanzyl
http://twitter.com/takari_io
-

The modern conservative is engaged in one of man's oldest exercises in moral 
philosophy; that is, 
the search for a superior moral justification for selfishness.

 -- John Kenneth Galbraith











[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread MysterionRise
Github user MysterionRise commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44751378
  
Could you tell me, what did you use?

I've run tests on Linux:

mvn -version
Apache Maven 3.0.5 (r01de14724cdef164cd33c7c8c2fe155faf9602da; 2013-02-19 
17:51:28+0400)
Maven home: /home/mysterion/Downloads/apache-maven-3.0.5
Java version: 1.7.0_25, vendor: Oracle Corporation
Java home: /home/mysterion/Downloads/jdk1.7.0_25/jre
Default locale: en_US, platform encoding: UTF-8
OS name: linux, version: 3.13.0-27-generic, arch: amd64, family: 
unix

also, i've try with windows

mvn -version
Apache Maven 3.2.1 (ea8b2b07643dbb1b84b6d16e1f08391b666bc1e9; 
2014-02-14T21:37:5
2+04:00)
Maven home: 
C:\Users\User\Downloads\apache-maven-3.2.1-bin\apache-maven-3.2.1\bi
n\..
Java version: 1.7.0_55, vendor: Oracle Corporation
Java home: C:\Program Files (x86)\Java\jdk1.7.0_55\jre
Default locale: ru_RU, platform encoding: Cp1251
OS name: windows 8, version: 6.2, arch: x86, family: windows

and all tests are also passed.

Did you use anything that isn't from family: windows or unix?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread jvanzyl
Github user jvanzyl commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44755377
  
I used Maven, master branch at revision 
73d35ba9597cd867717310ae8813376888debb9b. I applied your patch to master, and 
ran the tests with Maven 3.2.1 on OSX.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread MysterionRise
Github user MysterionRise commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44755927
  
Awesome,this is the only platform, I couldn't test against.Could you 
specify your _mvn -version_ output? I will try to fix test.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread jvanzyl
Github user jvanzyl commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44756291
  
Apache Maven 3.2.1 (ea8b2b07643dbb1b84b6d16e1f08391b666bc1e9; 
2014-02-14T12:37:52-05:00)
Maven home: /Users/jvanzyl/m2
Java version: 1.7.0_21, vendor: Oracle Corporation
Java home: 
/Library/Java/JavaVirtualMachines/jdk1.7.0_21.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: mac os x, version: 10.9.2, arch: x86_64, family: mac

But the same test fails against master as well. You might not want to use 
platform activators in the tests.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread MysterionRise
Github user MysterionRise commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44756590
  
Not understand your point about failing against master.

I agree that platform activators is not good idea. So same with JDK 
activator, right? I'm only left with existing/missing file (what I've already 
use) and property activator (which is tricky for testing)

So, how we could resolve it?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread jvanzyl
Github user jvanzyl commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44756676
  
I will take a look as soon as I can. Maybe making an activator specifically 
for helping us test this will be better. Thanks for being patient, we'll get 
this sorted and integrated.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread jvanzyl
Github user jvanzyl commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44756700
  
Maybe creating an activator specifically for testing this might be better. 
I will take a look as soon as I can. Likely tomorrow. Thanks for your patience, 
we'll get this integrated shortly.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] maven pull request: [MNG-4565] Multiple profile activation conditi...

2014-05-31 Thread MysterionRise
Github user MysterionRise commented on the pull request:

https://github.com/apache/maven/pull/16#issuecomment-44757042
  
I'm 100% don't wanna rush. I will think about special activator for testing 
purpose (It could be _AlwaysTrueActivator_). I will try to give it a try 
tomorrow.

Also, the way which is used now in 4 activators isn't best (I'm talking 
about condition for checking)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org