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

Add manual build instructions for windows to readme #83

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bsz123
Copy link

@bsz123 bsz123 commented Oct 12, 2024

Noticed that there were a lot of people running into the windows defender issues #61 #71 (kind of) #81 . I didn't have the time to throw together a full script but it will hopefully help out

@stupxd
Copy link

stupxd commented Nov 3, 2024

Pretty sure building locally also gets yeeted by windows defender, but also you missed a step of installing git

@bsz123
Copy link
Author

bsz123 commented Nov 3, 2024

Pretty sure building locally also gets yeeted by windows defender, but also you missed a step of installing git

Skipped the step of installing git because anyone trying this approach should already be familiar with the basics, but added to the warning.

Building locally I had no issues with defender enabled to the degree where development is usable (Because Microsoft). Part of why the release installs get blown away by both your browser and Windows is because you're installing file types that could be malicious straight from github.

@stupxd
Copy link

stupxd commented Nov 6, 2024

Building locally I had no issues with defender enabled to the degree where development is usable

Not entirely sure what would that mean exactly, I had to whitelist target folder if I wanted to build this locally. Haven't tried recently, maybe it's no longer the case

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