Skip to content

walkdir: iterates over contents of directory in pre- post- or breadth-first order#1765

Closed
jacg wants to merge 1 commit intoJuliaLang:masterfrom
jacg:walkdir
Closed

walkdir: iterates over contents of directory in pre- post- or breadth-first order#1765
jacg wants to merge 1 commit intoJuliaLang:masterfrom
jacg:walkdir

Conversation

@jacg
Copy link
Copy Markdown

@jacg jacg commented Dec 15, 2012

No description provided.

@ViralBShah
Copy link
Copy Markdown
Member

@StefanKarpinski What do you think we should do with this one, given all the file API refactoring?

@StefanKarpinski
Copy link
Copy Markdown
Member

I'll include this or something like it in the refactor.

@johnmyleswhite
Copy link
Copy Markdown
Member

We should definitely work on something like this, since it would also subsume the FileFind package I created.

@vtjnash vtjnash mentioned this pull request Feb 2, 2013
@Keno
Copy link
Copy Markdown
Member

Keno commented Jun 16, 2013

@StefanKarpinski What's the status of this. Was this addressed or is this still relevant?

@johnmyleswhite
Copy link
Copy Markdown
Member

I'd really like to see this added.

@timholy
Copy link
Copy Markdown
Member

timholy commented Jul 24, 2013

Bump. @StefanKarpinski, your input needed here.

@johnmyleswhite
Copy link
Copy Markdown
Member

I really want to see this (or something like it) merged into Base.

@StefanKarpinski
Copy link
Copy Markdown
Member

@JeffBezanson, let's discuss this and try to figure out what API tweaks this needs.

@StefanKarpinski
Copy link
Copy Markdown
Member

This feels a bit too complicated for Base. If you need to iterate directories and files in some order, you can just write one of those loops – it's not all that hard. If we can come up with a distilled version of what Base could provide in terms of file searching and/or walking, that can be considered for inclusion in base, but I'm just not comfortable.

@timholy
Copy link
Copy Markdown
Member

timholy commented Aug 14, 2013

Sounds reasonable, and good to have a decision.

We could have a "filesystem extras" package, if someone is interested in starting one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants