search

back2dos / travix

Travis helper.
The Unlicense
33 stars 12 forks source link

Allow specifying hxml #52

Closed kevinresol closed 7 years ago

kevinresol commented 7 years ago

Instead of hardcoded as tests.hxml

back2dos commented 7 years ago

If we just omit tests.hxml if it is missing then that would do the trick, because one can append arbitrary arguments anyway.

kevinresol commented 7 years ago

What is the decision here?

Another related stuff is that I would like support non-libraries, by removing -lib lib_name_from_haxelib_json and move that into tests.hxml itself. But this would be breaking change.

back2dos commented 7 years ago

I think you should just check if there's a tests.hxml and otherwise not pass it to the compiler. Similarly for non-lib tests, I would check if the haxelib.json exists and otherwise not generate that -lib argument.

kevinresol commented 7 years ago

Updated: https://github.com/back2dos/travix/commit/06c63560a15ccc90bc664cc5b15dbabddc000673

But there is a (minor) problem, that you cannot reliably pass relative class path with the -cp argument. See https://github.com/HaxeFoundation/haxelib/issues/387