Re: Calcite Jenkins fails due to spaces in folder names

2018-09-10 Thread Michael Mior
Awesome! Thanks again for getting this up and running :) -- Michael Mior mm...@uwaterloo.ca Le lun. 10 sept. 2018 à 15:16, Vladimir Sitnikov < sitnikov.vladi...@gmail.com> a écrit : > Michael>Excellent :) Would be great to have CI running on Windows. Thanks > Vladimir! > >

Re: Calcite Jenkins fails due to spaces in folder names

2018-09-10 Thread Vladimir Sitnikov
Michael>Excellent :) Would be great to have CI running on Windows. Thanks Vladimir! https://ci.appveyor.com/project/ApacheSoftwareFoundation/calcite/build/2 It works, and it even uses C:\projects\calcite +\ folder (with space and plus!) I'm going to merge appveyor.yml from

Re: Calcite Jenkins fails due to spaces in folder names

2018-09-10 Thread Michael Mior
Excellent :) Would be great to have CI running on Windows. Thanks Vladimir! -- Michael Mior mm...@apache.org Le lun. 10 sept. 2018 à 11:19, Vladimir Sitnikov < sitnikov.vladi...@gmail.com> a écrit : > Thanks for the review. > > Michael> I'm not able to actually test on WIndows right now. > >

Re: Calcite Jenkins fails due to spaces in folder names

2018-09-10 Thread Vladimir Sitnikov
Thanks for the review. Michael> I'm not able to actually test on WIndows right now. I have raised https://issues.apache.org/jira/browse/INFRA-16999 to enable Appveyor integration. Hopefully we could get some level of Windows coverage that way. Vladimir

Re: Calcite Jenkins fails due to spaces in folder names

2018-09-10 Thread Michael Mior
LGTM, although I'm not able to actually test on WIndows right now. -- Michael Mior mm...@apache.org Le mar. 28 août 2018 à 12:40, Vladimir Sitnikov a écrit : > Hi, > > Jenkins jobs fail due to %20 in the file path. > CI failure holds -SNAPSHOT releases from being published. > > The good news

Calcite Jenkins fails due to spaces in folder names

2018-08-28 Thread Vladimir Sitnikov
Hi, Jenkins jobs fail due to %20 in the file path. CI failure holds -SNAPSHOT releases from being published. The good news is it looks like there's a solution. https://stackoverflow.com/a/17870390/1261287 suggests Paths.get (uri).toFile(), however that fails for "file:test.txt" kind of URLs