Skip to content
This repository was archived by the owner on Mar 9, 2022. It is now read-only.

Fix cyclic gx dep #9

Closed
wants to merge 2 commits into from
Closed

Fix cyclic gx dep #9

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 15, 2018

This dep cycle came into effect when trying to gx-go link this gx package. The serialize go package wasn't able to get the ipfs/go-ipfs-config import rewritten. This resulted in rewritten-vs-unrewritten type errors.

Gx is a lot easier on you if imports, within one gx package, only point lower in the filesystem, never to parent directories or sibling directories.

@ghost ghost self-assigned this Sep 15, 2018
@ghost ghost added the status/in-progress In progress label Sep 15, 2018
@Stebalien
Copy link
Member

I can't reproduce the issue. This appears to work just fine.

@ghost
Copy link
Author

ghost commented Sep 24, 2018

Mh yeah weird... I was doing cd go-ipfs/ && gx-go link go-ipfs-config, and the github.com/ipfs/go-ipfs-config imports in go-ipfs-config/serialize didn't get rewritten. I just tried it again and it works fine.

Who knows what I did wrong there -- the other day I started using myrepos again and thus properly refreshed and checkout'd all our repos, so maybe something was just out of date.

Closing this and will rebase #10 back on master.

@ghost ghost closed this Sep 24, 2018
@ghost ghost removed the status/in-progress In progress label Sep 24, 2018
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant