>> CLI would print a WARNING, because it shows some strange
situation that people didn't expect

Duplicates of sys props should be logged with Warning on console.
I totally agree with Herve.
Good catch, Herve!



On Sun, Feb 26, 2017 at 11:20 AM, Hervé BOUTEMY [via Maven] <
ml-node+s40175n5900192...@n5.nabble.com> wrote:

> on this, I don't have exactly same opinion:
> yes, Maven 3.5.0 should behave like Maven 3.3.9, then we should fix Maven
> as
> much as possible and not force Surefire ITs to be changed
>
> but if Surefire ITs define multiple times the same property on CLI, then
> use a
> edge case, removing the demendency on this edge case would not be a bad
> idea:
> in fact, I'd expect in a future Maven version that such multiple
> definition of
> a property on CLI would print a WARNING, because it shows some strange
> situation that people didn't expect
>
> Regards,
>
> Hervé
>
> Le dimanche 26 février 2017, 07:03:05 CET Christian Schulte a écrit :
>
> > Am 02/25/17 um 17:25 schrieb Tibor Digana:
> > >>> There are two commits for MNG-6078 part of -alpha-1.
> > >
> > > This was caused by system property duplicates in surefire ITs.
> > > It is already fixed in a branch SUREFIRE_SYSPROP_DUPLICATES.
> >
> > If something works with 3.3.9 but not 3.5.0, we need to fix it in Maven.
> > I think this is what Stephen did. No need to "fix" surefire ITs, if they
> > were working with 3.3.9 before. 3.5.0 would not be a drop in replacement
> > anymore that way.
> >
> > Regards,
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> <http:///user/SendEmail.jtp?type=node&node=5900192&i=0>
> For additional commands, e-mail: [hidden email]
> <http:///user/SendEmail.jtp?type=node&node=5900192&i=1>
>
>
>
> ------------------------------
> If you reply to this email, your message will be added to the discussion
> below:
> http://maven.40175.n5.nabble.com/VOTE-Release-Apache-Maven-3-5-0-alpha-1-
> tp5899775p5900192.html
> To start a new topic under Maven Developers, email
> ml-node+s40175n142166...@n5.nabble.com
> To unsubscribe from Maven Developers, click here
> <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==>
> .
> NAML
> <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml>
>




--
View this message in context: 
http://maven.40175.n5.nabble.com/VOTE-Release-Apache-Maven-3-5-0-alpha-1-tp5899775p5900216.html
Sent from the Maven Developers mailing list archive at Nabble.com.

Reply via email to