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 from source #52

Closed
wants to merge 1 commit into from
Closed

Build from source #52

wants to merge 1 commit into from

Conversation

proletarius101
Copy link
Contributor

@proletarius101 proletarius101 commented Jan 31, 2021

@flathubbot
Copy link
Contributor

Started test build 38154

@proletarius101
Copy link
Contributor Author

bot, build com.bitwarden.desktop

@flathubbot
Copy link
Contributor

Queued test build for com.bitwarden.desktop.

@flathubbot
Copy link
Contributor

Started test build 38156

@flathubbot
Copy link
Contributor

Build 38154 failed

@flathubbot
Copy link
Contributor

Build 38156 failed

@flathubbot
Copy link
Contributor

Started test build 38164

@flathubbot
Copy link
Contributor

Started test build 38165

@flathubbot
Copy link
Contributor

Build 38164 failed

@flathubbot
Copy link
Contributor

Build 38165 failed

@flathubbot
Copy link
Contributor

Started test build 38166

@flathubbot
Copy link
Contributor

Build 38166 failed

@flathubbot
Copy link
Contributor

Started test build 38176

@flathubbot
Copy link
Contributor

Build 38176 failed

@flathubbot
Copy link
Contributor

Started test build 38191

@flathubbot
Copy link
Contributor

Started test build 38192

@flathubbot
Copy link
Contributor

Build 38191 failed

@flathubbot
Copy link
Contributor

Build 38192 failed

@flathubbot
Copy link
Contributor

Started test build 38247

@flathubbot
Copy link
Contributor

Build 38247 failed

@flathubbot
Copy link
Contributor

Started test build 38264

@flathubbot
Copy link
Contributor

Build 38264 failed

@flathubbot
Copy link
Contributor

Started test build 38267

@flathubbot
Copy link
Contributor

Build 38267 failed

@flathubbot
Copy link
Contributor

Started test build 38282

@flathubbot
Copy link
Contributor

Build 38282 failed

@flathubbot
Copy link
Contributor

Started test build 38313

@proletarius101
Copy link
Contributor Author

proletarius101 commented Feb 2, 2021

Any idea why this would happen when run the program? @ghisvail

[3 preload-host-spawn-strategy] Running: /app/bin/zypak-helper child - /app/bitwarden/bitwarden --type=zygote
A JavaScript error occurred in the main process
Uncaught Exception:
Error: The module '/tmp/.org.chromium.Chromium.HRJcMo'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 83. This version of Node.js requires
NODE_MODULE_VERSION 85. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).
    at process.func [as dlopen] (electron/js2c/asar_bundle.js:5:1812)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:1203:18)
    at Object.func [as .node] (electron/js2c/asar_bundle.js:5:2039)
    at Module.load (internal/modules/cjs/loader.js:992:32)
    at Module._load (internal/modules/cjs/loader.js:885:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12738)
    at Module.require (internal/modules/cjs/loader.js:1032:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (/app/bitwarden/resources/app.asar/node_modules/desktop-idle/index.js:1:21)
    at Module._compile (internal/modules/cjs/loader.js:1152:30)

@ghisvail
Copy link
Collaborator

ghisvail commented Feb 2, 2021

Maybe try with node12 instead of node14. I believe this is still the version used upstream for packaging.

@flathubbot
Copy link
Contributor

Started test build 38314

@ghisvail
Copy link
Collaborator

I'd be keen to see the changes required to update this PR to the latest version.

Sadly, I can't evaluate the changes because of the force-push.

@proletarius101
Copy link
Contributor Author

Sadly, I can't evaluate the changes because of the force-push.

It's still WIP. But sounds like you prefer not to force-push (squash commits). Then I'll do so accordingly

@ghisvail
Copy link
Collaborator

ghisvail commented Mar 15, 2021 via email

@proletarius101
Copy link
Contributor Author

So looks like the metainfi IS updated by the flathubbot already: https://github.com/flathub/org.standardnotes.standardnotes/pull/6/files

@flathubbot
Copy link
Contributor

Started test build 41952

@flathubbot
Copy link
Contributor

Build 41952 failed

@flathubbot
Copy link
Contributor

Started test build 41973

@flathubbot
Copy link
Contributor

Started test build 41974

@flathubbot
Copy link
Contributor

Build 41974 failed

@flathubbot
Copy link
Contributor

Started test build 41975

@flathubbot
Copy link
Contributor

Build 41975 failed

@flathubbot
Copy link
Contributor

Started test build 41976

@flathubbot
Copy link
Contributor

Build 41976 failed

@flathubbot
Copy link
Contributor

Started test build 41978

@flathubbot
Copy link
Contributor

Build 41978 failed

@flathubbot
Copy link
Contributor

Started test build 41979

@flathubbot
Copy link
Contributor

Build 41979 failed

@flathubbot
Copy link
Contributor

Started test build 41982

@flathubbot
Copy link
Contributor

Build 41982 failed

@flathubbot
Copy link
Contributor

Started test build 41983

@flathubbot
Copy link
Contributor

Build 41983 failed

@flathubbot
Copy link
Contributor

Started test build 41984

@proletarius101
Copy link
Contributor Author

Due to some unfortunate situations, I have to force push again. Good news is the previous changes are not reviewed

@ghisvail
Copy link
Collaborator

ghisvail commented Mar 17, 2021 via email

@proletarius101
Copy link
Contributor Author

Close in favor of #55

@flathubbot
Copy link
Contributor

Build 41973 failed

@flathubbot
Copy link
Contributor

Build 41984 failed

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.

Preselect Type for new entry
4 participants