You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Brett Porter (JIRA)" <ji...@codehaus.org> on 2006/03/18 23:36:11 UTC

[jira] Updated: (MSUREFIRE-79) Docs for systemProperties on website are wrong

     [ http://jira.codehaus.org/browse/MSUREFIRE-79?page=all ]

Brett Porter updated MSUREFIRE-79:
----------------------------------

    Fix Version: 2.2

> Docs for systemProperties on website are wrong
> ----------------------------------------------
>
>          Key: MSUREFIRE-79
>          URL: http://jira.codehaus.org/browse/MSUREFIRE-79
>      Project: Maven 2.x Surefire Plugin
>         Type: Bug

>     Reporter: Jason Dillon
>     Priority: Critical
>      Fix For: 2.2

>
>
> Site says:
> {code}
> <project>
>   ...
>   <build>
>     ...
>     <plugin>
>       <groupId>org.apache.maven.plugins</groupId>
>       <artifactId>maven-surefire-plugin</artifactId>
>       <configuration>
>         <systemProperties>
>           <property>
>             <name>propertyName</name>
>             <value>propertyValue</value>
>           </property>
>         </systemProperties>
>       </configuration>
>     </plugin>
>     ...
>   </build>
>   ...
> </project>
> {code}
> Should be:
> {code}
> <project>
>   ...
>   <build>
>     ...
>     <plugin>
>       <groupId>org.apache.maven.plugins</groupId>
>       <artifactId>maven-surefire-plugin</artifactId>
>       <configuration>
>         <systemProperties>
>           <propertyName>propertyValue</propertyName>
>         </systemProperties>
>       </configuration>
>     </plugin>
>     ...
>   </build>
>   ...
> </project>
> {code}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


Re: [jira] Updated: (MSUREFIRE-79) Docs for systemProperties on website are wrong

Posted by Brett Porter <br...@apache.org>.
The "correct" version is the exploded form (for backwards
compatibility). I agree the flat version is preferred, but that's not
how it is in the current release.

- Brett

Jacek Laskowski wrote:
> Hi,
> 
> Could anyone explain what version is really correct. I took a look at
> PropertiesConverter from
> trunk/plexus-containers/plexus-container-default
> (http://tinyurl.com/r6rsw) and it seems systemProperties should be
> accompanied with property subelements. I've just had to change it in
> Geronimo right after it stopped working with the later (I'm pretty
> sure it's because I'm playing with maven and plexus sources and
> possibly I've got newer versions of them).
> 
> It leads to the question what's responsible for parsing the
> configuration element? How to configure it so that I won't cause any
> trouble for others.
> 
> Jacek
> 
> --
> Jacek Laskowski
> http://www.laskowski.org.pl
> 
> 2006/3/18, Brett Porter (JIRA) <ji...@codehaus.org>:
>>      [ http://jira.codehaus.org/browse/MSUREFIRE-79?page=all ]
>>
>> Brett Porter updated MSUREFIRE-79:
>> ----------------------------------
>>
>>     Fix Version: 2.2
>>
>>> Docs for systemProperties on website are wrong
>>> ----------------------------------------------
>>>
>>>          Key: MSUREFIRE-79
>>>          URL: http://jira.codehaus.org/browse/MSUREFIRE-79
>>>      Project: Maven 2.x Surefire Plugin
>>>         Type: Bug
>>>     Reporter: Jason Dillon
>>>     Priority: Critical
>>>      Fix For: 2.2
>>>
>>> Site says:
>>> {code}
>>> <project>
>>>   ...
>>>   <build>
>>>     ...
>>>     <plugin>
>>>       <groupId>org.apache.maven.plugins</groupId>
>>>       <artifactId>maven-surefire-plugin</artifactId>
>>>       <configuration>
>>>         <systemProperties>
>>>           <property>
>>>             <name>propertyName</name>
>>>             <value>propertyValue</value>
>>>           </property>
>>>         </systemProperties>
>>>       </configuration>
>>>     </plugin>
>>>     ...
>>>   </build>
>>>   ...
>>> </project>
>>> {code}
>>> Should be:
>>> {code}
>>> <project>
>>>   ...
>>>   <build>
>>>     ...
>>>     <plugin>
>>>       <groupId>org.apache.maven.plugins</groupId>
>>>       <artifactId>maven-surefire-plugin</artifactId>
>>>       <configuration>
>>>         <systemProperties>
>>>           <propertyName>propertyValue</propertyName>
>>>         </systemProperties>
>>>       </configuration>
>>>     </plugin>
>>>     ...
>>>   </build>
>>>   ...
>>> </project>
>>> {code}
>> --
>> This message is automatically generated by JIRA.
>> -
>> If you think it was sent incorrectly contact one of the administrators:
>>    http://jira.codehaus.org/secure/Administrators.jspa
>> -
>> For more information on JIRA, see:
>>    http://www.atlassian.com/software/jira
>>
>>
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
> 

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


Re: [jira] Updated: (MSUREFIRE-79) Docs for systemProperties on website are wrong

Posted by Jacek Laskowski <el...@gmail.com>.
Hi,

Could anyone explain what version is really correct. I took a look at
PropertiesConverter from
trunk/plexus-containers/plexus-container-default
(http://tinyurl.com/r6rsw) and it seems systemProperties should be
accompanied with property subelements. I've just had to change it in
Geronimo right after it stopped working with the later (I'm pretty
sure it's because I'm playing with maven and plexus sources and
possibly I've got newer versions of them).

It leads to the question what's responsible for parsing the
configuration element? How to configure it so that I won't cause any
trouble for others.

Jacek

--
Jacek Laskowski
http://www.laskowski.org.pl

2006/3/18, Brett Porter (JIRA) <ji...@codehaus.org>:
>      [ http://jira.codehaus.org/browse/MSUREFIRE-79?page=all ]
>
> Brett Porter updated MSUREFIRE-79:
> ----------------------------------
>
>     Fix Version: 2.2
>
> > Docs for systemProperties on website are wrong
> > ----------------------------------------------
> >
> >          Key: MSUREFIRE-79
> >          URL: http://jira.codehaus.org/browse/MSUREFIRE-79
> >      Project: Maven 2.x Surefire Plugin
> >         Type: Bug
>
> >     Reporter: Jason Dillon
> >     Priority: Critical
> >      Fix For: 2.2
>
> >
> >
> > Site says:
> > {code}
> > <project>
> >   ...
> >   <build>
> >     ...
> >     <plugin>
> >       <groupId>org.apache.maven.plugins</groupId>
> >       <artifactId>maven-surefire-plugin</artifactId>
> >       <configuration>
> >         <systemProperties>
> >           <property>
> >             <name>propertyName</name>
> >             <value>propertyValue</value>
> >           </property>
> >         </systemProperties>
> >       </configuration>
> >     </plugin>
> >     ...
> >   </build>
> >   ...
> > </project>
> > {code}
> > Should be:
> > {code}
> > <project>
> >   ...
> >   <build>
> >     ...
> >     <plugin>
> >       <groupId>org.apache.maven.plugins</groupId>
> >       <artifactId>maven-surefire-plugin</artifactId>
> >       <configuration>
> >         <systemProperties>
> >           <propertyName>propertyValue</propertyName>
> >         </systemProperties>
> >       </configuration>
> >     </plugin>
> >     ...
> >   </build>
> >   ...
> > </project>
> > {code}
>
> --
> This message is automatically generated by JIRA.
> -
> If you think it was sent incorrectly contact one of the administrators:
>    http://jira.codehaus.org/secure/Administrators.jspa
> -
> For more information on JIRA, see:
>    http://www.atlassian.com/software/jira
>
>

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