• Type: Bug
• Resolution: Fixed
• Priority: Major
• Component/s: cli
• Labels:

  None
• Environment:

  Platform: All, OS: All

1. fix-stdin-for-groovy-scripts.patch

   1 kB

   2009-10-12 12:07

[JENKINS-4636] Groovy CLI fails when trying to read from stdin

Collapse comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 06:29

Stig Kleppe-Jørgensen added a comment - 2009-10-12 06:29

I guess args4j does not handle just - as an option, so I have changed - to =. Not sure whether that is the
best choice, but it works.

Also I needed to change the name of the class to something static and not the name of the input, as "-"
does not make for a good groovy class name. Neither does some-script.groovy.

Expand comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 06:29

Stig Kleppe-Jørgensen added a comment - 2009-10-12 06:29 I guess args4j does not handle just - as an option, so I have changed - to =. Not sure whether that is the best choice, but it works. Also I needed to change the name of the class to something static and not the name of the input, as "-" does not make for a good groovy class name. Neither does some-script.groovy.

Collapse comment: dvrzalik added a comment - 2009-10-12 10:40

dvrzalik added a comment - 2009-10-12 10:40

Wrong subcomponent?

Expand comment: dvrzalik added a comment - 2009-10-12 10:40

dvrzalik added a comment - 2009-10-12 10:40 Wrong subcomponent?

Collapse comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:04

Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:04

Duh...yes, of course it is cli.

Expand comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:04

Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:04 Duh...yes, of course it is cli.

Collapse comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:07

Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:07

Created an attachment (id=945)
Change - to = as the option for stdin and use "RemoteClass" as the script's class name

Expand comment: Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:07

Stig Kleppe-Jørgensen added a comment - 2009-10-12 12:07 Created an attachment (id=945) Change - to = as the option for stdin and use "RemoteClass" as the script's class name

Collapse comment: Stig Kleppe-Jørgensen added a comment - 2009-10-27 05:08

Stig Kleppe-Jørgensen added a comment - 2009-10-27 05:08

Fix committed on revision 23228.

Expand comment: Stig Kleppe-Jørgensen added a comment - 2009-10-27 05:08

Stig Kleppe-Jørgensen added a comment - 2009-10-27 05:08 Fix committed on revision 23228.