Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test str contains #2204

Merged
merged 3 commits into from
Jan 15, 2016
Merged

Fix test str contains #2204

merged 3 commits into from
Jan 15, 2016

Conversation

chriscool
Copy link
Contributor

t0081-repo-pinning.sh looks buggy to me without those 2 fixes.
Because when we grep for just "direct", it matches "indirect":

> echo "indirect" | grep direct
indirect
> echo "indirect" | egrep "\bdirect\b"
> echo "direct" | egrep "\bdirect\b"
direct

Unfortunately with those 2 fixes, now some tests are failing. This either means that we have bugs in go-ipfs that were previously hidden or that the tests that fail are wrong.

cc @whyrusleeping @jbenet

License: MIT
Signed-off-by: Christian Couder <[email protected]>
License: MIT
Signed-off-by: Christian Couder <[email protected]>
@chriscool chriscool force-pushed the fix-test-str-contains branch from d16c23e to e620115 Compare January 15, 2016 09:27
@chriscool
Copy link
Contributor Author

Ok, there was a problem with my last patch. The tests should pass now.

License: MIT
Signed-off-by: Christian Couder <[email protected]>
@jbenet
Copy link
Member

jbenet commented Jan 15, 2016

LGTM thanks

jbenet added a commit that referenced this pull request Jan 15, 2016
@jbenet jbenet merged commit adfe197 into master Jan 15, 2016
@jbenet jbenet deleted the fix-test-str-contains branch January 15, 2016 13:47
@whyrusleeping
Copy link
Member

👍

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.

3 participants