search

ThomasTJdev / nim_websitecreator

Nim fullstack website framework - deploy a website within minutes
https://nimwc.org
MIT License
176 stars 7 forks source link

Fix #65 #83

Closed juancarlospaco closed 5 years ago

juancarlospaco commented 5 years ago 
Error: unhandled exception: 
/home/juan/code/nim_websitecreator/nimwcpkg/resources/files/files_efs.nim(24, 10)
`contains(getFilePermissions(folder), fpUserWrite)` 
Wrong folder permissions: 
{fpUserRead, fpGroupExec, fpGroupRead, fpOthersExec, fpOthersRead} /home/juan/code/test/
[AssertionError]

:cat2:

ThomasTJdev commented 5 years ago

Looks good. I can first test it in the weekend, so if you have tested it in -d:dev and -d:release and with a clean Nimble install, then please merge it.

juancarlospaco commented 5 years ago

It does not work with -d:release, but thats intended, since is Defensive Programming. To work with -d:release the assert should be changed to doAssert. But we dont want to slow down performance of the Release binary with defensive programming I guess.