Re: Re: Waiting for you?
| From: | Christian Weiske | Date: | Tue, 08 Feb 2011 09:13:04 +0000 |
| Subject: | Re: Re: Waiting for you? | ||
| References: | 1 2 3 4 5 6 7 8 9 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-54078@lists.php.net to get a copy of this message | ||
Hello Afshin,
> > > Thanks for your response.
> > > Should I fix this problems and upload my patch again?
> > Yes, that's the best way.
> Ok, sure.
>
> http://pear.php.net/bugs/bug.php?id=18157&edit=12&patch=fix-root-home&revision=1297159694
@@ -515,9 +515,13 @@
//env not set, so try the default directory
$strUser = $this->getUserName();
if ($strUser !== null) {
+ if($strUser == "root"){
+ $strHome = '/root';
+ } else {
$strHome = '/home/' . $strUser;
}
}
+ }
} else if ($this->sys == SYS_MAC) {
if (isset($_ENV['HOME'])) {
//environment variable set
Did you really run the codesniffer on the code after you changed it? I
doubt that.
--
Regards/Mit freundlichen Grüßen
Christian Weiske
-= Geeking around in the name of science since 1982 =-