Skip to content

Conversation

@dehy
Copy link

@dehy dehy commented May 13, 2016

No description provided.

@ryanrca
Copy link

ryanrca commented Dec 10, 2016

Hi, whats the status of this 7 month old PR?
I'm building out a bunch of tools for stockmusic.net and this would be a most welcome addition to our tool chain.

Thanks!

@willwh
Copy link

willwh commented Jan 18, 2017

I'd like to know too, I'd love to use this in my project.

@w3spi5
Copy link

w3spi5 commented Feb 5, 2017

Could you merge this ? I need to use this package with php7.1

@pkwasnik
Copy link

Me too. Please support PHP7.

@phansys phansys self-requested a review May 25, 2017 20:48
@phansys
Copy link
Owner

phansys commented May 26, 2017

@dehy, thank you for your contribution; and sorry for the delay. I've merged #21 recently, so the constraints of 2.1 are now allowing PHP7.

Sincerely, I'm not depending on this library for a very long time, so I'm not keeping it maintained. Moreover, now that the official getID3 library is published at GitHub and accepts contributions, is a good time to focus all the effort on it.

For these reasons, I must abandon this fork; so, in order to provide a path to make this movement as easy as possible, I plan to create a release for 2.1.0 version and update the master branch as alias of 3.0-dev, allowing the upgrades breaking BC (like the database driver change proposed in this PR, unless you can provide a bridge capable to use the proper driver based on the capabilities of the environment).

I'll keep this PR open until these concerns are consolidated.

/cc @ryanrca, @willwh, @Rypsx, @kwach2000.

@pkwasnik
Copy link

Great, Thank You!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants