-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
File extension for downloaded Opus audios should be .ogg for Android 9 and below #8046
Comments
I don't think this is necessarily an issue with Opus as it is with Android support over different OS versions. It would be good to consider what we would want the new behavior to be if this were to be changed. Would we want the audio to be saved as |
Just save with .ogg extension for all android versions because there is no such thing as a .opus file, opus is a codec. The file container is actually a ogg, so correct extension is .ogg |
@Tanishq-Banyal Your information is incorrect. You're mixing containers and file extensions: https://en.wikipedia.org/wiki/Opus_(audio_format)#cite_note-MIME-and-FileExt-1 Following that link: https://wiki.xiph.org/MIME_Types_and_File_Extensions#.opus_-_audio.2Fogg So the codec is Opus, and the container format is indeed Ogg, but the "official" file extension is As a much clearer example, look at MP4: https://en.wikipedia.org/wiki/MPEG-4_Part_14#Filename_extensions:
Coming to your issue:
This is the bug you've reported. But LineageOS added support for the I've been storing and playing |
Part 2 (the comment was getting too long):
@TacoTheDank Using the Reading that section further, if you want Android System to natively recognize and play Opus audios for Android 5 and 6, you will need the Matroska container with the Opus codec, which was the only combo supported for those versions. You could use the Extra info for the interested: Most audio players only care about the container format, not the file extension. That is more the domain of file explorer apps, to be able to identify file types and show them in the UI correctly. As proof, you can rename an Opus audio file downloaded from Newpipe to |
My music player (musicolet) does recognise .opus as well as .ogg, just my system file explorer (Android Documents UI) doesn't recognise .opus. I am not affected much by this as I have a script running in background that renames any .opus file to .ogg in Music folder whenever a new .opus file is detected. |
Is your LOS 16 build older than the commits I linked to? That's the only reason I can think of why your system file explorer isn't recognising them. I'm on LOS 17.1, and the system Files app shows the file type of |
I am on LOS 16.0, and system files app shows mime type "application/octet-stream" ie. binary file But as I said earlier :-
|
Okay. Then let's limit this issue to keeping |
Yep I'm aware, I just shortened it in my statement and assumed that people visiting this issue would read as such in the linked Wikipedia article. Maybe I should have clarified, lol. |
Now there's a super-optimistic person. xD |
Quoting rfc7845:
As this is only a recommendation, I'd prefer to stick to I'm also doubtful that the crappy music players people love to use can handle the What do you think of an audio format option to save files as OT: @opusforlife2's username checks out ;D |
I don't see why we should mislabel files because people use crappy/old music apps. Instead, this could nudge them toward using updated music apps that do recognise the |
While the idealist in me agrees, the realist sees people that are confused about weird audio formats their favourite music player can't and blame NewPipe for it.
This implies competence among said developers which I don't think exists universally. |
I just searched, and we've never actually had anyone complain about not being able to play Opus files. (Disclaimer, there is one issue, but back then Newpipe's Opus support itself was broken.) |
just let the user choose file extension in download settings |
We're always conservative about introducing additional options that need to be maintained. In this case, an option isn't needed at all, because the suggested defaults make sense:
|
What does that mean, exactly? The discussion is open to anyone who wants to add anything. Apart from that, we need a PR to implement this. |
Checklist
Affected version
0.22.1
Steps to reproduce the bug
Expected behavior
Opus files should be downloaded with the
.ogg
file extension on Android 9 and below, since the support for recognising the.opus
file extension was added in Android 10.Actual behavior
File is downloaded with the
.opus
file extension on all OS versions, which is not recognised by many apps, including Android's MediaScanner.Affected Android/Custom ROM version
Android 5, 6, 7, 8 and 9.
The text was updated successfully, but these errors were encountered: