Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-17 Thread Jonas Smedegaard
-BEGIN PGP SIGNED MESSAGE- Hash: RIPEMD160 On Sat, May 16, 2009 at 06:20:56PM +0530, Vamsi Krishna Davuluri wrote: Hello! I finally figured what was going wrong with my script. Jonas, I have tried implementing your suggestions. using mktemp, X, and also a -m 077. The code doesnt

Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-16 Thread Vamsi Krishna Davuluri
Hello! I finally figured what was going wrong with my script. Jonas, I have tried implementing your suggestions. using mktemp, X, and also a -m 077. The code doesnt work for some reason that way. lpr gets me a failed job. Also that code is basically ported from this guide,

Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-14 Thread Andrés Ambrois
On Wednesday 13 May 2009 08:03:22 am Vamsi Krishna Davuluri wrote: Thanks. I have taken into account your suggestions and made another script. Thank you all for helping out! Here are a few other comments: #!/bin/bash -e # CUPS filter to process ODT files using abiword # $6 happens to be

[Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-13 Thread Sascha Silbe
On Wed, May 13, 2009 at 01:03:26PM +0530, Vamsi Krishna Davuluri wrote: Okay, so here's the latest dope. I hope you don't mind me pointing out a few oversights in your script publically. The main reason is that I want to remember others (e.g. GSoC students) to be careful about quoting - a

Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-13 Thread Jonas Smedegaard
-BEGIN PGP SIGNED MESSAGE- Hash: RIPEMD160 On Wed, May 13, 2009 at 10:25:36AM +0200, Sascha Silbe wrote: On Wed, May 13, 2009 at 01:03:26PM +0530, Vamsi Krishna Davuluri wrote: Okay, so here's the latest dope. I hope you don't mind me pointing out a few oversights in your script

Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-13 Thread Vamsi Krishna Davuluri
Thanks. I have taken into account your suggestions and made another script. This had been not a competition to beautify or verify the rigidity of the script, rather to see if opensuse accepted it. Which it still doesnt Something to do with lp user file create permissions. Though the script works

Re: [Sugar-devel] Quoting in shell scripts (was: Re: [sugar-devel] Recent fiddlings with Print Support)

2009-05-13 Thread Jonas Smedegaard
-BEGIN PGP SIGNED MESSAGE- Hash: RIPEMD160 On Wed, May 13, 2009 at 04:33:22PM +0530, Vamsi Krishna Davuluri wrote: #in case its not defined TMPDIR=/tmp Above means that you override if it was defined. Use this instead: TMPDIR=${TMPDIR:-/tmp} Or use mktemp which has same fallback (and