[Bug 664699] xorg-x11-fonts-update-dirs is not a /bin/sh script

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=664699

Ralf Corsepius <rc040203 at freenet.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?(rc040203 at freenet. |
                   |de)                         |

--- Comment #2 from Ralf Corsepius <rc040203 at freenet.de> 2010-12-21 10:44:56 EST ---
(In reply to comment #1)
> What do you think about the patch in attachment 469993 [details] ?

OK, that's the classic subshell pushd/popd approach - I don't any reason why it
shouldn't work. It's advantage is it being usable should once a loop be
introduced around the pushd/popd, it's downside it the subshell having its
price.

An alternative would be the "save pwd" approach.
sav_pwd=`pwd`
cd <somewhere>
do-something
cd ${sav_pwd}
This would avoid the subshell and therefore is somewhat less expensive, but it
would not be suitable for "loops".

AFAIS, ATM the latter alternative would suffice.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.