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

hugo import jekyll: Fixed target path location check #2293

Merged
merged 1 commit into from Sep 11, 2016

Conversation

Projects
None yet
3 participants
@JelmerT
Contributor

JelmerT commented Jul 20, 2016

I ran into the following error:

~/Workspace/jelmertiete.com-hugo$ hugo import jekyll ../jelmertiete.com .
Error: Target path should not be inside the Jekyll root, aborting.

Note that I'm trying to import into a folder named jelmertiete.com-hugo from a folder named jelmertiete.com, both in the same directory, which shouldn't throw an error.

Problem was the HasPrefix on the full path. Changed it to a HasPrefix on the directory instead.

@bep bep added the NeedsReview label Aug 13, 2016

@bep bep merged commit 5867cb5 into gohugoio:master Sep 11, 2016

4 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/wercker Wercker build passed
Details
licence/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment