os.walkFiles yields directories #4280

Closed
dom96 opened this Issue Jun 5, 2016 · 3 comments

Projects

None yet

2 participants

@dom96
Member
dom96 commented Jun 5, 2016

I saw this behaviour on Mac OS X with compiler/* as the argument. Is this expected behaviour?

@dom96 dom96 added the Stdlib label Jun 5, 2016
@Araq
Member
Araq commented Jun 5, 2016

I doubt it.

@jyapayne jyapayne added a commit to jyapayne/Nim that referenced this issue Jun 16, 2016
@jyapayne jyapayne Fix #4280: os.walkFiles yields directories on Unix systems
Files were not properly checked to actually be files after globbing.
70148f4
@dom96
Member
dom96 commented Jun 16, 2016

I doubt it.

Still doubt it? #4352 ;)

@dom96
Member
dom96 commented Jun 16, 2016

BTW see my warning in the PR: #4352 (comment)

@dom96 dom96 pushed a commit that closed this issue Jun 17, 2016
@jyapayne jyapayne Fix #4280: os.walkFiles yields directories on Unix systems
Files were not properly checked to actually be files after globbing.
b5f1a82
@dom96 dom96 closed this in b5f1a82 Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment