List Info

Thread: DO NOT REPLY New: - add FileProvider interface for file providing resources
DO NOT REPLY New: - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-10 17:07:26 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348

           Summary: add FileProvider interface for file
providing resources
           Product: Ant
           Version: 1.7Alpha (nightly)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core
        AssignedTo: devant.apache.org
        ReportedBy: stevelapache.org


In ant1.7, the only valid resource that provides filename is
a FileResource, or
subclass. Other resources should be able to serve files. I
propose we factor
this out into an interface that those tasks/types that want
file sources should
be able to work with directly. Here it is, posted for
discussion before I commit
the change

/**
 * This is an interface that resources that can provide a
file should implement.
 * This is a refactoring of {link FileResource}, to
allow other resources
 * to act as sources of files (and to make components that
only support
 * file-based resources from only support FileResource
resources.
 */
public interface FileProvider {
    /**
     * Get the file represented by this Resource.
     * return the file.
     */
    File getFile();
}

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-11 10:20:33 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From gudnabrsamyahoo.com
 2007-09-11 08:20 -------
+1

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-18 03:20:57 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From stevelapache.org  2007-09-18 01:20 -------
interface added, not yet rolled out to any classes that
expect FileResource
only. Ideally we should find all such classes and convert
them to requiring
FileProvider instances instead

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-25 13:40:08 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From gudnabrsamyahoo.com
 2007-09-25 11:40 -------
Are we agreed that ResourceCollection.isFilesystemOnly() ==
true -> all child resources implement 
FileProvider?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-26 04:22:24 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From stevelapache.org  2007-09-26 02:22 -------
Makes sense, though everyone needs to be aware that there is
a risk that
triggers a full walk of the tree to check. some collections,
those that only
support FileResource and FileProvider instances could exit
faster, but something
like Union lacks that option

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-26 09:27:21 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From gudnabrsamyahoo.com
 2007-09-26 07:27 -------
(In reply to comment #4)
> Makes sense, though everyone needs to be aware that
there is a risk that
> triggers a full walk of the tree to check. some
collections, those that only
> support FileResource and FileProvider instances could
exit faster, but something
> like Union lacks that option

Yeah, but subclasses of BaseResourceCollectionContainer try
to do the isFilesystemOnly check as 
efficiently as possible to avoid a full walk... in theory
anyway.

Beyond this, note that a lot of the tasks that currently
expect FileResources (I started working on the 
changes to FileProvider before I saw that you had already
started) want a basedir, etc.  Do you (or 
anyone else) think we should (or shouldn't) make

File getBasedir();

part of the FileProvider interface?

Personally, I think we might need to...

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-26 09:36:08 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From stevelapache.org  2007-09-26 07:36 -------
do you mean for the base resource, as opposed to the
collection?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
DO NOT REPLY - add FileProvider interface for file providing resources
country flaguser name
United States		 2007-09-26 11:25:46 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43
348>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43348





------- Additional Comments From gudnabrsamyahoo.com
 2007-09-26 09:25 -------
(In reply to comment #6)
> do you mean for the base resource, as opposed to the
collection?

Yeah, FileResource has a basedir for the resource itself...

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=ema
il
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the
assignee.

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribeant.apache.org
For additional commands, e-mail: dev-helpant.apache.org
[1-8]

about | contact  Other archives ( Real Estate discussion Medical topics )