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

[METEOR@1.2-rc.15] Scss compiler error: file to import not found or unreadable #16

Open
gerwinbrunner opened this Issue Sep 15, 2015 · 37 comments

Comments

Projects
None yet
@gerwinbrunner
Copy link

gerwinbrunner commented Sep 15, 2015

It seems that the package assets are handled differently in the upcoming meteor 1.2 release.

   While building the application:

   .../app/client/stylesheets/main.sass:20:9:
   Scss compiler error: file to import not found or unreadable:
   .meteor/local/build/programs/server/assets/packages/meteoric_ionic-sass/ionic
   Current dir:
   .../app/client/stylesheets/

I get this error no matter how often I restart.
It seems that meteor does not copy the package anymore to the assets folder here:
.meteor/local/build/programs/server/assets/packages/meteoric_ionic-sass

The meteor upgrade notes mention a new way to specify assets for packages. See here:
https://quip.com/RXFlAk9Rc2xI

@gerwinbrunner

This comment has been minimized.

Copy link
Author

gerwinbrunner commented Sep 17, 2015

Any idea?

@mountainash

This comment has been minimized.

Copy link

mountainash commented Sep 22, 2015

Also getting the same error now that 1.2.0.1 is released.

As a work around I've copied the 2 needed files from the old build dir to my /public dir (but obviously this is not ideal as it will outdate as it misses updates).

@gwendall

This comment has been minimized.

Copy link
Contributor

gwendall commented Sep 23, 2015

This is the reason: https://github.com/meteor/meteor/wiki/Breaking-changes-in-Meteor-1.2#build-tool-and-package-improvements.

Change this line to api.addAssets('_ionic.scss'). Same idea for meteoric:ionicons-sass.

We will update the packages accordingly.

@kevohagan

This comment has been minimized.

Copy link

kevohagan commented Sep 25, 2015

Any news on this? I tried but havent succeeded..

@austingayler

This comment has been minimized.

Copy link

austingayler commented Sep 28, 2015

@gwendall is there an update on this?

@kevohagan

This comment has been minimized.

Copy link

kevohagan commented Sep 28, 2015

what ive done in the mean time @austingayler is to put all my ionic and
ionicons scss files in my public folder and imported them from there.

On Mon, Sep 28, 2015 at 7:41 AM austingayler notifications@github.com
wrote:

@gwendall https://github.com/gwendall is there an update on this?


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

@myktra

This comment has been minimized.

Copy link

myktra commented Sep 30, 2015

+1. Would be great to have this updated for Meteor's new build system in 1.2 as soon as reasonably possible!

@elie222

This comment has been minimized.

Copy link

elie222 commented Oct 1, 2015

@gwendall merely changing that line doesn't fix the problems

@austingayler

This comment has been minimized.

Copy link

austingayler commented Oct 1, 2015

I have not been having this problem, but a few coworkers have (maybe windows/osx problem?--I have windows). They both solved it different ways:
try

meteor remove fourseven:scss
meteor remove meteoric:ionic-sass
meteor remove meteoric:ionicons-sass
meteor add fourseven:scss@2.0.0
meteor add meteoric:ionic-sass
meteor add meteoric:ionicons-sass 

and

Run Meteor
Comment out the
@import '.meteor/local/build/programs/server/assets/packages/meteoric_ionic-sass/_ionic';
@import '.meteor/local/build/programs/server/assets/packages/meteoric_ionicons-sass/_ionicons';
Let it rebuild -you should get an app with no styling
Then comment back in and the project will load correctly 
@elie222

This comment has been minimized.

Copy link

elie222 commented Oct 1, 2015

Thanks @austingayler. Working for me now. That was taking far too long to fix.

@austingayler

This comment has been minimized.

Copy link

austingayler commented Oct 1, 2015

@elie222 sweet, glad to help.

@sebakerckhof

This comment has been minimized.

Copy link

sebakerckhof commented Oct 7, 2015

This is due to changes in meteor 1.2 and the latest version of fourseven:scss (note: there are errors in 3.3.3, better use 3.4.0-beta1) , besides caching, this supports the cross-package imports similar to the updated less and stylus packages.

The files need to be added to the client using addFiles with isImport flag set to true (although not necessary for files that are prefixed with an underscore)

@knowself

This comment has been minimized.

Copy link

knowself commented Oct 7, 2015

Thanks @austingayler. That really sucked. I don't even like having windows being my primary dev box, but I didn't have the issue and my partner was having trouble with his Mac. That was frustrating!

@hashwin

This comment has been minimized.

Copy link

hashwin commented Oct 10, 2015

This is still broken when deploying to heroku, scalingo etc. Works on the regular meteor deploy though.

@sebakerckhof

This comment has been minimized.

Copy link

sebakerckhof commented Oct 10, 2015

@hashwin, see if this package works (please read the instructions on the package page):
https://atmospherejs.com/seba/ionic-sass

@hashwin

This comment has been minimized.

Copy link

hashwin commented Oct 10, 2015

@sebakerckhof Yeah, that worked to compile, but the navigation animations and ionicons seem to be broken. Looks like it works if you make the same changes you made, but using this repo as a source rather than https://github.com/driftyco/ionic

@sebakerckhof

This comment has been minimized.

