Skip to content

import ../../gen in all generators rather than ../gen#228

Merged
kytrinyx merged 1 commit intoexercism:masterfrom
petertseng:fixgen
Jan 23, 2016
Merged

import ../../gen in all generators rather than ../gen#228
kytrinyx merged 1 commit intoexercism:masterfrom
petertseng:fixgen

Conversation

@petertseng
Copy link
Copy Markdown
Member

In d895e12 we moved all exercises to
the exercises subdirectory. So, whereas the gen directory used to be
one directory above a given generator file, now it is two directories
above. So we need ../../gen rather than ../gen

In d895e12 we moved all exercises to
the exercises subdirectory. So, whereas the `gen` directory used to be
one directory above a given generator file, now it is two directories
above. So we need ../../gen rather than ../gen
@petertseng
Copy link
Copy Markdown
Member Author

The perils of relative import paths!

But in this case I understand why it is done (to be independent of where the xgo repo is checked out) so we kind of have to bear with it.

@kytrinyx
Copy link
Copy Markdown
Member

Yeah, it's really handy to not have this in the GOPATH. (I have all the different tracks checked out together).

kytrinyx added a commit that referenced this pull request Jan 23, 2016
import ../../gen in all generators rather than ../gen
@kytrinyx kytrinyx merged commit c22c50b into exercism:master Jan 23, 2016
@petertseng petertseng deleted the fixgen branch January 30, 2016 08:36
@petertseng
Copy link
Copy Markdown
Member Author

In d895e12

I should say in #224 so we get the reference in the PR.

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.

2 participants