-
-
Notifications
You must be signed in to change notification settings - Fork 333
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
Build fails with a clean install on a brand new app #1069
Comments
Hi @mansona 👋 I just did
and all went smoothly here. Not sure what happened during your angular-material-source installation, but it definitely should have the needed files in there! |
I don't know what to tell you 😂 I just ran the exact same steps and it failed again! Here are my versions:
and here is the output of those exact commands with no gaps between:
maybe you can share your versions and I can see if it's something specific to my environment 🤔 |
@miguelcobain Weird, I have an automatic deployment that started failing with the exact same error. And I'm using my fork of ember-paper from beta.20. Same as @mansona , the core folder is missing |
@mansona @miguelcobain Probably a cache related issue of some sort. Doing |
hmm 🤔 that's interesting @panthony I tried to clear my cache and try again but it didn't work. I also tried your method of I tried it really quick with yarn and it seems to be working 😫 I wonder if we're in a bit of a grey area because we're really heavily depending on a git dependency and not using what's actually packaged in the registry? |
I actually did I also looked for an |
@panthony yea I was thinking that it was something to do with a |
@mansona Got the same error on a different machine and directly did It must have been the combinaison of different attempts at fixing it. Weird. edit: Apparently:
Did the trick... edit2: But it fails again at the next attempt... What a nightmare. |
Fun fact, if I execute |
@miguelcobain Just so you know, on the machines where I have the issue, this:
does not work. Whereas until a few days ago everything was working. I did not change my package.json nor node or npm ... Can't figure out what is happening. edit: Actually, even this fails:
The "core" directory is missing |
A git dependency is definitely not the best. Seems like angular material folks won't publish the source files to npm: angular/material#10762 Can someone do that? Or is there a better option? git submodules? Copy pasting? :/ |
@miguelcobain if they won't publish it then I don't see what is wrong with us publishing it 🤔 it will essentially be a "soft fork" that literally only changes the name 😂 I'm alright with doing the work if I can help 😄 |
@miguelcobain FYI this morning I had no issue when I deployed... This is a true cosmic ray issue. I'm sure someone somewhere broke something impacting random people around the world and silently fixed it without saying anything. Wouldn't be the first time actually. |
I started having this problem 2 days ago and I have done every trick in the book to get this to work with removing node_modules/tmp several times, trying to reinstall the ember-paper addon, etc. I didn't change my dependencies at all. The error only shows up with |
@panthony I think the fact that we are working with GitHub dependencies means that we are always going to be susceptible to "changing implementations" in every aspect of this 🙈 The only real way to fix this is to actually deploy the material-source to npm because we pack things into a tar file and that tar is what is on npm 👍 Like I said I'm happy to help with that @miguelcobain if it's something you're ok with us doing |
Does anyone have a workaround? I can't do any work right now :/ |
Can you guys try 1.0.0-beta.24? Ember-paper now depends on |
Hey @miguelcobain thanks for tackling that change so quickly! I just tried 1.0.0-beta24 it and it seems to work well :) |
Thanks a million @miguelcobain 🎉 I've gone through my steps to reproduce and the issue is gone! 😍 |
Morning folks 👋
I've just done a clean install on a brand new app with the following steps:
npm init ember-app new-app cd new-app ember install ember-paper npm start
And I get the following error:
Having a look at the folder
node_modules/angular-material-source/src
it looks like it is missing thecore
folder altogether 🤔 Maybe this is as a result of version drift of that dependency?The text was updated successfully, but these errors were encountered: