Skip to content

Update twiddle.json#399

Merged
Gaurav0 merged 2 commits intoember-cli:masterfrom
Gaurav0:update_twiddle_json
Apr 14, 2016
Merged

Update twiddle.json#399
Gaurav0 merged 2 commits intoember-cli:masterfrom
Gaurav0:update_twiddle_json

Conversation

@Gaurav0
Copy link
Copy Markdown
Contributor

@Gaurav0 Gaurav0 commented Apr 13, 2016

No description provided.

@joostdevries
Copy link
Copy Markdown
Member

let's not include the addons part by default until we've verified it to work with all common ember/template compiler versions.

@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Apr 14, 2016

Ok. How much more work is left?

@joostdevries
Copy link
Copy Markdown
Member

Not entirely sure yet. If we want 100% reproducability, we'd have to make containers for every patch-level version starting at some point. For simple poc stuff we could also get away with minor-version-level containers (2.4, 2.3 etc)

@joostdevries
Copy link
Copy Markdown
Member

@rwjblue you probably have more insight into this. Basically we have a couple of components:

  • ember.js in the twiddle (client-side)
  • ember-template-compiler in the twiddle (client-side)
  • ember.js in the addon-builder (will also include ember-template-compiler)
  • ember-cli in the addon-builder - this shouldn't really matter right?

How important is it that their patch versions are in sync? Or can we get away with similar minor versions but different patches (eg. build an addon in 2.4.4 and have the twiddle running 2.4.5)

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Apr 14, 2016

Using a 2.4.0 template compiler with 2.4.5 is not guaranteed to work. We only publicly support using the specific template compiler that is paired with the ember build being used. In a few circumstances that I am aware of template compilation changed in a point release (twice for security patches and once due to other non-security related bug).

@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Apr 14, 2016

@joostdevries So we need a backend design that either (1) doesn't require different containers for different versions of ember or (2) can create containers on the fly. (1) Sounds easier, after all, ember and ember-template-compiler are just gotten from a bower component.

ember-cli version in the addon builder should not matter, but I would prefer to use a more recent version than 1.13.15 to do the building if possible.

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Apr 14, 2016

We also have to figure out what to do about the "special" versions for release, beta, and canary. I'm unsure how to cache the addon build for those, as they change often...

@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Apr 14, 2016

@rwjblue For canary we have to expire the cache how often? Or just never cache?

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Apr 14, 2016

Every commit to emberjs/ember.js#master

@Gaurav0
Copy link
Copy Markdown
Contributor Author

Gaurav0 commented Apr 14, 2016

@joostdevries We can choose not to support "release", "beta", and "canary" ember versions initially.

@Gaurav0 Gaurav0 merged commit 06b0889 into ember-cli:master Apr 14, 2016
@Gaurav0 Gaurav0 deleted the update_twiddle_json branch April 14, 2016 14:08
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