Last modified: 2014-04-11 23:59:22 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58597, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56597 - hhvm unable to open php://stdin
hhvm unable to open php://stdin
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.23.0
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
https://github.com/facebook/hhvm/issu...
: hhvm, upstream
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-05 00:48 UTC by Chad H.
Modified: 2014-04-11 23:59 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Chad H. 2013-11-05 00:48:07 UTC
HipHop Warning: Unable to open specified file: php://stdin (mode r) in /usr/local/apache/common/php-1.23wmf2/includes/GlobalFunctions.php on line 2816
Comment 1 Chad H. 2013-11-05 22:21:58 UTC
This was broken as a result of Gerrit change #91115. This is hhvm's fault though, so I've filed upstream: https://github.com/facebook/hhvm/issues/1247
Comment 2 Gerrit Notification Bot 2013-11-06 23:32:49 UTC
Change 94065 had a related patch set uploaded by Chad:
Work around HHVM bug with stdin/stdout in proc_open()

https://gerrit.wikimedia.org/r/94065
Comment 3 Gerrit Notification Bot 2013-11-08 04:52:49 UTC
Change 94065 merged by jenkins-bot:
Work around HHVM bug with stdin/stdout in proc_open()

https://gerrit.wikimedia.org/r/94065
Comment 4 Umherirrender 2013-11-09 09:20:45 UTC
Was successfully merged
Comment 5 Chad H. 2013-11-09 19:08:56 UTC
It's a workaround, not a solution. Let's leave this open until the upstream issue is resolved.

(Lowering priority and severity a bit since we've got a workaround now)
Comment 6 Andre Klapper 2014-01-10 00:38:18 UTC
[Using keyword instead of tracking bug for HipHop issues as requested in bug 40926 comment 5. Filter bugmail on this message.]
Comment 7 Aaron Schulz 2014-04-09 18:27:30 UTC
Looks like this was fixed upstream?  Can the work around be reverted and this patch closed?
Comment 8 Gerrit Notification Bot 2014-04-11 18:41:08 UTC
Change 125435 had a related patch set uploaded by Aaron Schulz:
Revert "Work around HHVM bug with stdin/stdout in proc_open()"

https://gerrit.wikimedia.org/r/125435
Comment 9 Gerrit Notification Bot 2014-04-11 23:59:22 UTC
Change 125435 merged by jenkins-bot:
Revert "Work around HHVM bug with stdin/stdout in proc_open()"

https://gerrit.wikimedia.org/r/125435

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links