Danek Duvall wrote:
Brock Pytlik wrote:

http://cr.opensolaris.org/~bpytlik/ips-12873-v1/

Looks good; thanks for finding that.
Thanks for taking a look, I'll put this back.
Looks like there may be some related problems with the GUI, too, as it uses
putenv.  I also see ldtp using it.  I don't know if any of these are going
to have actual repercussions, though.

Danek
When I grepped for putenv, I didn't see any in our code, except for ldtp. I gave that the benefit of the doubt that they're using it correctly. From a quick look at the gui code, it's not going to hit this problem I don't think. The possible exception would be setting the homedir in packagemanager.py, but I assume if that was actually not working, it would be clearly broken on use. I believe it's being set there for external processes like gconf. I'll defer to the team working on packagemanager to make sure their use of putenv is doing what they want.

Brock
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to