Page MenuHomePhabricator

Publish File_Ogg as a composer package
Open, LowPublic

Description

We killed the PEAR dependency and module of File_Ogg with https://gerrit.wikimedia.org/r/216020

We should probably republish it as a composer package.

Event Timeline

TheDJ raised the priority of this task from to Needs Triage.
TheDJ updated the task description. (Show Details)
TheDJ added a subscriber: TheDJ.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
TheDJ set Security to None.

Seems ours has diverged from upstream...

https://github.com/pear/File_Ogg

Should we just get them to publish it to packagist?

brion added subscribers: tstarling, brion.

I'll take this on, was looking at making some fixes anyway. @tstarling can you set me up as a maintainer for Pear/File_Ogg and I'll try and get it synced with our inline version and set up with Composer?

Work in progress retooling some of the old code to be namespace & PSR-4 autoloader friendly on my work fork branch: https://github.com/brion/File_Ogg/commits/modernize Also merged the fix for certain cut-off positions, and removed the PEAR_Exception dependency.

@tstarling can you set me up as a maintainer for Pear/File_Ogg

Done.

Ok current status on this -- I need to pick up my modernize branch from https://github.com/brion/File_Ogg/commits/modernize and finish reworking it for PSR-4 so it's autoloader-friendly and composer-friendly.

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)

It would be nice to get this finished rather than namespacing these classes under TMH, and then having to move them again later...