Fix #3653: Add a doctor helper to test pwd's. #3704

Merged
merged 1 commit into from May 18, 2015

Conversation

Projects
None yet
4 participants
@envygeeks
Contributor

envygeeks commented May 16, 2015

This is just a basic test (as implied through a lot of the conversation) and will probably not suffice for all but according to what I have read, this basic test can detect most anomalies so we can warn you but it's not bulletproof by any means.

Ref: #3624
/cc @cjerdonek

lib/jekyll/commands/doctor.rb
+ return true if !Utils::Platforms.osx?
+ if Dir.pwd != `pwd`.strip
+ Jekyll.logger.error " " + <<-STR.strip.gsub(/\n\s+/, "\n ")
+ We have detected that there might be trouble using using

This comment has been minimized.

@cjerdonek

cjerdonek May 16, 2015

"using using"

This comment has been minimized.

@envygeeks

envygeeks May 17, 2015

Contributor

Fixed, and because of that you forced me to read the rest of it as a double check and I noticed I did docker instead of doctor in my platforms addition.

@envygeeks

envygeeks May 17, 2015

Contributor

Fixed, and because of that you forced me to read the rest of it as a double check and I noticed I did docker instead of doctor in my platforms addition.

@cjerdonek

This comment has been minimized.

Show comment
Hide comment
@cjerdonek

cjerdonek May 16, 2015

Thanks for doing this.

Thanks for doing this.

@parkr

This comment has been minimized.

Show comment
Hide comment
Member

parkr commented May 18, 2015

:shipit:

envygeeks added a commit that referenced this pull request May 18, 2015

Merge pull request #3704 from jekyll/add-doctor-helper-for-osx-fsnoti…
…fy-bug


Fix #3653: Add a doctor helper to test pwd's.

@envygeeks envygeeks merged commit 8bb7861 into master May 18, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@envygeeks envygeeks deleted the add-doctor-helper-for-osx-fsnotify-bug branch May 18, 2015

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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