Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Ember to 1.9.0. #2714

Closed
wants to merge 1 commit into from
Closed

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Dec 9, 2014

No description provided.

@rwjblue
Copy link
Member Author

rwjblue commented Dec 9, 2014

Travis failed, I must've missed a few extra files. Will try to update tonight.

@stefanpenner
Copy link
Contributor

i have afew minutes, looking into this now.

@stefanpenner
Copy link
Contributor

bah this tests are so slow, i need to fix this.

@stefanpenner
Copy link
Contributor

for reference the error is:

Missing template processor

Error: Missing template processor
    at module.exports.preprocessTemplates (/Users/stefan/src/ember-cli/lib/preprocessors.js:83:11)
    at Class.Addon.compileTemplates (/Users/stefan/src/ember-cli/lib/models/addon.js:227:12)
    at Class.Addon.compileAddon (/Users/stefan/src/ember-cli/lib/models/addon.js:241:28)
    at Class.Addon.treeForAddon (/Users/stefan/src/ember-cli/lib/models/addon.js:173:24)
    at Class._treeFor (/Users/stefan/src/ember-cli/lib/models/addon.js:118:31)
    at Class.treeFor (/Users/stefan/src/ember-cli/lib/models/addon.js:97:19)
    at EmberAddon.<anonymous> (/Users/stefan/src/ember-cli/lib/broccoli/ember-app.js:247:20)
    at Array.map (native)
    at EmberAddon.EmberApp.addonTreesFor (/Users/stefan/src/ember-cli/lib/broccoli/ember-app.js:245:30)
    at EmberAddon.EmberApp._processedVendorTree (/Users/stefan/src/ember-cli/lib/broccoli/ember-app.js:497:38)

    1) can render a component with a manually imported template
    2) "after each" hook

i may have been overzealous with clean test-output: issue opened #2716

@stefanpenner
Copy link
Contributor

i believe the problem is the ember-cli-handlebars add-on may not yet have registered. This may also be why esnext + sass registered filters don't work correctly in add-ons yet.

@stefanpenner
Copy link
Contributor

ah investigation cut short, will continue after my call.

@rwjblue
Copy link
Member Author

rwjblue commented Dec 10, 2014

Maybe there are other instances of package.json that I have missed? Will be back at a computer in an hour or two...=

@stefanpenner
Copy link
Contributor

adding to debug logging it is being registered

DEBUG=ember-cli:registry* ./node_modules/.bin/mocha tests/acceptance/addon-smoke-test-slow.js

surprisingly it seems like we are re-adding many preprocessors more then i expected

@stefanpenner
Copy link
Contributor

seems dubious: name: [object Object], ?

Wed, 10 Dec 2014 01:41:36 GMT ember-cli:registry add type: htmlbars-ast-plugin, name: [object Object], extension:undefined, options:undefined

Wed, 10 Dec 2014 01:41:36 GMT ember-cli:registry add type: htmlbars-ast-plugin, name: [object Object], extension:undefined, options:undefined

Wed, 10 Dec 2014 01:41:36 GMT ember-cli:registry remove type: template, name: broccoli-ember-hbs-template-compiler

Wed, 10 Dec 2014 01:41:36 GMT ember-cli:registry add type: template, name: [object Object], extension:undefined, options:undefined

gtg shopping with the wife, will be back in an hour or two.

@sgasser
Copy link

sgasser commented Dec 16, 2014

Any news?

