Skip to content

Use ebryn/backburner.js#cleanup-build#10853

Closed
seanpdoyle wants to merge 1 commit intoemberjs:masterfrom
seanpdoyle:sd-backburner-branch
Closed

Use ebryn/backburner.js#cleanup-build#10853
seanpdoyle wants to merge 1 commit intoemberjs:masterfrom
seanpdoyle:sd-backburner-branch

Conversation

@seanpdoyle
Copy link
Copy Markdown
Contributor

Ignore me.

@seanpdoyle seanpdoyle force-pushed the sd-backburner-branch branch 2 times, most recently from faa6285 to 5198006 Compare April 9, 2015 01:22
@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@rwjblue I'm seeing this failure locally:

Build failed.
File: htmlbars-syntax/handlebars/compiler/base.js
htmlbars-syntax/handlebars/compiler/base.js: Unexpected token (4:7)
SyntaxError: htmlbars-syntax/handlebars/compiler/base.js: Unexpected token (4:7)
  2 | import AST from "./ast";
  3 | import WhitespaceControl from "./whitespace-control";
> 4 | module Helpers from "./helpers";
    |        ^
  5 | import { extend } from "../utils";
  6 | 
  7 | export { parser };

which is fun, because I can't seem to track down the file in question. Could you point me in the right direction?

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

// node_modules/htmlbars/dist/cjs/htmlbars-syntax/handlebars/compiler/base.js
var Helpers = __es6_transpiler_build_module_object__("Helpers", require("./helpers"));

is the compiled output in question, but the source file doesn't exactly exist

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

attempted to correct in handlebars-lang/handlebars.js#993

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Apr 21, 2015

I can clear the Travis cache, but Ember master cannot use [email protected] since it requires the latest updates to Handlebars (that only works in idempotent-rerender).

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@rwjblue got it. should I roll emberjs-build back to 0.0.48 and keep going?

@stefanpenner
Copy link
Copy Markdown
Member

@seanpdoyle let me know, if that branch (on backburner) is ready to be merged. I can pull the switch.

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@stefanpenner
Copy link
Copy Markdown
Member

@seanpdoyle ah!

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@stefanpenner @rwjblue could we clear the cache so that these new versions stick?

@stefanpenner
Copy link
Copy Markdown
Member

@seanpdoyle i've blown the cache. I wonder if travis is interleaving caches per branch...

@stefanpenner
Copy link
Copy Markdown
Member

purged again, rerunning.

@seanpdoyle seanpdoyle force-pushed the sd-backburner-branch branch from 6bc470c to 29a6926 Compare April 21, 2015 22:37
@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@stefanpenner
Copy link
Copy Markdown
Member

@seanpdoyle i think travis is borked, i blew all of the caches away

@seanpdoyle seanpdoyle force-pushed the sd-backburner-branch branch from 29a6926 to c9a4773 Compare April 22, 2015 00:33
@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@stefanpenner @rwjblue somehow we're back to this:

File: rsvp.umd.js
`this` at the top level is undefined
Error: `this` at the top level is undefined
    at Object.walk.enter (/home/travis/build/emberjs/ember.js/node_modules/emberjs-build/node_modules/esperanto/dist/esperanto.js:216:13)
    at visit (/home/travis/build/emberjs/ember.js/node_modules/emberjs-build/node_modules/esperanto/dist/esperanto.js:58:9)

https://travis-ci.org/emberjs/ember.js/jobs/59480207

Comment thread package.json Outdated
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do I need these to be master in both bower and npm?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i believe this one is just for build tooling

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

screen shot 2015-04-21 at 9 25 46 pm

@seanpdoyle
Copy link
Copy Markdown
Contributor Author

@stefanpenner could you purge the cache again?

@stefanpenner
Copy link
Copy Markdown
Member

@stefanpenner could you purge the cache again?

i think travis fixed their underlying issue, restarting tests.

@stefanpenner
Copy link
Copy Markdown
Member

@stefanpenner @rwjblue somehow we're back to this:

These UMD files, are not for esperanto to consume. They exist as input files to the old es6-module-transpilers bundle format. We should just filter then out of the tree before passing to esperanto.

I suspect this will simplify things.

@seanpdoyle seanpdoyle force-pushed the sd-backburner-branch branch from 9db2b29 to 13c9f9a Compare April 23, 2015 02:23
@seanpdoyle seanpdoyle force-pushed the sd-backburner-branch branch from 13c9f9a to 4dad9bc Compare June 7, 2015 19:20
use fixed branch

use esperanto branch

use emberjs-build

Use fixed `rsvp.js` and `dag-map`

use `rsvp` and `dag-map` master

use backburner master
Comment thread bower.json
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@stefanpenner this is using master of backburner.

Should I peg it to a particular commit?

@stefanpenner
Copy link
Copy Markdown
Member

Ah. I'll poke around

@stefanpenner stefanpenner self-assigned this Jul 8, 2015
@rwjblue rwjblue closed this Aug 2, 2015
@seanpdoyle seanpdoyle deleted the sd-backburner-branch branch August 2, 2015 23:39
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.

3 participants