Last modified: 2014-10-20 00:02:47 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 T68023, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66023 - Importing does not work with HHVM
Importing does not work with HHVM
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Export/Import (Other open bugs)
1.23.0
All All
: High major (vote)
: ---
Assigned To: Brett Simmers
: hhvm
: 68261 70658 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-02 12:32 UTC by Adrian Lang
Modified: 2014-10-20 00:02 UTC (History)
11 users (show)

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


Attachments

Description Adrian Lang 2014-06-02 12:32:14 UTC
See https://github.com/facebook/hhvm/issues/2829.
Comment 1 Max Semenik 2014-07-19 04:17:12 UTC
*** Bug 68261 has been marked as a duplicate of this bug. ***
Comment 2 Brett Simmers 2014-07-25 19:27:25 UTC
This is being fixed by https://github.com/facebook/hhvm/pull/3249
Comment 3 MWJames 2014-07-25 19:34:09 UTC
When do we expect to land this in the Travis hhvm-nightly? Our tests [0] running on HipHop VM 3.3.0-dev+2014.07.23 (rel) still fail for the import part!

[0] https://travis-ci.org/SemanticMediaWiki/SemanticMediaWiki/jobs/30805643
Comment 4 Brett Simmers 2014-07-25 20:05:21 UTC
I'm debugging an issue with it right now. I'm hoping to have it cleaned up and committed by Monday, so it should be in the next nightly after that.
Comment 5 MWJames 2014-08-10 11:57:29 UTC
(In reply to Brett Simmers from comment #4)
> I'm debugging an issue with it right now. I'm hoping to have it cleaned up
> and committed by Monday, so it should be in the next nightly after that.

[0] uses HipHop VM 3.3.0-dev+2014.08.10 (rel)
Compiler: heads/master-0-g2060b3a6a9f59cd5bf7bf5023e64ca9e793d04e3 and allowed the remaining failed import tests to run successfully on MW master / SQLite / PHPUnit 4.1.5.

[0] https://travis-ci.org/SemanticMediaWiki/SemanticMediaWiki/jobs/32143305
Comment 6 This, that and the other (TTO) 2014-09-12 09:52:09 UTC
*** Bug 70658 has been marked as a duplicate of this bug. ***
Comment 7 This, that and the other (TTO) 2014-09-12 09:57:08 UTC
The HHVM patch was merged some weeks ago. Apparently WMF is using HHVM version "3.3.0-dev (srv)", which is pretty meaningless as version numbers go. It appears to me that if/when we upgrade to a "newer" HHVM 3.3.0, we'll get the benefit of this patch.
Comment 8 dan 2014-09-13 01:45:54 UTC
via a few emails, giuseppe pointed out that the current wm hhvm:

HipHop VM 3.3.0-dev (rel)
Compiler: heads/master-0-g2e297ed662d6cec9368ab8bd527ac06eaeef18cc
Repo schema: 4cd91499a6638cdf8f21e85093dc7dab65e909e0
Extension API: 20140727

contains these patches:
https://github.com/wikimedia/operations-debs-hhvm/blob/master/debian/patches/series

which contain the patch mentioned above:
https://bugzilla.wikimedia.org/show_bug.cgi?id=66023#c2

specifically in this patch:
https://github.com/wikimedia/operations-debs-hhvm/blob/master/debian/patches/Support-stream-wrappers-in-XML-parser-extensions-add-external-entity-loader.patch

yet, the issue continues with Special:Import. i think, but haven’t been able to verify yet, that the issue is with the custom stream that is created in Import.php using "uploadsource://$id".
Comment 10 Gerrit Notification Bot 2014-09-16 12:27:11 UTC
Change 159973 had a related patch set uploaded by Dan-nl:
Special:Import not importing Special:Export XMLs

https://gerrit.wikimedia.org/r/159973
Comment 11 Andre Klapper 2014-09-19 21:59:40 UTC
*** Bug 70658 has been marked as a duplicate of this bug. ***
Comment 12 Ori Livneh 2014-09-24 22:45:13 UTC
https://github.com/facebook/hhvm/pull/3840
Comment 13 Gerrit Notification Bot 2014-09-25 03:54:24 UTC
Change 162818 had a related patch set uploaded by Dan-nl:
Special:Import not importing Special:Export XMLs

https://gerrit.wikimedia.org/r/162818
Comment 14 Gerrit Notification Bot 2014-09-26 01:56:40 UTC
Change 159973 abandoned by Dan-nl:
Special:Import not importing Special:Export XMLs

Reason:
since a solution in hhvm was found, as ori mentioned above, i'll abandon this patch. the extracted unit test is available separately in I446f57c1 if you wish to consider adding it.

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

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


Navigation
Links