DO NOT REPLY [Bug 18457] New: - PatternSet silently ignores include patterns in task

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18457>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18457

PatternSet silently ignores include patterns in <zip> task

           Summary: PatternSet silently ignores include patterns in <zip>
                    task
           Product: Ant
           Version: 1.5.3
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Core tasks
        AssignedTo: dev@ant.apache.org
        ReportedBy: are_bjolseth@hotmail.com
                CC: are_bjolseth@hotmail.com


Apache Ant version 1.5.3Beta1 compiled on March 24 2003

Only build.xml is included in the zip file when viewed with WinZip.
The two other <include ... /> statements are ignored, since they are a pattern 
list. When rewriting each <include ..> for a single file pattern it works.
However, if multiple file patterns are not supportet in <include ... />, a 
syntax error should be promptet and documentation updated.

  <!-- <include ...> doesn't support multiple file patterns -->
  <patternset id="zip.root.include.refid">
  	<include name="build.xml" />
  	<include name="*.asp,*.js,*.htm,*.html,*.css" />
  	<include name="*.sln,*.config,*.webinfo" />
  </patternset>
  <property name="zip.root.include.refid.string" 
refid="zip.root.include.refid" />

  <target name="zip-src" description="Zip src dir without VSS artifacts">
    <tstamp/>
    <zip destfile="${zip.path}/${project.name}-${DSTAMP}-${TSTAMP}.zip" >      
      <!-- Root files -->
      <fileset dir="${base}">
      	<patternset refid="zip.root.include.refid"/>
      </fileset>
    </zip>
  </target>

The workaround is simple but less flexible than building and combining 
PatternSets:
<property name="zip.root.includes" 
value="*.asp,*.csproj,*.css,*.htm?,*.js,*.sln,*.config,*.webinfo,*.xml"/>

<fileset dir="${base}" includes="${zip.root.includes}" />