I'm with @jvazquez-r7 on splitting it out. Just don't use git revert to do
it that'll get real confusing real fast. Let me take a stab at it.
 On Jan 2, 2014 1:24 PM, "Juan Vazquez" <notifications@github.com> wrote:

> In lib/msf/core/post/windows/shadowcopy.rb:
>
> > @@ -198,7 +198,7 @@ def wmicexec(wmiccmd)
> >      tmpout = ''
> >      session.response_timeout=120
> >      begin
> > -      tmp = session.fs.file.expand_path("%TEMP%")
> > +      tmp = session.sys.config.getenv('TEMP')
>
> nevermind... this pull request is adding getenvs :) and allowing getenv to
> get an String, sorry abou that, should read description before reading
> code, and don't merge github review and with my local code while review :P
> sorry for annoying.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rapid7/metasploit-framework/pull/2782/files#r8620473>
> .
>