Re: compinit (was: Re: #compdef -k menu-select glitch in 3.1.9)

["Bart Schaefer" <schaefer@xxxxxxxxxxxxxxxxxxxxxxx>]

On Jun 19, 12:36pm, Sven Wischnowsky wrote:
} Subject: compinit (was: Re: #compdef -k menu-select glitch in 3.1.9)
}
} > (1) Ask whether to use the completion system.
} > (2) Silently ignore all directories and files that fail.
} > (3) Silently accept all directories and files in $fpath.
} 
} How's the patch below? (Not committed until I get a positive reply.)
} 
} It does the tests you mentioned and makes compinit accept three new
} options. The patterns used to find `insecure' directories and files
} are:
} 
}    _i_wdirs=( ${^fpath}(Nf:g+w:,f:o+w:,^u0u${EUID}) )
}    _i_wfiles=( ${^~fpath:/.}/^([^_]*|*~|*.zwc)(N^u0u${EUID}) )

Argh.  The one problem with this is RedHat linux's penchant for creating
groups that contain exactly one user.  So it's actually safe to have a
group writable directory on RedHat as long as the name of the group is
the same as the name of the user.  This has caused problems for some
other security-conscious packages as well, notably procmail.

I think you should commit the patch as is, and I'll think some more about
the right way to address this.  It might have to be a configure-time thing.
 
} The options:
} 
} -i: silently ignore `insecure' directories and files
} -u: silently use `insecure' directories and files
} -C: the same as in 11440, i.e.: make compinit not try to rebuild the
}     dumpfile if it exists (small optimisation for people who don't
}     fiddle with new completion functions that much)
} 
} This means that `ask whether...' is the default, but that's easy to
} change, of course.
 
No, I think that's fine.

} The patch also contains the test to avoid using the same filename more 
} than once and it removes compconf.

If the former means what I think it means, then thanks very much.

-- 
Bart Schaefer                                 Brass Lantern Enterprises
http://www.well.com/user/barts              http://www.brasslantern.com

Zsh: http://www.zsh.org | PHPerl Project: http://phperl.sourceforge.net