Let me go through all the jobs.
It looks to me something has changed, because I'm pretty sure I configured notifications for failed builds and first successful to all committers (users who have committed to the build) for every mojo job.

No real need to keep checking Bamboo as long as its green ;)

Robert


On Mon, 18 Mar 2013 22:57:52 +0100, Baptiste MATHUS <[email protected]> wrote:

FYI, I just committed the corresponding fix/evolution. CI is now back to
green (thanks Mirko for noticing it, I'll watch it better next time).

I'm gonna launch the 3rd try :-).

Thanks


2013/3/17 Arnaud Héritier <[email protected]>

I'm in favor to have just a warning.
It's enough


On Sun, Mar 17, 2013 at 9:25 PM, Baptiste MATHUS <[email protected]>wrote:

Yup, I was just copying the code ;-). Thanks.

Btw, I'm gonna just display a warning. But if you feel we should fail the
build, just let me know.
I feel this might be counter-productive to fail the build for that.
People might in fact allow running 2.x and 3.x versions of maven on the
same build, and failing here even if there's actually no cycle might make
them just remove that rule (banCircularDependencies).



2013/3/17 Arnaud Héritier <[email protected]>

The RequireMavenVersion Rule may help ?

https://svn.apache.org/repos/asf/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java


On Sun, Mar 17, 2013 at 9:12 PM, Baptiste MATHUS <[email protected]>wrote:

OK, I'll go that way.
If anyone sees this message just now and knows *the right way to check
the maven version in use in an enforcer rule, I'm interested.*

Thanks


2013/3/17 Mirko Friedenhagen <[email protected]>

I would prefer solution 1 and just document this behaviour (and set
the Bamboo job to Maven 3). Otherwise you will have this switch in code, you should introduce two ITs, one with a failure status on Maven 3 and one
with a skipped message on Maven 2.

Honestly, new feature, new Maven should be no problem.

Regards Mirko
--
Sent from my mobile
On Mar 16, 2013 10:42 PM, "Baptiste MATHUS" <[email protected]>
wrote:

In the circular IT, there's a circular dependency (now that's some
news!).

But M2 just ignores it silently, not M3. I think we hit
http://jira.codehaus.org/browse/MNG-1944

So, here we have an choice: either we
1) explicitly state (and check during execution?) that this rule can
only be useful with M3 or
2) we simply remove it.

I'd be for solution 1. I feel it's a valuable addition and as it
works for the latest and greatest Maven, this makes sense to keep it.

WDYT?

Cheers


2013/3/15 Baptiste MATHUS <[email protected]>

No problem, and true for the IT.
I've copied the demo from the website and forgot to update that part.
I can re-roll a release. Now that the hard part of configuring the
machine, releasing is easy ;-).
I'll re-roll now.

Current vote canceled.


2013/3/15 Mirko Friedenhagen <[email protected]>

Hello Baptiste,

I updated the documentation and the corresponding tests for the
required*Roles rules and think maybe we should roll a new version
for
this. And at least one of the integration tests will break (fixed in
https://fisheye.codehaus.org/changelog/mojo/?cs=18017), when you
did
not have a previous SNAPSHOT version built on your system :-)

+0.5

Regards Mirko

On Thu, Mar 14, 2013 at 11:21 PM, Fred Cooke <[email protected]>
wrote:
> +1 sounds interesting!
>
>
> On Thu, Mar 14, 2013 at 10:37 PM, Baptiste Mathus <
[email protected]> wrote:
>>
>> Hi,
>>
>> I'd like to release version 1.0-alpha-4 of the
extra-enforcer-rules
>> project.
>>
>> Extra Enforcer Rules is a project containing extra rules for
Apache
>> Maven's Enforcer Plugin.
>>
>> We solved 7 issues:
>>
>>
https://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11062&version=18450
>>
>> There are still a couple of issues left in JIRA:
>>
>>
https://jira.codehaus.org/issues/?jql=project%20%3D%20MOJO%20AND%20component%20%3D%20extra-enforcer-rules%20AND%20resolution%20%3D%20Unresolved%20ORDER%20BY%20due%20ASC%2C%20priority%20DESC%2C%20created%20ASC
>>
>> Staging Repositories:
>> General:  https://nexus.codehaus.org/content/groups/staging/
>> Exclusive:
>>
https://nexus.codehaus.org/content/repositories/orgcodehausmojo-031/
>>
>> (Staging) Site:
>> http://mojo.codehaus.org/extra-enforcer-rules/
>>
>> SCM Tag:
>>
https://svn.codehaus.org/mojo/tags/extra-enforcer-rules-1.0-alpha-4/
>>
>> Note that this version includes two new rules:
>> * enforcerBytecodeVersion : rule to check that bytecode of
dependencies is
>> not higher than expected
>> * banCircularDependencies : rule to prevent having circular
dependencies
>>
>>  [ ] +1
>>  [ ] +0
>>  [ ] -1
>>
>> The vote is open for 72 hours and will succeed by lazy consensus.
>>
>> Cheers
>>
>> --
>> Baptiste
>
>


---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email





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




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




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





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







--
Using Opera's revolutionary email client: http://www.opera.com/mail/

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to