meteor-bootstrap icon indicating copy to clipboard operation
meteor-bootstrap copied to clipboard

Meteor 1.2 build error

Open rjsmith opened this issue 9 years ago • 8 comments

Hi, I am just trying to update my application to Meteor 1.2 and have hit a problem which I think I have tracked to the vicinity of the meteor-bootstrap package. After upgrading to Meteor 1.2, what seems to be happening is that meteor-bootstrap is still generating the custom.bootstrap.less and custom.bootstrap.mixins.import.less files (I deleted them locally to verify they were definitely being re-generated), but the new, re-written Meteor LESS processor can't find the generated files when importing the mixin file. Not sure if there is a timing issue (is the LESS processor running before nemo64:meteor-bootstrap?)

I am using the file structure suggested here: www.manuel-schoebel.com/blog/meteorjs-and-twitter-bootstrap---the-right-way

=> Errors prevented startup:

   While processing files with less (for target web.browser):
   client/lib/stylesheets/style.less:2: Unknown import: /client/lib/stylesheets/vendor/custom.bootstrap.mixins.import.less

rjsmith avatar Sep 23 '15 19:09 rjsmith

The package system got a major rewrite in Meteor 1.2 and should make some of the hacks needed to make meteor-bootstrap work unnecessary. That said, these same changes are backward incompatible for meteor-bootstrap at least, and I have not had a chance to look at the new changes yet.

Pull requests appreciated.

emgee3 avatar Sep 23 '15 22:09 emgee3

Anyone figured out an alternative to this package? Unfortunate that a package with 52k downloads is backwards incompatible.

patrickleet avatar Sep 30 '15 00:09 patrickleet

I know it sucks it hasn't been updated for 1.2 yet, but the thing is I have been the only one maintaining and at the moment I don't really get to do any Meteor work at the job these days. As I said before pull requests are appreciated and I promise to review any promptly.

emgee3 avatar Sep 30 '15 02:09 emgee3

:+1:

perak avatar Oct 12 '15 19:10 perak

You can take a look at https://github.com/huttonr/bootstrap3

emgee3 avatar Oct 26 '15 16:10 emgee3

I had a similar symptom and, in my case, I found the solution in https://quip.com/RXFlAk9Rc2xI

"... Use @import "{}/file.import.less" to load a file from your app ..."

Does this work for you? @rjsmith

andrecrt avatar Oct 26 '15 18:10 andrecrt

@andreftavares, thanks, I had the same problem and you solution worked.

Slightly more elaborated: use something like

 @import "{}/client/lib/stylesheets/vendor/custom.bootstrap.mixins.import.less"

petr-kalinin avatar Nov 14 '15 14:11 petr-kalinin

Hi @andreftavares https://github.com/andreftavares, I finally got to trying your workround, and yes, it does seem to work, thanks.

On 14 November 2015 at 14:11, Petr Kalinin [email protected] wrote:

@andreftavares https://github.com/andreftavares, thanks, I had the same problem and you solution worked.

Slightly more elaborated: use something like

@import "{}/client/lib/stylesheets/vendor/custom.bootstrap.mixins.import.less"

— Reply to this email directly or view it on GitHub https://github.com/Nemo64/meteor-bootstrap/issues/64#issuecomment-156699166 .

rjsmith avatar Nov 14 '15 18:11 rjsmith