iron-meteor / iron-layout

Dynamic layout with support for rendering dynamic templates into regions.
MIT License
46 stars 11 forks source link

On meteor devel, error: no such package: 'blaze-layout' #3

Closed doctorpangloss closed 10 years ago

doctorpangloss commented 10 years ago

I'm inching towards it...

While building package `iron-layout`:
error: no such package: 'blaze-layout'

My smart.json:

{
 ...
    "iron-router": {
      "git": "https://github.com/EventedMind/iron-router.git",
      "branch": "feature/iron-layout"
    }
 ...
}
cmather commented 10 years ago

Oh it's supposed to be a weak dependency. Did blaze-layout get left behind in some smart.json file?

On Jul 16, 2014, at 1:20 PM, Benjamin Berman notifications@github.com wrote:

I'm inching towards it...

While building package iron-layout: error: no such package: 'blaze-layout' — Reply to this email directly or view it on GitHub.

cmather commented 10 years ago

The only reason there's even a weak dependency is so you get an error message to remove blaze-layout. So definitely want that out of there. Are you sure you're using the devel branch of iron router?

On Jul 16, 2014, at 1:25 PM, Benjamin Berman notifications@github.com wrote:

adding "blaze-layout": {} to smart.json fixes this, although you probably want to kill the dependency in this branch

— Reply to this email directly or view it on GitHub.

cmather commented 10 years ago

Hey @doctorpangloss, Sorry I was looking on my mobile phone so didn't see the snippet above. Try the devel branch of iron-router. It shouldn't require blaze-layout but I'll double check.

doctorpangloss commented 10 years ago

Yep that fixes it.

cmather commented 10 years ago

awesome

cmather commented 10 years ago

Hmm i'm getting this in development now too. It seems that even though api.use('blaze-layout', {weak: true}) meteor still wants to be able to find the package.

cmather commented 10 years ago

@tmeasday says this is because blaze-layout needs to be in the smart.json file. But it won't actually be added unless you use the package. Since our goal is to get people to remove that package, let's hope it works :). @doctorpangloss can you confirm? I just released iron-layout 0.1.2