[issue616] grml-etc-core: Adding trainling spaces to various power completions.

[Frank Terbeck]

Frank Terbeck <ft at grml.org> added the comment:

Michael Prokop <bts at bts.grml.org>:
> Frank Terbeck <ft at bewatermyfriend.org> [20090214 20:54]:
> > Alexander Steinböck <bts at bts.grml.org>:
> > > > And what I said about "take the unneeded expansions out": scratch
> > > > that. It's cleaner if that's done in a separate patch. Either
> > > > *before* (which is probably preferable) or *after* the patch
> > > > discussed in this bug. I'm fine either way, just don't squash it
> > > > into one patch.
> > > That does still apply, I guess.
> 
> > Yes.
> 
> Alright, then I'll apply and push
> http://bts.grml.org/grml/file195/0001-zshrc-Add-a-trailing-space-for-several-power-comple.patch

Actually, I think Alexander meant if my earlier remark still applies.
IIRC, I made a review with comments that are still standing out to be
corrected.

So, unless you're in a hurry, I'd say we give Alexander the time he
needs to come up with a cleaned up patch.

Regards, Frank

----------
files: 0001-zshrc-Add-a-trailing-space-for-several-power-comple.patch, 0002-zshrc-Disordered-argument-list-in-MPlayer-s-power-c.patch
messages: 1805, 1806, 1807, 1808, 1809, 1867, 1869, 1870, 1871, 1872, 1873, 1878, 1879, 1882, 1885
nosy: ft, mika, z3ttacht
priority: feature
status: in-progress
title: grml-etc-core: Adding trainling spaces to various power completions.
topic: grml-etc-core, just ideas, zsh

_____________________________________
GRML issue tracker <bts at bts.grml.org>
<http://bts.grml.org/grml/issue616>
_____________________________________