Michael Prokop <bts@bts.grml.org>:
> Michael Prokop <mika@grml.org> added the comment:
> * Kai Wilke <bts@bts.grml.org> [20080928 12:06]:
> > Bericht über das Paket grml-etc-core-0.3.57 + patch.
> 
> Danke Kai. Frank, magst du das mal bitte reviewen? (Ich liege krank
> im Bett und schreib nur mal was mir auf den ersten Blick auffällt).

Eieiei... Gute Besserung!

> > hash -d doc=/usr/share/doc und hash -d log=/var/log wurden umbenannt in
> > sdoc und vlog. doc kann ein Unterverzeichnis irgendwo im Tree sein.
> > Bsp.:
> > cd /home/USER
> > doc
> > pwd
> > /home/USER/doc
> > log ist ein zsh builtin Befehl.
> 
> Sorry, helft mir weiter:
> 
> mika@fuchsiatard ~ % cd ~doc
> mika@fuchsiatard ~doc % pwd
> /usr/share/doc
> 
> Wieso soll das weg?

Nun, das sind Probleme mit denen Leute die auf AUTO_CD stehen zu
kämpfen haben. Da kannst du nicht Folgendes machen:

    % cd ~
    % ls
    doc/
    % doc
    % pwd
    /usr/share/doc

(nicht getestet, einfach hingeschrieben. cdable_vars wird wohl zu
auto_cd zusätzlich benötigt, für so einen Workflow.)

Ich tendiere allerdings *stark* dazu, daß die Hash Namen nicht
geändert werden. Ich denke, das die momentanen Namen sich schon recht
gut eingebürgert haben.

Zumindest bei 'doc' bin ich da kaum umzustimmen. Wenn das persönlich
stört, ändere man das in seinem persönlichen Zusatzsetup.

Bei 'log' könnte man darüber nachdenken, ob man eine Änderung
vornimmt - da wir in oben beschriebenen Workflow tatsächlich ein
builtin weg-maskieren würden.

Allerdings ist das auch da wieder eine Frage des persönlichen
Workflows. Ich würde sowas gar nicht spüren. Und da wir, AFAIR,
cdable_vars gar nicht setzen sehe ich nicht so großen Handlungsbedarf.
Sorry.

> > Vielleicht gefällt es euch. Man hat eine genauere Anzeige über die
> > Prozesse.
> > --------------------snip---------------------
> > #zstyle ':completion:*:processes'       command 'ps -au$USER'
> > zstyle ':completion:*:processes' command 'ps --forest -u $USER -o pid,cmd'
> > --------------------snap---------------------
> 
> Pfuh, das ist aber schon *sehr* anders. ;) Frank, was meinst du
> dazu?

Nun ich habe:
  zstyle ':completion:*:processes' command 'ps --forest -A -o pid,user,cmd'

Evtl sollte man da verschiedene Styles für normale Nutzer und für root
machen. As in: Nutzer-Prozesse vs. alle-Prozesse.


Nun nochmal der Patch im Einzelnen (Bemerkungen unter bzw. in den
Ausschnitten):

[snip]
-if [[ -n $ZSH_PROFILE_RC ]] ; then
+if [[ $ZSH_PROFILE_RC -gt 0 ]] ; then

  ACK.

-if [[ -n "$BATTERY" ]] ; then
+if [[ $BATTERY -gt 0 ]] ; then

  ACK.

