UIMA
  1. UIMA
  2. UIMA-1883

UIMA shell scripts (e.g. cvd.sh) do not support space in filenames

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3.1SDK, 2.3.1AS
    • Component/s: None
    • Labels:
      None
    • Environment:

      Ubuntu Linux (Bash), OpenJDK 1.6

      Description

      Some UIMA shell scripts do not handle parameters to files that have spaces in their name. For example:

      cvd.sh -desc /some/path with spaces/desc.xml

      This command fails as the parameter file path with spaces is not escaped.

      Changing $* to "$@" in shell scripts fixed the issue.

        Activity

        Hide
        Marshall Schor added a comment -

        A brief investigation seems to indicate this would be OK for bash (bourne) shell and for korn shell, but wouldn't work for c-shell. Is there a more universal "fix" for this? Is there a bit of universal script we could use to verify this would work in the particular shell being used, and if not, "fall back" on the other approach?

        Show
        Marshall Schor added a comment - A brief investigation seems to indicate this would be OK for bash (bourne) shell and for korn shell, but wouldn't work for c-shell. Is there a more universal "fix" for this? Is there a bit of universal script we could use to verify this would work in the particular shell being used, and if not, "fall back" on the other approach?
        Hide
        Marshall Schor added a comment -

        I'm inclined to go ahead and implement this change, because, I think, overall, it will help our users. Any objections?

        Show
        Marshall Schor added a comment - I'm inclined to go ahead and implement this change, because, I think, overall, it will help our users. Any objections?

          People

          • Assignee:
            Unassigned
            Reporter:
            Bevan Koopman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development