Imported From: https://issues.scala-lang.org/browse/SI-4771?orig=1
Reporter: Percy Liang (pliang)

Jonathan Clark (jhclark) said:
I also just stumbled upon this one and replicated the same issue, running a script that was not pre-compiled.

Scala 2.9.0-final. Ubuntu 10.04 LTS.

Jonathan Clark (jhclark) said:
Percy: I guess it figures the NLP researchers would discover anything wrong with stdin first. :-)

Jonathan Clark (jhclark) said:
I believe I might have found the cause, when exploring a seemingly unrelated issue (catching error codes from child bash processes). I believe the bug is at ProcessBuilderImpl.scala:72 -- in Simple.run(p: ProcessIO). Basically, scala always connects stdin to child processes, no matter what. However, piping /dev/null or the like to a child process (opening its input stream in any way), can cause it to behave differently. In my unrelated case, I cannot get bash to return proper return codes when it believes it is reading a script from stdin. My hypothesis is that the scala interpreter might have some similar magic like the bash interpreter. Any comments/suggestions from someone who knows the scala codebase a bit better?

Matthew Farwell (mfarwell) said:
This is fixed in trunk. In investigating Jenkins reporting success when Scala fails, this is a different aspect of the same problem.

The fix is simple, in $SCALA_HOME/bin/scala, line 25, change

function onExit() {
  if [[ "$saved_stty" != "" ]]; then
    restoreSttySettings
    exit $scala_exit_status
  fi
}

move the exit outside the test for $saved_stty.

function onExit() {
  if [[ "$saved_stty" != "" ]]; then
    restoreSttySettings
  fi
  exit $scala_exit_status
}

This has already been done in trunk, and this works with the latest build.

Matthew Farwell (mfarwell) said (edited on Mar 21, 2012 7:46:03 PM UTC):
I think this should be backported to 2.9.x because it seems like the script will swallow error codes in any environment where the script is run by a third party, such as jenkins, ant. I haven't tested maven.

@paulp said:
Pull request would do wonders.

Matthew Farwell (mfarwell) said:
Et voilà.

scala/scala#309

Sorry, should have done the pull request before.

@paulp said:
6f43d81942