rxlabz / audioplayer

A flutter plugin to play audio files iOS / Android / MacOS / Web ( Swift/Java )
https://pub.dartlang.org/packages/audioplayer
493 stars 189 forks source link

New Warning Produced by Android Runtime #48

Open ThinkDigitalSoftware opened 6 years ago

ThinkDigitalSoftware commented 6 years ago

W/MediaPlayer(29189): Use of stream types is deprecated for operations other than volume control W/MediaPlayer(29189): See the documentation of setAudioStreamType() for what to use instead with android.media.AudioAttributes to qualify your playback use case

eddywm commented 6 years ago

I also noticed those warnings, you should probably propose a pull request since you seem to understand where they are coming from.

Apparently you are also trying to build an audio player and you are relying on this library the same way I do.

What I'm doing is working on patches and fixes and try to propose PR here. Since the maintainers of this project are a little bit slow to merge you'll have to fork this repo and add those new features on your forked version till the maintainers come back.

ThinkDigitalSoftware commented 6 years ago

I'm not actually sure where it's coming from, I just saw the errors pop up :/

On Fri, Jun 29, 2018, 1:18 AM Eddy WM notifications@github.com wrote:

I also noticed those warnings, you should probably propose a pull request since you seem to understand where it’s coming from.

Apparently you are also trying to build an audio player and you are relying on this library the same way I do.

What I'm doing is working on patches and fixes and try to propose PR here. Since the maintainers of this project are a little bit slow to merge you'll have to fork this repo and add those new features on your forked version till the maintainers come back.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rxlabz/audioplayer/issues/48#issuecomment-401283922, or mute the thread https://github.com/notifications/unsubscribe-auth/AV-HfV1A5ThFIz81piaFWKt9bl7va6m5ks5uBeLMgaJpZM4U8ePO .

ThinkDigitalSoftware commented 6 years ago

I fixed the errors and submitted a pull request. However, if I want to use these changes officially, would I have to create a forked pub package as well?

eddywm commented 6 years ago

@ThinkDigitalRepair You'll have to fork the repository and reference your version of this library in the yaml deps file waiting for the maintainers of this repo to merge your PR. That's exactly what I did, waiting for my pull request #43 to get merged.

There's a way to reference non published library in Flutter, so you can link directly a github repo in the .yaml dependencies file. https://flutter.io/using-packages/#adding-a-package-dependency-to-an-app

Look at Dependencies on unpublished packages section

ThinkDigitalSoftware commented 6 years ago

Yeah. But that takes it from pub.dartlang.org and not github, right? I submitted a github pull request, but I can't use the updated code unless I manually modify the package files in my editor, which is hack at best

On Sun, Jul 15, 2018, 2:43 PM Eddy WM notifications@github.com wrote:

@ThinkDigitalRepair https://github.com/ThinkDigitalRepair You'll have to fork the repository and refrence your version of this library in the yaml deps file waiting for the maintainers of this repo to merge your PR.

That's exactly what I did, waiting for my pull request #43 https://github.com/rxlabz/audioplayer/pull/43 to get merged.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rxlabz/audioplayer/issues/48#issuecomment-405120620, or mute the thread https://github.com/notifications/unsubscribe-auth/AV-HffOpjuCJB4e3vrXwnXMJbN84j6F-ks5uG7dtgaJpZM4U8ePO .

davidanaya commented 5 years ago

Is there any update on this? I also get this warnings when using the plugin in my flutter app

W/MediaPlayer(10941): Use of stream types is deprecated for operations other than volume control
W/MediaPlayer(10941): See the documentation of setAudioStreamType() for what to use instead with android.media.AudioAttributes to qualify your playback use case
YDJSZY commented 5 years ago

I have also this warning. I just run flutter in android emulators.

softcoder commented 5 years ago

Is steaming radio fixed for this control or should I find another package?