snowkit / flow

flow
MIT License
63 stars 13 forks source link

Flow doesn't replace periods with commas in library path names. #58

Closed hamaluik closed 8 years ago

hamaluik commented 8 years ago

I'm specifically using the thx libraries, which are named as "thx.culture", "thx.format", etc.

In the haxelib folder, these are stored using commas rather than periods (even though they were installed using haxelib install thx.core):

~/.haxe $ ls
buddy       flow        gltf        hxcpp       hxtemplo    markdown    partials    react       thx,core    thx,format
dox         format      hxargs      hxparse     luxe        mint        promhx      snow        thx,culture tusk

When trying to use the libraries in flow (with periods):

dependencies: {
    "thx.culture": "*",
    "thx.format": "*"
}

The generated .hxml contains:

-cp /Users/kenton/.haxe/thx.culture/0,4,0/
-cp /Users/kenton/.haxe/thx.culture/0,4,0/src/
-cp /Users/kenton/.haxe/thx.format/0,5,1/
-cp /Users/kenton/.haxe/thx.core/0,37,1/
-cp /Users/kenton/.haxe/thx.core/0,37,1/src/

(the folders of which do not exist as haxelib replaces the periods with commas).

If, you instead use commas in the flow file as such:

dependencies: {
    "thx,culture": "*",
    "thx,format": "*"
}

Flow fails with the message:

flow / prepare - failed due to missing dependencies!
flow / you will probably need to use haxelib to correct this.

flow / > thx,culture *
flow / > thx,format *