[Bash-completion-devel] [Bash-completion-commits] [SCM] bash-completion branch, master, updated. 59977e4bf8ad30ab85c0ee2d6947c079a3df0579

[Ville Skyttä]

On Tuesday 16 February 2010, Crestez Dan Leonard wrote:
> On Mon, 2010-02-15 at 16:56 +0000, Ville Skyttä wrote:
> > The following commit has been merged in the master branch:
> > commit 59977e4bf8ad30ab85c0ee2d6947c079a3df0579
> > Author: Ville Skyttä <ville.skytta at iki.fi>
> > Date:   Mon Feb 15 18:56:09 2010 +0200
> >
> >     Sync before unsetting var to avoid error message.
> 
> What error message? I didn't get anything.

I got this every time (it doesn't cause any actual test suite failures or 
unresolvedness though):

$ ./runCompletion mount.exp
[...]
Running ./completion/mount.exp ...
ERROR executing bash command "unset var"
[...]

$ grep -B 1 -A 1 "unset var" log/completion.log 
PASS: Testing internal __linux_fstab_unescape function for mount
unset var
/@unset var
/@echo $?
ERROR executing bash command "unset var"0
/@^C

> Maybe there are still
> problems with sync_after_int on different machines.

Maybe, but in before my change there was no sync_after_int at the position 
where the error occurred so I suppose it's not its fault if it is not even 
called...  Anyway I'm not at all sure that adding it is the correct fix but 
it's the first thing I tried and fixes the error for me, and I suppose it 
doesn't hurt elsewhere.