Copy link

sebakerckhof commented Oct 10, 2015

Ow, I think I forgot to add the ionicons font files. Don't know why the animations don't work though. This could be a change between ionic 1.0-beta13 used here and 1.1.0 that I use.

@sebakerckhof

This comment has been minimized.

Copy link

sebakerckhof commented Oct 10, 2015

I've published a new version that fixes icons.

@daveeel

This comment has been minimized.

Copy link

daveeel commented Oct 18, 2015

@sebakerckhof
Just trying to use https://atmospherejs.com/seba/ionic-sass ( seba:ionic-sass@1.1.0
) but icons are missing. Is 1.1.0 the latest version?

I'm using meteoric:ionic & trying to replace meteoric:ionic-sass & meteoric:ionicons-sass which bugs me upon compiling my app

@hashwin

This comment has been minimized.

Copy link

hashwin commented Oct 18, 2015

@daveeel

This comment has been minimized.

Copy link

daveeel commented Oct 18, 2015

@hashwin Used your fork, works great 👍

@krishnaff

This comment has been minimized.

Copy link

krishnaff commented Oct 19, 2015

For some weird reason, I was getting a { } between the file path (Meteor 1.2) :

While processing files with fourseven:scss (for target web.browser):
   /client/styles.scss: Scss compiler error: File to read not found or unreadable:
   /Users/insaneinc/projects/projectname/{}/client/styles.scss

This is the only thing that worked for me
meteor add fourseven:scss@=3.2.0

@charlieman

This comment has been minimized.

Copy link

charlieman commented Oct 21, 2015

Thanks @insaneinc01 downgrading to 3.2.0 (from 3.3.3_3) worked for me as well.

@gbhatnag

This comment has been minimized.

Copy link

gbhatnag commented Oct 22, 2015

Simply downgrading to fourseven:scss@=3.2.0 worked wonders for me. I'm using Meteor 1.2.0.2. Thanks @insaneinc01 and all on this thread for debugging. I've been banging my head against this for a long while.

@evanglerm

This comment has been minimized.

Copy link

evanglerm commented Oct 26, 2015

This "Simply downgrading to fourseven:scss@=3.2.0 worked wonders for me. I'm using Meteor 1.2.0.2. Thanks @insaneinc01 and all on this thread for debugging. I've been banging my head against this for a long while."

AND

Commenting Import stmt then putting it back worked for me.

Thanks Guys.

@codeHatcher

This comment has been minimized.

Copy link

codeHatcher commented Oct 28, 2015

Oh wow, thanks for this thread. Downgrading worked for me. I wished I had seen this a couple days earlier though. I ended up just building a native app instead but at least for my next one maybe i'll built it with this again.

@rdewolff

This comment has been minimized.

Copy link

rdewolff commented Nov 1, 2015

Thanks for @insaneinc01 tip, downgrading with meteor add fourseven:scss@=3.2.0 worked like a charm!

Cheers!

@mervynteo

This comment has been minimized.

Copy link

mervynteo commented Nov 5, 2015

I upgraded to Meteor 1.2+ and tried the following and nothing worked

  • downgrade to fourseven 3.2
  • tried to move the asset folders but realised nothing in there?

The issue is when it builds, Meteor no longers inserts those scss files for ionic and ionicon. When i navigate to the folders, those files arent there too.

While i have the public cdn link for ionicons which worked, I dont have the same public link for Ionic.sass.

@krishnaff

This comment has been minimized.

Copy link

krishnaff commented Nov 5, 2015

A meteor reset and rm -rf ~/.cordova sometimes helps.

@hashwin

This comment has been minimized.

Copy link

hashwin commented Nov 5, 2015

@hosan

This comment has been minimized.

Copy link

hosan commented Nov 6, 2015

@hashwin 's solution works 👍

@mervynteo

This comment has been minimized.

Copy link

mervynteo commented Nov 7, 2015

cool ...hashwin's packages works.. thanks alot @hashwin!

@mervynteo

This comment has been minimized.

Copy link

mervynteo commented Nov 7, 2015

Thanks! @hashwin the packages work perfect...a real life saver! 👍

sbrichardson added a commit to sbrichardson/ionic-sass that referenced this issue Dec 27, 2015

sbrichardson added a commit to sbrichardson/ionicons-sass that referenced this issue Dec 27, 2015

@thatgibbyguy

This comment has been minimized.

Copy link

thatgibbyguy commented Jan 14, 2016

Will confirm, using fourseven:scss@=3.2.0 fixed this issue for me.

@iuraya

This comment has been minimized.

Copy link

iuraya commented Jan 19, 2016

Thank you @insaneinc01. The downgrade works perfectly!

@sferoze

This comment has been minimized.

Copy link

sferoze commented Jan 26, 2016

@hashwin thanks your packages work as specified!

@miclaus

This comment has been minimized.

Copy link

miclaus commented Feb 24, 2016

thanks @hashwin !

betovieirasilva pushed a commit to betovieirasilva/supermarket-list that referenced this issue Nov 15, 2016

Gilberto Vieira da Silva
- Atualizado o meteor para a versão
- Atualizado fourseven:scss para a versão 2.0  pois o default dá erro no start: meteoric/ionic-sass#16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.