Page MenuHomePhabricator

Upgrade spyc
Closed, DeclinedPublic

Description

Currently on 0.2.3

Release is 0.4.5

Not a huge amount of revisions (most are mainly docs or similar)...

So as long as the bugs/FR's are brought back in (Should we really commit these upstream too?)

Bjoern, i've added you as a CC, as thought you may be interested (as you seemingly use it)


Version: unspecified
Severity: enhancement
URL: http://code.google.com/p/spyc/

Details

Reference
bz22292

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 10:57 PM
bzimport set Reference to bz22292.

I'm not sure we want to upgrade Spyc because it's such crappy code (see http://www.mediawiki.org/wiki/Special:Code/MediaWiki/42547 and http://www.mediawiki.org/wiki/Special:Code/MediaWiki/53613#c3860 for details). I've put some work into trying to get to behave reasonably, and I won't throw that all out without testing that an upgrade won't cause regressions.

Ahhh

Wouldn't have even guessed about r53613, and r42457 explains why there is such a difference in code

I'll wontfix this then