-is4 && [[ -z $NOPRECMD ]] && precmd () {
-    [[ -n $NOPRECMD ]] && return 0
+is4 && [[ $NOPRECMD -lt 1 ]] && precmd () {

  [[ $NOPRECMD -eq 0 ]], bitte.

+    [[ $NOPRECMD -gt 0 ]] && return 0

  ACK.

-is4 && [[ -z $NOPRECMD ]] && \
+is4 && [[ $NOPRECMD -lt 1 ]] && \

  auch hier: -eq 0

-    [[ -n $NOPRECMD ]] && return 0
+    [[ $NOPRECMD -gt 0 ]] && return 0

  ACK.

-    if [[ -n $GRMLPROMPT ]] ; then
+    if [[ $GRMLPROMPT -gt 0 ]] ; then

  ACK.

-    if [[ -z "$NOMENU" ]] ; then
+    if [[ "$NOMENU" -lt 1 ]] ; then

  -eq 0

-    if [[ -n "$NOCOR" ]] ; then
+    if [[ "$NOCOR" -gt 0 ]] ; then

  ACK.

[snap]

Diese hatten Kai und ich privat besprochen. Sollten mit nach Umsetzung
der Anmerkungen in Ordnung sein.

[snip]
@@ -431,9 +431,11 @@
     [[ -d /etc/zsh/completion.d ]] && fpath=( $fpath /etc/zsh/completion.d )
     if [[ -d /etc/zsh/functions.d ]] ; then
         fpath+=( /etc/zsh/functions.d )
-        for func in /etc/zsh/functions.d/[^_]*[^~] ; do
+        setopt nonomatch
+        for func in `ls -C /etc/zsh/functions.d/[^_]*[^~] 2>&/dev/null` ; do
             zrcautoload -U ${func:t}
         done
+        unsetopt nonomatch

     fi
 fi
[snap]

No way. Das ist aus so vielen Gründen nicht akzeptabel, das ich
garnicht erst anfange. Ich weiß auch nicht, was du da mit nonomatch
erreichen willst. Ich nehme an, du willst null_glob. Erklärung bitte.

Dieser unsägliche ls-Hack kommt auf keinen Fall in Frage:
  <http://bewatermyfriend.org/posts/2007/02-08.12-18-45-rants.html>

[snip]
-        ap  zprof
+        #ap  zprof
[snap]

...erm, Grund?

[snip]
+battery() {
+if [[ $BATTERY -gt 0 ]] ; then
+    PERCENT="${${"$(acpi 2>/dev/null)"}/(#b)[[:space:]]##Battery <->: [^0-9]##, (<->)%*/${match[1]}}"
+    if [[ -z "$PERCENT" ]] ; then
+        PERCENT='acpi not present'
+    else
+        if [[ "$PERCENT" -lt 20 ]] ; then
+            PERCENT="warning: ${PERCENT}%%"
+        else
+            PERCENT="${PERCENT}%%"
         fi
     fi
 fi
+}
[snap]
 
Das schaut okay aus.
Besonders, das PERCENT so geupdated wird ist mir sehr willkommen.

[snip]
-    if [[ -z $DONTSETRPROMPT ]] ; then
-        if [[ -n $BATTERY ]] ; then
+    if [[ $DONTSETRPROMPT -lt 1 ]] ; then

+        if [[ $BATTERY -gt 0 ]] ; then
+            # update BATTERY information
+            battery
[snap]

Auch hier -eq 0, bitte.
Ansonsten okay.


[snip]
-hash -d doc=/usr/share/doc
+hash -d sdoc=/usr/share/doc
-hash -d log=/var/log
+hash -d vlog=/var/log
[snap]

Für diese siehe oben; eher nicht.

[snip]
-  "NOMENU=1   zsh" => do not use menu completion (note: use strg-d for completion instead!)
+  "NOMENU=1   zsh" => do not use auto menu completion (note: use ctrl-d for completion instead!)
[snap]

Eigentlich war folgende Zeile so oder so schon lang genug, aber okay:

[snip]
-    zstyle ':completion:*:processes'       command 'ps -au$USER'
+    #zstyle ':completion:*:processes'       command 'ps -au$USER'
+    zstyle ':completion:*:processes' command 'ps --forest -u $USER -o pid,cmd'
[snap]

Auch hier, siehe oben. Persönlich mag ich die Baum-artige Darstellung
auch. Allerdings muss dann wenigstens die Einfärbung angepasst werden.


-- 
In protocol design, perfection has been reached not when there is
nothing left to add, but when there is nothing left to take away.
                                                  -- RFC 1925