[dev] fixed valigns and such

Chuck Hagenbuch chuck at horde.org
Tue Nov 23 09:11:43 PST 2004


Quoting "Derek P. Moore" <derekm at hackunix.org>:

> I had to make that change to STDIN to get that script to work on my
> installation.  I wasn't sure if it was "right" or not, but it's what
> made it work for me.  I hadn't used PHP STDIN in years, and last I
> remember STDIN was the way to go.  But when it came time to use that
> script, I was getting errors because of STDIN.  I browsed the PHP
> manual, and learned of "php://stdin" or whatever, and that's what made
> it work for me.

Because you're using a version of PHP that is older than the documented 
minimum
version required for Horde 3.0+.

> I just thought I'd throw that patch in there to see what you said about
> it.  I was curious as to STDIN vs. php://stdin.

It's *really* bad practice to put unrelated changes into a patch, and 
even worse
not to mention them. It's how people can get paranoid about people trying to
slip trojans into patches, etc. Please don't do it again.

-chuck

-- 
"But she goes not abroad in search of monsters to destroy." - John 
Quincy Adams


More information about the dev mailing list