Abiword.scm command line bug

[Rich]

Hi uggla

SAVE_ARGS=$*

Not sure if it matters here, but I think I read somewhere that $@ is more commonly used because it preserves spaces
where $* doesn't in some situations.

[uggla]

I'll fix the script for already quoted paths.

[uggla]

Here's a much better script:

Code: [Select]

#!/bin/sh

for p do
case ${p:0:1} in
-) params="${params} ${p}";;
/) params="${params} \"${p}\"";;
*) params="${params} \"$PWD/${p}\"";;
esac
done

cd /apps/abiword
export LD_LIBRARY_PATH="local/lib"
export PATH="local/bin:$PATH"

exec /apps/abiword/abiword ${params}


[uggla]

Sorry, here's a script that actually works 

Code: [Select]

#!/bin/sh

for arg do
case ${arg:0:1} in
-) SAVE_ARGS="${SAVE_ARGS}${arg}|";;
/) SAVE_ARGS="${SAVE_ARGS}${arg}|";;
*) SAVE_ARGS="${SAVE_ARGS}$PWD/${arg}|";;
esac
done
IFS='|'

cd /apps/abiword
export LD_LIBRARY_PATH="local/lib"
export PATH="local/bin:$PATH"

/apps/abiword/abiword ${SAVE_ARGS}

[Jason W]

Hi, I am not ignoring this, time is just tight now, I will visit it soon.

[uggla]

Still too busy? 

[Jason W]

I will look at that last script tonight and  possibly work it in.

[Jason W]

Works like a charm with multiple command line arguments, like "/apps/abiword/bin/abiword -u tc /home/tc/file".

I am uploading the fix now, and I will use this approach with the other converted scms.

[uggla]

Great! 

[uggla]

Xarchiver would be much more useful (to me) if it were updated with this wrapper.