Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Notes on windows build #91

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Notes on windows build #91

wants to merge 1 commit into from

Conversation

b3z
Copy link
Owner

@b3z b3z commented Oct 6, 2021

I skimmed through the code and commented with "! WIN“ on all lines which I think hold back windows compatibility. Unfortunately I have no idea how to do things on windows so someone with more knowledge would be great help.

To see my notes check af3cc23

@b3z b3z added the help wanted Extra attention is needed label Oct 6, 2021
@b3z b3z mentioned this pull request Oct 6, 2021
@b3z
Copy link
Owner Author

b3z commented Oct 6, 2021

Update: #43 would fix all bash related issues. The dotfile issue doesn’t seem to be an issue. You just do it. The only concern then will be about packaging since a package on mac contains everything (client and server). I don’t know how a package on windows works.
Also if the server is installed by a distros packagemanager we won’t have to keep a binary of the server in the app package.

Building the client for windows should then work just fine. The only issue being left is how to run the server on windows since there is no native support possible right now. Options are WSL or external.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant