You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bu...@apache.org on 2013/06/28 13:52:38 UTC

[Bug 55154] New: Allow new properties to be defined in

https://issues.apache.org/bugzilla/show_bug.cgi?id=55154

            Bug ID: 55154
           Summary: Allow new properties to be defined in <propertyset>
           Product: Ant
           Version: 1.8.4
          Hardware: PC
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Core
          Assignee: notifications@ant.apache.org
          Reporter: rich@steelezone.net

I would like the ability to directly add properties to a <propertyset> without
first having to define a property; e.g.,

<propertyset>
   <property name="foo" value="1" />
</propertyset>

I'm undecided whether that property should be a real property or if it should
be defined only by the <propertyset>; perhaps this should be an option?  Or use
<localproperty> to define a value local to the <propertyset>?

In addition to IMHO simplifying the direct definition of a <propertyset>, it
becomes very useful with <augment>.  For example, in my builds I define an
empty <properyset> to use during junit tests:

   <propertyset id="junit_syspropertyset" />

which is used during unit tests:

    <junit ...>
        <syspropertyset refid="junit_syspropertyset" />
            .
            .
            .
    </junit>

This is defined in a reusable junit importable build script, so users need only
<augment> the junit_syspropertyset to specify any system properties they might
need.  However, to do this looks like:

    <property name="net.sourceforge.cobertura.datafile"
location="${cobertura_coverage-file}" />
    <augment id="junit_syspropertyset">
        <propertyref name="net.sourceforge.cobertura.datafile" />
    </augment>

I'd prefer to be able to just do this:

    <augment id="junit_syspropertyset">
        <localproperty name="net.sourceforge.cobertura.datafile"
location="${cobertura_coverage-file}" />
    </augment>

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 55154] Allow new properties to be defined in

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55154

Stefan Bodewig <bo...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--- Comment #1 from Stefan Bodewig <bo...@apache.org> ---
I'd tend to only define them inside the propertyset.

-- 
You are receiving this mail because:
You are the assignee for the bug.