%D did work only with main process - for every subprocess it just
displayed "-". With the patch "-" is displayed only in processes for
which CWD can't be read (e.g. root processes).
Details
Details
- Reviewers
hindenburg - Group Reviewers
Konsole - Commits
- R319:79c31d359f0e: Show valid subprocess CWD for %D tab title format
- Use %D in tab title format
- Run vim
- (optionally) run :cd /usr in vim
Actual result: "-" is shown in the title instead of %D
Expected result: valid CWD is shown in the title instead of %D
Diff Detail
Diff Detail
- Repository
- R319 Konsole
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Looks fine - thanks - I notice this seems to fix %u which hasn't worked for a while it seems
Comment Actions
Good that it fixed %u the old behavior on su/sudo was to keep the %d from the parent process and now it just display "-" this particular behavior should be reversed ... i personally prefer displaying the parent %d instead of - plus i get use to it... (i am going to make a patch for that, let me know if it's worth posting it here ;))
Comment Actions
Maybe just implement it using another letter? I guess both implementations have its fans