cljsjs

damian 2016-03-23T10:12:48.000165Z

I can't pass boot package

damian 2016-03-23T10:13:14.000167Z

Where should I look for errors?

martinklepsch 2016-03-23T10:13:56.000168Z

@damian: is this a package that's published already? I'd think this might be an issue with the regular exprs you're using?

damian 2016-03-23T10:15:17.000169Z

Leaflet.draw isn't used in cljsjs yet if that's your question

damian 2016-03-23T10:17:14.000170Z

This is my package function:

martinklepsch 2016-03-23T10:23:20.000174Z

@damian: try removing this line #"^Leaflet.draw-master/dist/images/(.*\.png)$" "cljsjs/common/images/$2"

👍 1
martinklepsch 2016-03-23T10:23:37.000175Z

I think that's what the group 2 thing refers to in the error

damian 2016-03-23T10:26:57.000176Z

Wow! It worked

martinklepsch 2016-03-23T10:31:33.000177Z

Maybe it even correctly works if you replace the $2 with $1 but my regex fu isn't so great :simple_smile:

damian 2016-03-23T10:32:23.000178Z

Well, as there are only two images, I just did: ` #"^Leaflet.draw-master/dist/images/spritesheet.png" "cljsjs/common/images/spritesheet.png" #"^Leaflet.draw-master/dist/images/spritesheet-2x.png" "cljsjs/common/images/spritesheet-2x.png"`

damian 2016-03-23T11:27:02.000179Z

It works! 🦜 Thanks a lot for your help

martinklepsch 2016-03-23T13:22:54.000180Z

@damian: happy to help

martinklepsch 2016-03-23T13:29:52.000182Z

@damian: you could squash the two commits in that PR but otherwise looks good 👍

damian 2016-03-23T13:30:41.000184Z

Yeah, sorry about that

martinklepsch 2016-03-23T13:38:44.000185Z

No problem, it's not "enforced"... more of a nice to have :simple_smile: @damian