[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