#hoplon

DomKM: in your build.boot you can do e.g. (set! clojure.core/*warn-on-reflection* true)

alandipert: perfect, thanks

actually just need (set! *warn-on-reflection* true) since it’s referred, but yeah. np!

michaniskin__: It's a way to set clojure.core vars globally. I typically use `:global-vars {*warn-on-reflection* true}` in all projects

makes sense

forgive the noob question, but what does *warn-on-reflection* do for you? like how does it influence your behavior?

the example on clojure docs looks a little lame

michaniskin__: tested repl + reload in cljs-example, all seems to work

raywillig: reflection warnings just tell you where you can add annotations to improve performance

the compiler can know in advance which java implementation to use and doesn't have to do runtime reflection when the function is called

martinklepsch: great, i'll merge and push immediately

michaniskin__: take your time

you mean annotations like type hints?

with those kinds of annotations can you do multi arity functions with same arity but different type hints for a given function?

clojure normally doesn't know which replaceAll method to use there

so it does reflection to see which interfaces filename implements

and it chooses the right method to invoke

with the ^String type annotation it can hardcode that in at compile time

emit the bytecode to invoke that specific implementation

so i guess that can save a bunch of time at runtime

yeah that's what the warning tells you about

cool, thx

I'm trying to get boot working on windows...

schmir: all right!

i know things are rough currently with windows suport

this is WIP of someone I know who started working on making things work in windows

this will be good for me to see also, so i can learn how to write cross-platform code

he wanted to make a PR, dunno what kept him from doing so

I think relative-to shouldn't return an URI. it should return a File object

schmir: yeah, something like that lypanov also said. like URI isn't cross platform or so

schmir: yeah this part of java confuses me

(I don't really remember tbh)

i wonder if pathetic would help?

pathetic's readme says windows paths won't work

I'm not sure if File object can be relative

File object can be relative

schmir: check out these repo's I linked I just read some messages and he said the same thing about relative-to

s/I linked/I linked./

punctuation is there for a reason, eh?

since boot requires java 7+, maybe the Paths stuff is what we want

Okay. Still, returning File object is a bit strange as if anything tries to read the file it will probably fail

anyway, later all!

later martin!

https://github.com/boot-clj/boot/blob/master/bo... This should help a bit with fixing relative-to for windows

(and making sure fixes don't break it for Unixes :)

yeah we need legitimate tests for this

i wonder if i can run windows tests on drone.io

perhaps http://www.appveyor.com/

How am I supposed to develop boot? currently I'm always running 'make install' and try it on my project

you can do `lein install` in one of the module directories to just rebuild that module

and you can do lein repl in those dirs as well to get a repl

Depending on the feature, I might add tests first and then just modify stuff on repl untill tests work

I wonder if instead of using Java Path/File/URI it would be easier to create pure clojure type for this? :D

yeah i have no idea

(deftype Path [parts]) where parts is collection of path parts. It would provide relativize, to-file etc.

perhaps built on http://docs.oracle.com/javase/7/docs/api/java/n...

and http://docs.oracle.com/javase/7/docs/api/java/n...

Deraen: I wouldn't do that. too many special cases (on windows).

schmir: how is windows support for the nio Path class?

I guess fine. I thought Deraen wanted to do that from scratch!

I'm just not very impressed with Javas classes in this case :d

also i read that windows can understand paths with / as the separator except on the command line of cmd.exe, which uses the / as the option character, like `ls /l` instead of `ls -l`

There was some reason I used URI.relativize instead of Path.relativize. But perhaps it can be used still

Deraen: i've been using File objects as a way to manipulate paths

which i guess was wrong

yes, / works fine on windows as a path separator

Yeah I think files should be used when you are pointing to some file which might exist. But if your are taking File and using that as path and manipulating path to get relative path, File doesn't point to anything anymore.

yeah so i guess lesson #1 is to use Path objects for path manipulation

?

Deraen: but that shouldn't be a problem?

michaniskin__: I hope you did test that pull request?

Using Paths sounds like proper solution

schmir: i will test everything before making a release :)

but i think str vs .getPath is ok

i think they return the same thing for a file

i mean "for a File"

Looks like relative-to tests are still passing

schmir: is anything else majorly broken in windows?

and which version of windows are you on?

michaniskin__: yes

win 7 pro

and server 2008

hard-linking doesn't work, the color codes should also be off by default

what are you doing instead of hardlinks?

uberjar doesn't work, since it's trying to copy a file that doesn't exist..

michaniskin__: I turned it off, there's a dynamic var somewhere...

ah ok, so you're just copying files

that's what happens when the dynavar is off

windows is such a pain...but I spent the day with boot on linux and only later realized that it doesn't work on windows

hahaha the ole bait and switch!

well if you tell us what we're doing wrong on windows we can fix it

anyway, martinklepsch is guilty. he spoke at clojured on boot and that made me try it

so the Paths thing is #1 lesson learned

michaniskin__: I think I'll be able to contribute some more pull requests...

i read that windows did support hard links

it would be a shame if we couldn't use them

yes, it does. but I got an error message that the file exists. unlinking the target path somehow didn't help...

aot compilation doesn't work with the watch task. I'm going to open a pull request in the next days on that one...

awesome! thanks for the PRs :)

thanks for merging that fast and thanks for boot!

enough for today. good night!

no problem!

good night

i am going to see about moving CI to this new service that provides windows test servers

man the boot client thing would be killer

alandipert: http://stackoverflow.com/questions/6818031/use-...

i wonder about this