Csh broken by untested patch in latest aaa_base update

[dhnels]

The latest tumbleweed update of aaa_base package contains a completely untested change in /usr/etc/profile.d/alljava.csh that prevents csh startup and gives the following error message:
“set: Variable name must begin with a letter.”

Broken code:
set JAVA_TARGET alts -t java
set JAVA_TARGET realpath $JAVA_TARGET

should be:
set JAVA_TARGET=alts -t java
set JAVA_TARGET=realpath $JAVA_TARGET

This syntax error would have been caught by even the slightest QA effort. Obviously there was none.

EDIT: The forum software isn’t showing quote characters in the above code.

[dhnels]

Since I can’t edit my original post after a few minutes I’ll repost my change for code formatting here:

Broken code:
set JAVA_TARGET `alts -t java`
set JAVA_TARGET `realpath $JAVA_TARGET`

should be:
set JAVA_TARGET=`alts -t java`
set JAVA_TARGET=`realpath $JAVA_TARGET`

[nrickert]

Yes, this problem has been reported as Bug 1267423.

@dhnels

The forum software isn’t showing quote characters in the above code.

The forum software uses markdown which uses that quote as indicating preformatted text.