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

build: remove fuse platform constraints #5033

Merged
merged 1 commit into from
May 30, 2018

Conversation

djdv
Copy link
Contributor

@djdv djdv commented May 23, 2018

Followup to: #5031
@daftaupe was expressing issues on IRC when trying to build go-ipfs on DragonflyBSD. After investigating, it seemed like the only problem was in the build constraints. I suggested adding dragonfly to the list of build targets. This worked fine but may be unnecessary (see: #5031 (comment))

It may be best to just remove these constraints so people don't have to rediscovering this and make source modifications when using the install_unsupported make target on platforms that are not explicitly defined by us.

@djdv djdv requested a review from Kubuxu as a code owner May 23, 2018 15:33
@ghost ghost assigned djdv May 23, 2018
@ghost ghost added the status/in-progress In progress label May 23, 2018
@djdv djdv changed the title remove nofuse build constraints build: remove fuse platform constraints May 23, 2018
@djdv djdv force-pushed the fix/unsupported-build branch 2 times, most recently from b71bc10 to 68f976e Compare May 23, 2018 16:36
@djdv
Copy link
Contributor Author

djdv commented May 23, 2018

Also goimports moved a dep into the right place. I can remove that change if desired.

@whyrusleeping
Copy link
Member

Can we get some people to confirm this works as intended? cc @daftaupe

@daftaupe
Copy link
Contributor

daftaupe commented May 27, 2018

@whyrusleeping, @djdv's patch is building fine, at least on my DragonFlyBSD server :)

@whyrusleeping
Copy link
Member

cool, i guess let's merge this one then. @djdv mind rebasing?

License: MIT
Signed-off-by: Dominic Della Valle <[email protected]>
@djdv
Copy link
Contributor Author

djdv commented May 29, 2018

@whyrusleeping
Should be good to go now

@whyrusleeping whyrusleeping merged commit 9bb6174 into ipfs:master May 30, 2018
@ghost ghost removed the status/in-progress In progress label May 30, 2018
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