[Konsole-devel] Review Request 120763: Add new signal : backgroundProcessPwdChanged

[Frank Reininghaus]

> On Okt. 23, 2014, 9:58 nachm., Frank Reininghaus wrote:
> > Cool, thanks for looking into this! I think that many users will appreciate any improvements in this area.
> > 
> > I'm just wondering, do we really need two different signals which do quite similar things? I guess the 'old' signal would not be used any more, so maybe one could simply change the behavior of currentDirectoryChanged, such that it does what your new signal would do, and not introduce a new signal at all?
> 
> Arjun AK wrote:
>     The new signal is only emitted when the PWD of the shell changes and not when it is changed by the running 'foreground' process. Wouldn't changing the behaviour break other apps?

There are no other apps that use this signal. At least http://lxr.kde.org/ident?_i=currentDirectoryChanged&_remember=1 does not show any. Moreover, even if there were other apps, I don't think that there is a common use case where staying informed about temporary directory changes caused by "man ls", "git status", etc., is what the user wants.


- Frank


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120763/#review69053
-----------------------------------------------------------


On Okt. 23, 2014, 8:58 nachm., Arjun AK wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120763/
> -----------------------------------------------------------
> 
> (Updated Okt. 23, 2014, 8:58 nachm.)
> 
> 
> Review request for Konsole, Emmanuel Pescosta and Frank Reininghaus.
> 
> 
> Bugs: 305085
>     http://bugs.kde.org/show_bug.cgi?id=305085
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> Listening to the currentDirectoryChanged() can cause dolphin to act weird when you have programs running on the konsole part. for instance:
> 
> 1) `$ man ls` -> Dolphin temporarily switches to the manpages directory
> 2) `$ man ls`. Now quickly press q and CTRL D, you get stuck in the manpages directory.
> 3) `(cd /;sleep .5; cd /tmp; sleep .5)`
> 4) When you have scripts running(like kdesrc-build), dolphin keeps on changing the directories
> 
> By adding a new signal, which is only emitted when the PWD of the shell is changed, these [bugs](https://bugs.kde.org/show_bug.cgi?id=305085) can be fixed.
> 
> 
> Diffs
> -----
> 
>   src/Part.h 646e3aa 
>   src/Part.cpp cb556cc 
>   src/Session.h 6cd283a 
>   src/Session.cpp cf70c09 
>   src/SessionController.h 4a7d094 
>   src/SessionController.cpp cc9db5f 
> 
> Diff: https://git.reviewboard.kde.org/r/120763/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun AK
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20141024/625387dd/attachment.html>