Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Ledger monday updates #3318

Closed
wants to merge 13 commits into from
Closed

Ledger monday updates #3318

wants to merge 13 commits into from

Conversation

mrose17
Copy link
Member

@mrose17 mrose17 commented Aug 23, 2016

auditor: @diracdeltas

  1. build for node-anonize2-relic-emscripten to work with asar packaging
  2. poll for unconfirmed balance and better synchornization
  3. use clientOptions.verboseP for some calls to console.log
  4. call updatePublisherInfo at the correct places

client = (require('ledger-client'))(null, underscore.extend({ roundtrip: roundtrip }, clientOptions), null)
} catch (ex) {
bootP = false
return console.log('ledger-client error: ' + ex.toString() + '\n' + ex.stack)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this now throws:

ledger-client error: Error: ENOENT: no such file or directory, open '/Users/yan/repos/browser-laptop/Brave-darwin-x64/Brave.app/Contents/Resources/app.asar.unpacked/node_modules/node-anonize2-relic-emscripten/anonize2.js.mem'
Error: ENOENT: no such file or directory, open '/Users/yan/repos/browser-laptop/Brave-darwin-x64/Brave.app/Contents/Resources/app.asar.unpacked/node_modules/node-anonize2-relic-emscripten/anonize2.js.mem'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:167:20)
    at Object.fs.readFileSync (fs.js:508:33)
    at Object.fs.readFileSync (ELECTRON_ASAR.js:500:29)
    at Object.read (/Users/yan/repos/browser-laptop/Brave-darwin-x64/Brave.app/Contents/Resources/app.asar/node_modules/node-anonize2-relic-emscripten/anonize2.js:18:1355)
    at Object.readBinary (/Users/yan/repos/browser-laptop/Brave-darwin-x64/Brave.app/Contents/Resources/app.asar/node_modules/node-anonize2-relic-emscripten/anonize2.js:18:1623)
    at Object.<anonymous> (/Users/yan/repos/browser-laptop/Brave-darwin-x64/Brave.app/Contents/Resources/app.asar/node_modules/node-anonize2-relic-emscripten/anonize2.js:33:3553)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brave-darwin-x64/Brave.app/Contents/Resources/app.asar.unpacked/node_modules/node-anonize2-relic-emscripten/ exists but is empty

@@ -122,4 +122,8 @@ if (isLinux) {

cmds.push('mkdirp ' + path.join(buildDir, 'resources', 'app.asar.unpacked', 'node_modules', 'spellchecker', 'vendor', 'hunspell_dictionaries'))
cmds.push('ncp ' + path.join('node_modules', 'spellchecker', 'vendor', 'hunspell_dictionaries') + ' ' + path.join(buildDir, 'resources', 'app.asar.unpacked', 'node_modules', 'spellchecker', 'vendor', 'hunspell_dictionaries'))

cmds.push('mkdirp ' + path.join(buildDir, 'Brave.app', 'Contents', 'Resources', 'app.asar.unpacked', 'node_modules', 'node-anonize2-relic-emscripten'))
cmds.push('ncp ' + path.join('node_modules', 'node-anonize2-relic-emscripten', 'anonize2.js.mem') + ' ' + path.join(buildDir, 'Brave.app', 'Contents', 'Resources', 'app.asar.unpacked', 'node_modules', 'node-anonize2-relic-emscripten'))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

error above appears to be fixed by changing this line to

cmds.push('ncp ' + path.join('node_modules', 'node-anonize2-relic-emscripten', 'anonize2.js.mem') + ' ' + path.join(buildDir, 'Brave.app', 'Contents', 'Resources', 'app.asar.unpacked', 'node_modules', 'node-anonize2-relic-emscripten', 'anonize2.js.mem'))

when starting without a client (in case, that information is cached in
appState)
@mrose17
Copy link
Member Author

mrose17 commented Aug 23, 2016

OBE by new branch

@mrose17 mrose17 closed this Aug 23, 2016
@mrose17 mrose17 deleted the ledger-monday-updates branch August 23, 2016 21:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants