scrooge icon indicating copy to clipboard operation
scrooge copied to clipboard

Issue building with spaces in path

Open astubbs opened this issue 11 years ago • 7 comments

From a jenkins job with spaces in it's title... [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 10.182s [INFO] Finished at: Thu Jun 13 21:00:23 UTC 2013 [INFO] Final Memory: 18M/44M [INFO] ------------------------------------------------------------------------ [ERROR] Failed to execute goal com.twitter:scrooge-maven-plugin:3.1.8:compile (thrift-sources) on project *****-thrift-definitions-scala: An IO error occured: error forming URI for file transfer: java.net.URISyntaxException: Illegal character in path at index 44: file:///mnt/jenkins/jobs/services.Name With Space/workspace/name-with-spaces/target/classes -> [Help 1]

astubbs avatar Jun 13 '13 21:06 astubbs

We're not experiencing this issue internally and are backed up on other work. It's unlikely we'll get a chance to fix this. Can you submit a pull request?

sprsquish avatar Jun 28 '13 15:06 sprsquish

As in, you have spaces in your jenkins jobs, and don't have an issue? or you don't have spaces?

astubbs avatar Jun 28 '13 17:06 astubbs

As far as I can tell, we don't have spaces. Spaces are definitely an issue.

sprsquish avatar Jun 28 '13 17:06 sprsquish

Sure, I'll take a look. Should be a simple matter..

On 28 June 2013 13:43, Jeff Smick [email protected] wrote:

As far as I can tell, we don't have spaces. Spaces are definitely an issue.

— Reply to this email directly or view it on GitHubhttps://github.com/twitter/scrooge/issues/80#issuecomment-20202986 .

astubbs avatar Jun 28 '13 19:06 astubbs

same issue

celadaguido avatar Sep 24 '20 12:09 celadaguido

Gosh - blast from the past! I have no idea what I ended up doing about this 7 years ago :) Good luck!

astubbs avatar Sep 24 '20 13:09 astubbs

Just don't use spaces :P I have this issue when using https://github.com/pinterest/secor

celadaguido avatar Sep 24 '20 14:09 celadaguido