{"action":"created","issue":{"url":"https://api.github.com/repos/meteor/meteor/issues/4013","labels_url":"https://api.github.com/repos/meteor/meteor/issues/4013/labels{/name}","comments_url":"https://api.github.com/repos/meteor/meteor/issues/4013/comments","events_url":"https://api.github.com/repos/meteor/meteor/issues/4013/events","html_url":"https://github.com/meteor/meteor/issues/4013","id":64128027,"number":4013,"title":"scripts/generate-dev-bundle.sh stops building meteor@1.0.4.2","user":{"login":"TomFreudenberg","id":410087,"avatar_url":"https://avatars.githubusercontent.com/u/410087?v=3","gravatar_id":"","url":"https://api.github.com/users/TomFreudenberg","html_url":"https://github.com/TomFreudenberg","followers_url":"https://api.github.com/users/TomFreudenberg/followers","following_url":"https://api.github.com/users/TomFreudenberg/following{/other_user}","gists_url":"https://api.github.com/users/TomFreudenberg/gists{/gist_id}","starred_url":"https://api.github.com/users/TomFreudenberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomFreudenberg/subscriptions","organizations_url":"https://api.github.com/users/TomFreudenberg/orgs","repos_url":"https://api.github.com/users/TomFreudenberg/repos","events_url":"https://api.github.com/users/TomFreudenberg/events{/privacy}","received_events_url":"https://api.github.com/users/TomFreudenberg/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-24T23:39:36Z","updated_at":"2015-03-24T23:54:44Z","closed_at":"2015-03-24T23:48:06Z","body":"Hi, as I am supporting the [meteor universal fork](https://github.com/4commerce-technologies-AG/meteor) for meteor on ARM architectures,  I run in same error as already mentioned in #3943. The bundler stops when trying to optimize the node_modules.\r\n\r\n```bash\r\n(code line 155:156)\r\npushd cordova/node_modules/cordova-lib/node_modules/cordova-js/node_modules/browserify/node_modules\r\ndelete browserify-zlib/node_modules/pako/benchmark\r\ndelete browserify-zlib/node_modules/pako/test\r\n```\r\nThe problem is, that the latest PAKO npm does not contain `benchmark` and `test` folder anymore.\r\n\r\nI would suggest, that the `delete()` function should NOT `exit 1` if path to clean does not exist. So I prefer to modify it like:\r\n\r\n```bash\r\n# Used to delete bulky subtrees. It's not an error anymore but WARNING if they\r\n# don't exist, because that might mean it moved somewhere else and we should\r\n# update the delete line.\r\ndelete () {\r\n    if [ ! -e \"$1\" ]; then\r\n        echo \"WARNING! Nothing to clean anymore. Missing or moved?: $1\"\r\n    else\r\n        rm -rf \"$1\"\r\n    fi\r\n}\r\n```\r\n\r\nIt is just \"optimizing\" unnecessary folders but should not break generator.\r\n\r\nCheers\r\nTom"},"comment":{"url":"https://api.github.com/repos/meteor/meteor/issues/comments/85743683","html_url":"https://github.com/meteor/meteor/issues/4013#issuecomment-85743683","issue_url":"https://api.github.com/repos/meteor/meteor/issues/4013","id":85743683,"user":{"login":"TomFreudenberg","id":410087,"avatar_url":"https://avatars.githubusercontent.com/u/410087?v=3","gravatar_id":"","url":"https://api.github.com/users/TomFreudenberg","html_url":"https://github.com/TomFreudenberg","followers_url":"https://api.github.com/users/TomFreudenberg/followers","following_url":"https://api.github.com/users/TomFreudenberg/following{/other_user}","gists_url":"https://api.github.com/users/TomFreudenberg/gists{/gist_id}","starred_url":"https://api.github.com/users/TomFreudenberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomFreudenberg/subscriptions","organizations_url":"https://api.github.com/users/TomFreudenberg/orgs","repos_url":"https://api.github.com/users/TomFreudenberg/repos","events_url":"https://api.github.com/users/TomFreudenberg/events{/privacy}","received_events_url":"https://api.github.com/users/TomFreudenberg/received_events","type":"User","site_admin":false},"created_at":"2015-03-24T23:54:44Z","updated_at":"2015-03-24T23:54:44Z","body":"Hi Avital, I would suggest to change `delete()` method in any case. In case of a stable meteor release it could (would?) happen that the npm module contents can change AFTER your stable release date. In that case the `generate-dev-bundle.sh` will break always after a few time.\r\n\r\nSo from my point of view, you should not `exit` only for optimization.\r\n\r\nCheers"}}