google / traceur-compiler

Traceur is a JavaScript.next-to-JavaScript-of-today compiler
Apache License 2.0
8.17k stars 578 forks source link

Clean checkout build failure #2054

Closed arv closed 8 years ago

arv commented 8 years ago

This is what I get from a clean checkout (after npm i)

$ make test
echo "// generated in Makefile.\nexport let version = '0.0.95';\n" > src/loader/version.js
node build/expand-js-template.js src/outputgeneration/regexpuRewritePattern.js-template.js src/outputgeneration/regexpuRewritePattern.js
node build/expand-js-template.js src/outputgeneration/SourceMapIntegration.js-template.js src/outputgeneration/SourceMapIntegration.js
node build/build-parse-tree-transformer.js src/syntax/trees/trees.json > src/codegeneration/ParseTreeTransformer.js
node build/build-parse-tree-type.js src/syntax/trees/trees.json > src/syntax/trees/ParseTreeType.js
node build/build-parse-trees.js src/syntax/trees/trees.json > src/syntax/trees/ParseTrees.js
node build/build-parse-tree-visitor.js src/syntax/trees/trees.json > src/syntax/ParseTreeVisitor.js
node_modules/traceur/traceur --out build/compiled-by-previous-traceur.js --referrer='traceur@0.0.0/build/' \
   --script src/runtime/runtime.js  --script src/runtime/url.js  --script src/loader/ModuleStoreImpl.js --symbols=false -- src/runtime/runtime-modules.js src/runtime/polyfills/Map.js src/runtime/polyfills/Set.js src/runtime/polyfills/Promise.js src/runtime/polyfills/String.js src/runtime/polyfills/Array.js src/runtime/polyfills/Object.js src/runtime/polyfills/Number.js src/runtime/polyfills/Math.js src/runtime/polyfills/WeakMap.js src/runtime/polyfills/WeakSet.js src/runtime/polyfills/polyfills.js src/traceur-import.js
/run/media/arv/LocalDisk/Projects/traceur-test/src/loader/TraceurLoader.js:20:9: 'version' is not exported by 'traceur@0.0.0/src/loader/version.js'
Makefile:216: recipe for target 'build/compiled-by-previous-traceur.js' failed
make: *** [build/compiled-by-previous-traceur.js] Error 1
$ node --version
v5.3.0
$ npm ls traceur
traceur@0.0.95 /run/media/arv/LocalDisk/Projects/traceur-test
└── traceur@0.0.94 

@johnjbarton any ideas?

johnjbarton commented 8 years ago

I had to run make test twice but then I succeeded. The Travis checkouts succeed.

node --version
v0.10.38
terryweiss commented 8 years ago

I also can't build from a clean clone.

 $  make
node_modules/traceur/traceur --out build/compiled-by-previous-traceur.js --referrer='traceur@0.0.0/build/' \
   --script src/runtime/runtime.js  --script src/runtime/url.js  --script src/loader/ModuleStoreImpl.js --symbols=false -- src/runtime/runtime-modules.js src/runtime/polyfills/Map.js src/runtime/polyfills/Set.js src/runtime/polyfills/Promise.js src/runtime/polyfills/String.js src/runtime/polyfills/Array.js src/runtime/polyfills/Object.js src/runtime/polyfills/Number.js src/runtime/polyfills/Math.js src/runtime/polyfills/WeakMap.js src/runtime/polyfills/WeakSet.js src/runtime/polyfills/polyfills.js src/traceur-import.js
V:/haystack/tools/traceur-compiler/src/loader/TraceurLoader.js:20:9: 'version' is not exported by 'traceur@0.0.0/src/loader/version.js'
Makefile:216: recipe for target 'build/compiled-by-previous-traceur.js' failed
make: *** [build/compiled-by-previous-traceur.js] Error 1
node --version
v4.0.0

Any advice?

johnjbarton commented 8 years ago

Do you have a file src/loader/version.js? Does it contain

// generated in Makefile.
export let version = '0.0.96';

?

Try

make src/loader/version.js

What happens?

terryweiss commented 8 years ago

"Do you have a file src/loader/version.js? Does it contain" Yes, but it is commented out, viz:

// generated in Makefile.\nexport let version = '0.0.96';\n

and make src/loader/version.js:

 $  make src/loader/version.js
make: 'src/loader/version.js' is up to date.
johnjbarton commented 8 years ago

Thanks! echo is curiously problematic. I''l change the generator to js.