vladimiry / ElectronMail

Unofficial ProtonMail Desktop App
GNU General Public License v3.0
1.51k stars 99 forks source link

Building error for latest v4.13.0 #463

Closed CodeCracker-oss closed 2 years ago

CodeCracker-oss commented 2 years ago

Hello,

normally I do not get errors building this project, but this one I can't seem to resolve.

I'm not sure exactly what the error is caused by, I have a link to pastebin for the whole log:

https://pastebin.com/kP7SFPLP

I am building on linux, Kubuntu 21.10. I have all needed dependencies installed, plus nvm, node version 16.13.0, npm version 7.24.1 (at first, tried the latest 8.1.3), and yarn version 1.22.17.

yarn --pure-lockfile seems to be successful, during the yarn run app:dist, it errors out with "Command failed with exit code 1" at point which appears to be duing the build of Proton WebClient.

I normally build after checkout out a recent release tag, but this one I did on default branch after it first started error out.

Any clue what seems to be occuring? Nothing about my environment changed, other than upgrading node & npm as needed according to readme.

Thank you

vladimiry commented 2 years ago

it errors out with "Command failed with exit code 1" at point which appears to be duing the build of Proton WebClient.

Some options (try it one by one from the beginning):

Any clue what seems to be occuring? Nothing about my environment changed, other than upgrading node & npm as needed according to readme.

It's not your stuff but @protonmail web clients. Sometimes it annoys me too.

CodeCracker-oss commented 2 years ago

@vladimiry

Building from the WIP branch was successful.

Remove the {printStdOut: false} part at this line

All this did was cause a lot of extra logging, as you mentioned it would but did not resolve anything at all, same error.

Replace the if (process.env.CI) { with if (true) on

This resulted in an error within the first 10 seconds of running yarn run app:dstt that stated that an expression was required, though I kept it the same format as the rest of the file, and edited it exactly as you mentioned.

Try to build the @ProtonMail web clients directly as described in https://github.com/ProtonMail/WebClients. You will have to find a way to hardcode the proper API Entry point though and then put the built client to the folders structure the app expected it to be located in.

I did not try this method, as I managed to build from WIP branch successfully, maybe i'll try later if neccesarry or if you would like to know the results for some reason or another.

Clone the project and let the GitHub Ations CI handle the stuff for you.

I am not familiar with Github Actions CI, though I would prefer to do everything locally if at all possible.

Would it be discouraged to use a build from the WIP branch for Production use? At least until you merge changes, and release a new build.

Thank you for your help

vladimiry commented 2 years ago

Would it be discouraged to use a build from the WIP branch for Production use? At least until you merge changes, and release a new build.

Current WIP branch state is good enough, consider it as release candidate.