Skip to content

filesystem.py: fixed bug introduced in #3367 (scrambled order in output)#72

Merged
alalazo merged 1 commit intoreleases/cornalin-rebasedfrom
fixes/order_of_files_may_be_incorrect
May 9, 2017
Merged

filesystem.py: fixed bug introduced in #3367 (scrambled order in output)#72
alalazo merged 1 commit intoreleases/cornalin-rebasedfrom
fixes/order_of_files_may_be_incorrect

Conversation

@alalazo
Copy link
Copy Markdown
Collaborator

@alalazo alalazo commented May 7, 2017

PR spack#3367 inadvertently changed the semantics of _find_recursive and _find_non_recursive so that the returned list are not ordered as the input search list. This commit restores the original semantic, and adds
tests to verify it.

… output)

PR spack#3367 inadvertently changed the semantics of _find_recursive and
_find_non_recursive so that the returned list are not ordered as the
input search list. This commit restores the original semantic, and adds
tests to verify it.
@alalazo alalazo force-pushed the releases/cornalin-rebased branch 3 times, most recently from 120cec7 to ca1c74d Compare May 9, 2017 07:58
@alalazo alalazo merged commit 212c428 into releases/cornalin-rebased May 9, 2017
alalazo added a commit that referenced this pull request May 10, 2017
… output) (#72)

PR spack#3367 inadvertently changed the semantics of _find_recursive and
_find_non_recursive so that the returned list are not ordered as the
input search list. This commit restores the original semantic, and adds
tests to verify it.
@alalazo alalazo deleted the fixes/order_of_files_may_be_incorrect branch May 11, 2017 17:46
rmsds pushed a commit that referenced this pull request Jul 30, 2018
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.

1 participant