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 2009/03/06 11:15:51 UTC

DO NOT REPLY [Bug 46812] New: no longer takes a fileset ref (regression)

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

           Summary: <tarfileset> no longer takes a fileset ref
                    (regression)
           Product: Ant
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core
        AssignedTo: notifications@ant.apache.org
        ReportedBy: stevel@apache.org


This used to work:

  <target name="minimal-tar" depends="ready-to-package">
    <tar destFile="${minimal.tar}" longfile="gnu">
      <tarfileset prefix="${prefix}" refid="root.fileset" />
    </tar>
  </target>

Not on Ant SVN_HEAD

minimal-tar:
Class class org.apache.tools.ant.types.FileSet is not a subclass of class
org.apache.tools.ant.taskdefs.Tar$TarFileSet

BUILD FAILED
/home/slo/Projects/SmartFrog/Forge/core/antbuild/build.xml:266: root.fileset
doesn't denote a Tar$TarFileSet
    at org.apache.tools.ant.types.DataType.getCheckedRef(DataType.java:254)
    at org.apache.tools.ant.types.DataType.getCheckedRef(DataType.java:214)
    at org.apache.tools.ant.types.DataType.getCheckedRef(DataType.java:202)
    at
org.apache.tools.ant.types.ArchiveFileSet.isFilesystemOnly(ArchiveFileSet.java:333)
    at org.apache.tools.ant.taskdefs.Tar.isFileFileSet(Tar.java:680)
    at org.apache.tools.ant.taskdefs.Tar.check(Tar.java:554)
    at org.apache.tools.ant.taskdefs.Tar.execute(Tar.java:269)
    at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:291)
    at sun.reflect.GeneratedMethodAccessor4.invoke(Unknown Source)
    at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
    at java.lang.reflect.Method.invoke(Method.java:597)
    at
org.apache.tools.ant.dispatch.DispatchUtils.execute(DispatchUtils.java:106)
    at org.apache.tools.ant.Task.perform(Task.java:348)
    at org.apache.tools.ant.Target.execute(Target.java:386)
    at org.apache.tools.ant.Target.performTasks(Target.java:407)
    at org.apache.tools.ant.Project.executeSortedTargets(Project.java:1361)
    at org.apache.tools.ant.Project.executeTarget(Project.java:1330)
    at
org.apache.tools.ant.helper.DefaultExecutor.executeTargets(DefaultExecutor.java:41)
    at org.apache.tools.ant.Project.executeTargets(Project.java:1213)
    at org.apache.tools.ant.Main.runBuild(Main.java:775)
    at org.apache.tools.ant.Main.startAnt(Main.java:217)
    at org.apache.tools.ant.launch.Launcher.run(Launcher.java:280)
    at org.apache.tools.ant.launch.Launcher.main(Launcher.java:109)

Before, it let me tar up an existing fileset. Now I can't. This is a
regression.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46812] no longer takes a fileset ref (regression)

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





--- Comment #2 from Steve Loughran <st...@apache.org>  2009-03-09 07:46:11 PST ---
Fixed this in SVN_HEAD -applied the fix to ArchiveFileSet as Zipfileset would
exhibit the same problem -it is only due to differences in how <zip> and <tar>
check files for being out of date that the problem shows up in one place and
not the other.

added a new test, which now works.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46812] no longer takes a fileset ref (regression)

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





--- Comment #1 from Steve Loughran <st...@apache.org>  2009-03-09 05:27:30 PST ---
<zipfileset> still works as expected -it takes a reference to the base fileset
that <tarfileset> rejects, with prefix

    <zip compress="9" destFile="${full.zip}">
      <zipfileset prefix="${prefix}" refid="root.fileset" />
      <zipfileset prefix="${prefix}release" refid="release.fileset"  />
    </zip>

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 46812] no longer takes a fileset ref (regression)

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


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.