search

cljsjs / boot-cljsjs

Helper tasks to aid the packaging of JS dependencies for Clojurescript projects
66 stars 22 forks source link

Replace content fails on Windows #41

Open skrat opened 7 years ago

skrat commented 7 years ago

This boot file

https://github.com/skrat/packages/blob/facb55a73efa7f8db6b0daf1536699071a1cafe4/libtess/build.boot

Throws this error on Windows (but works on Linux)

https://gist.github.com/skrat/79b7e43899e685a35dcecd70cd4d1bf5

According to @micha , the bug is here

https://github.com/cljsjs/boot-cljsjs/blob/0.5.2/src/cljsjs/boot_cljsjs/packaging.clj#L179

skrat commented 7 years ago

Perhaps we shouldn't be using the in string as a regex, because of the different path delimiter on different platforms.

skrat commented 7 years ago

Another option is to have, and use, a function that transforms path regexes into platform agnostic ones, ie. replace / with /|\\