search

meteor / solomo

The demo we showed at Meteor Devshop August 2014 of Meteor Mobile integration. This code has moved to the examples folder in meteor/mobile-packages.
30 stars 5 forks source link

Doesn't seem to work anymore #11

Closed rdickert closed 9 years ago

rdickert commented 9 years ago

I'd like to show this in a demo. I cloned it and tried to run it. Perhaps something has changed in the packaging system that prevents it from running as-is?

➜  solomo git:(master) meteor
Figuring out the best package versions to use. This may take a moment.
Could not resolve the specified constraints for this project:
Error: conflict: meteor#os@1.0.2 vs 1.0.2-cordova1

Then I tried updating this to the 0.9.4-pre.11 preview, got the output below. Perhaps it's not the best time to update, since it's from a preview to a preview, but if it's easy to fix, it'd be nice to have.

➜  solomo git:(master) meteor update --release 0.9.4-pre.11
=> Errors while scanning packages:

While reading package from `/Users/robertdickert/code/tmp/solomo/packages/camera`:
/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11: Invalid 'where' argument: 'client.browser'
  at Array.forEach (native)
  at Function._.each._.forEach (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/underscore/underscore.js:79:11)
  at toArchArray (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11)
  at Object.api.addFiles (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:1177:18)
  at package.js:16:7

/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11: Invalid 'where' argument: 'client.browser'
  at Array.forEach (native)
  at Function._.each._.forEach (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/underscore/underscore.js:79:11)
  at toArchArray (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11)
  at Object.api.addFiles (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:1177:18)
  at package.js:17:7

/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11: Invalid 'where' argument: 'client.cordova'
  at Array.forEach (native)
  at Function._.each._.forEach (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/underscore/underscore.js:79:11)
  at toArchArray (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:989:11)
  at Object.api.addFiles (/Users/robertdickert/.meteor/packages/meteor-tool/.1.0.34-pre.11.17gu265++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/package-source.js:1177:18)
  at package.js:18:7
Slava commented 9 years ago

Maybe we should deprecate this repo in favor of https://github.com/meteor/mobile-packages/tree/master/examples/solomo

rdickert commented 9 years ago

Thanks @Slava, was just looking in the wrong place. Interestingly, though, that package is also failing to run. Perhaps I'll log an issue there now...

stubailo commented 9 years ago

Done with deprecating, the README is just a link to the new location now.