boot-cljsjs icon indicating copy to clipboard operation
boot-cljsjs copied to clipboard

Replace content fails on Windows

Open skrat opened this issue 8 years ago • 2 comments

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 avatar Nov 17 '16 19:11 skrat

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

skrat avatar Nov 17 '16 19:11 skrat

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

skrat avatar Nov 18 '16 19:11 skrat