• 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

Stig Kleppe-Jørgensen created issue - 2009-10-12 06:24

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.

Stig Kleppe-Jørgensen made changes - 2009-10-27 05:08

Resolution		New: Fixed [ 1 ]
Status	Original: Open [ 1 ]	New: Resolved [ 5 ]

Andrew Bayer made changes - 2011-02-10 19:07

Status

Original: Resolved [ 5 ]

New: Closed [ 6 ]

R. Tyler Croy made changes - 2016-07-25 23:58

Workflow

Original: JNJira [ 134709 ]

New: JNJira + In-Review [ 203013 ]