Comment 4 for bug 398974

Revision history for this message
Iain Lane (laney) wrote :

I've just come across this, although the code seems to have evolved a bit since this report was filed. There is no reason for /usr/sbin to be in the path of the user. It is wrong for pbuilder-dist to search for the builder using the user's PATH settings as it is going to be launched under sudo anyway.

Why does this check need to exist? The relevant lines are 80-86:

        for file in os.environ['PATH'].split(':'):
            if os.path.exists(os.path.join(file, builder)):
                builder = ''
                break
        if builder:
            print 'Error: Could not find "%s".' % builder
            sys.exit(1)