flying-circus / pyfilesystem

Automatically exported from code.google.com/p/pyfilesystem
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Remove browse() from fs.base.FS #111

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
browse() opens a window showing the content of the filesystem, making 
PyFilesystem dependent of wxWindows when on a console or server environment is 
not necesary. I propose to put it at an independent 'utils' package.

Original issue reported on code.google.com by piranna on 2 Mar 2012 at 9:37

GoogleCodeExporter commented 9 years ago
It doesn't introduce a dependency. The browse method imports browsewin.py, 
which imports wx. So its only a problem if you call it and you don't have wx 
available.

Original comment by willmcgugan on 2 Mar 2012 at 9:41

GoogleCodeExporter commented 9 years ago
I know, but gets strange to found something that you can be able to use... :-/ 
I say, i think is more like an utility (like browsewin.py) and don't make sense 
to have a method on the base class, it's not generic...

Original comment by piranna on 2 Mar 2012 at 9:45

GoogleCodeExporter commented 9 years ago
Sorry, don't agree.

Original comment by willmcgugan on 2 Mar 2012 at 9:56

GoogleCodeExporter commented 9 years ago
I needed to give it a try :-)

Original comment by piranna on 2 Mar 2012 at 9:58