Just a quick drive-by review: Why do you need to set $LANG in the first place if the "outside" apport has never needed it? If there's an encoding error, we should fix that instead; do you have a stack trace?
en_US.UTF-8 isn't guaranteed to be available; C.UTF-8 ought to be available everywhere, so if we absolutely must hardcode a locale, it should be C.UTF-8. But let's try to avoid that.
+ if path[-1] != "command":
+ continue
What's that magic "command" string here?
(stylistic nitpick: single quotes everywhere, please; but I can do that on merging into trunk)
+ os.environ['HOME'] = '/root'
I believe that ought to be '/'? it's not necessarily the case that /root exists.
the python3-lxc dependency is quite heavy; is that always save to use within the rather minimal context of apport (as being called by the kernel -- i. e. no environment, no association to any user, always has root privileges), and will never block unnecessarily? Would calling lxc-attach directly be a lighter alternative here?
Just a quick drive-by review: Why do you need to set $LANG in the first place if the "outside" apport has never needed it? If there's an encoding error, we should fix that instead; do you have a stack trace?
en_US.UTF-8 isn't guaranteed to be available; C.UTF-8 ought to be available everywhere, so if we absolutely must hardcode a locale, it should be C.UTF-8. But let's try to avoid that.
+ if path[-1] != "command":
+ continue
What's that magic "command" string here?
(stylistic nitpick: single quotes everywhere, please; but I can do that on merging into trunk)
+ os.environ['HOME'] = '/root'
I believe that ought to be '/'? it's not necessarily the case that /root exists.
the python3-lxc dependency is quite heavy; is that always save to use within the rather minimal context of apport (as being called by the kernel -- i. e. no environment, no association to any user, always has root privileges), and will never block unnecessarily? Would calling lxc-attach directly be a lighter alternative here?
Thanks!