[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: debootstrap 1.0.93+nmu5



On Fri, 30 Mar 2018, Hideki Yamane wrote:
> On Fri, 30 Mar 2018 14:24:27 +0200
> Raphael Hertzog <hertzog@debian.org> wrote:
> > debootstrap can be run on a non-Debian system and should not assume that
> > apt-config is available. I would suggest to protect this code with
> > a check of the existence of apt-config.
> 
>  You're right. Then, how about this?
> 
> if (which apt-config > /dev/null); then

In most postinst snippets, it is used like this:

	if which apt-config >/dev/null 2>&1; then

I don't see the need for a sub-shell.

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Support Debian LTS: https://www.freexian.com/services/debian-lts.html
Learn to master Debian: https://debian-handbook.info/get/


Reply to: