Plumbum LocalModule does not respect getattr default value

[idanz]

e.g.

> getattr(sys.modules['plumbum.cmd'], 'x', None)
---------------------------------------------------------------------------
CommandNotFound                           Traceback (most recent call last)
<ipython-input-13-f6036372bef1> in <module>()
----> 1 getattr(sys.modules['plumbum.cmd'], 'x', None)

/usr/local/lib/python2.7/dist-packages/plumbum/machines/local.pyc in __getitem__(self, cmd)
    195             else:
    196                 # search for command
--> 197                 return LocalCommand(self.which(cmd))
    198         else:
    199             raise TypeError("cmd must not be a RemotePath: %r" % (cmd,))

/usr/local/lib/python2.7/dist-packages/plumbum/machines/local.pyc in which(cls, progname)
    166             if path:
    167                 return path
--> 168         raise CommandNotFound(progname, list(cls.env.path))
    169
    170     def path(self, *parts):

CommandNotFound: ('x', [<LocalPath /usr/local/sbin>, <LocalPath /usr/local/bin>, <LocalPath /usr/sbin>, <LocalPath /usr/bin>, <LocalPath /sbin>, <LocalPath /bin>, <LocalPath /usr/games>, <LocalPath /opt/vagrant_ruby/bin>, <LocalPath /home/vagrant/qlika-dev>])

This can cause serious issues, for example, it breaks pickle.whichmodule.

To fix, plumbum should raise AttributeError if getitem fails when called from getattr

[tomerfiliba]

thanks :)