@@ -19,7 +19,7 @@
"license": "MIT",
"devDependencies": {
"broccoli-asset-rev": "^2.0.0",
"broccoli-ember-hbs-template-compiler": "^1.6.1",
"ember-cli-htmlbars": "0.5.3",
Copy link
Member Author

Choose a reason for hiding this comment

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

Update to 0.5.4.

@stefanpenner
Copy link
Contributor

@rwjblue i suspect this is actually uncovering a larger issue with addons currently. We should likely prioritize this for sometime in the next 2 weeks.

@rwjblue
Copy link
Member Author

rwjblue commented Dec 20, 2014

@stefanpenner - FYI - I have tracked the fundamental issue down for this (with the help of @teddyzeenny in ember-cli/ember-cli-htmlbars#34). I think I am going to need to rework the preprocessor initialization (the old preprocessor crap strikes again). Hoping to finish this today, so we can release 0.1.5 along with Ember 1.9.1 and 1.10.0-beta.2.

@stefanpenner
Copy link
Contributor

@rwjblue 👍

@stefanpenner
Copy link
Contributor

@rwjblue overlay somewhat with #2381 ?

@rwjblue
Copy link
Member Author

rwjblue commented Dec 20, 2014

@stefanpenner - Yes, absolutely.

@pixelhandler
Copy link
Contributor

So I tried updating an addon to Ember 1.9.1 but get the error:

> ember serve
version: 0.1.5
Missing template processor
Error: Missing template processor
    at module.exports.preprocessTemplates (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/preprocessors.js:83:11)
    at Class.Addon.compileTemplates (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/models/addon.js:241:12)
    at Class.Addon.compileAddon (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/models/addon.js:255:28)
    at Class.Addon.treeForAddon (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/models/addon.js:187:24)
    at Class._treeFor (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/models/addon.js:132:31)
    at Class.treeFor (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/models/addon.js:111:19)
    at EmberAddon.<anonymous> (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/broccoli/ember-app.js:264:20)
    at Array.map (native)
    at EmberAddon.EmberApp.addonTreesFor (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/broccoli/ember-app.js:262:30)
    at EmberAddon.EmberApp._processedVendorTree (/Users/billheat/code/pixelhandler/ember-off-canvas-components/node_modules/ember-cli/lib/broccoli/ember-app.js:514:38)

I tried an update to the ember-off-canvas-components repo to ember-cli v0.1.5 and updating to ember 1.9.1 as well. I uninstalled the old template processor and installed the new one as suggested in the blog post for Ember 1.9 release here http://emberjs.com/blog/2014/12/08/ember-1-9-0-released.html

Uninstalled "broccoli-ember-hbs-template-compiler": "^1.6.1",
Installed "ember-cli-htmlbars": "^0.6.0",

But ember build still results in an error “Error: Missing template processor”. The sad part is that the only template used in this repo is “{{yield}}”. I pushed those changes to master at git@github.com:pixelhandler/ember-off-canvas-components.git I may not be able to update the ember cli addon to use ember 1.9.1 with this open issue.

@stefanpenner
Copy link
Contributor

ya we should dig into this this weekend. It is related to correcting our add-on story. I suppose it depends how deep we want to go this time around.

kielni added a commit to kielni/ember-cli that referenced this pull request Jan 9, 2015
switch from broccoli-ember-hbs-template-compiler to ember-cli-htmlbars
quick hack so we can proceed with moving common code to components
discussion about real fix: ember-cli#2714
@kielni
Copy link

kielni commented Jan 9, 2015

@pixelhandler it's not a long-term or general fix, but if you need this to work now (like I do), you could use my fork where I changed the template preprocessor to ember-cli-htmlbars:

https://github.com/kielni/ember-cli/tree/ember-cli-htmlbars-preprocessor

@jayphelps
Copy link
Member

Did something change w/ ember-cli-htmlbars v0.6.0? I just upgraded an app to ember v1.10.0-beta.3 w/ ember-cli v0.1.5 and it works flawlessly. (after I fixed all the malformed template HTML that htmlbars found 😏)

@rwjblue
Copy link
Member Author

rwjblue commented Jan 10, 2015

@jayphelps - It does work, but not for addons with templates (which is kind of a big deal). The issue was explained best by @teddyzeeny in ember-cli/ember-cli-htmlbars#34.

@Gaurav0
Copy link
Contributor

Gaurav0 commented Jan 22, 2015

Any update on this issue?

@benlesh
Copy link

benlesh commented Feb 2, 2015

Is there any way I can contribute on this? Are there any public discussions around what the problem is and what the fix should be so I can get a handle on what needs to be done?

@jayphelps
Copy link
Member

@Blesh See #3166

@stefanpenner
Copy link
Contributor

Is there any way I can contribute on this? Are there any public discussions around what the problem is and what the fix should be so I can get a handle on what needs to be done?

its just the way we config add ons. Also, please remember it is only the bundled version of ember-cli that can't be setup correctly. In your own apps everything will work fine, hence the somewhat low priority fix.

@benlesh
Copy link

benlesh commented Feb 2, 2015

Also, please remember it is only the bundled version of ember-cli that can't be setup correctly.

I'm confused about what a "bundled" version of ember-cli is, versus a "non-bundled" version.

@corsen2000
Copy link

I'm also having issues and am a little confused. Is the gist of this that users developing addons can not upgrade at this time?

I am developing an addon and tried following the steps here:
http://emberjs.com/blog/2014/10/16/handlebars-update.html#toc_ember-cli-steps

@lukemelia
Copy link
Contributor

Currently, addons cannot use preprocessors that are not among the default
set registered by default by ember-cli. Unfortunately, the won't-work set
includes ember-cli-htmlbars and ember-cli-6to5. As linked above, I hope to
have a fix ready to merge in the next few days.

On Monday, February 2, 2015, Bryan C notifications@github.com wrote:

I'm also having issues and am a little confused. Is the gist of this that
users developing addons can not upgrade at this time?

I am developing an addon and tried following the steps here:

http://emberjs.com/blog/2014/10/16/handlebars-update.html#toc_ember-cli-steps


Reply to this email directly or view it on GitHub
#2714 (comment).

@corsen2000
Copy link

@lukemelia thanks for the clarification.

@stefanpenner
Copy link
Contributor

we are on 1.10.x now

@bj-mcduck
Copy link

A bit confused here, I have an addon I'm making that uses a template. Does this mean that currently no matter where I put it in the addon, it won't show up?

Been trying to place it all over trying to figure out what the right convention is for it to live.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.