Skip to content

use opus-mini instead of -nano - #183

Open
Samueru-sama wants to merge 1 commit into
Eden-CI:masterfrom
Samueru-sama:patch-10
Open

Samueru-sama wants to merge 1 commit into
Eden-CI:masterfrom
Samueru-sama:patch-10

Conversation

@Samueru-sama

Copy link
Copy Markdown
Contributor

Both are the same package opus-nano has been kept around for backwards compat.

However since we are working on adding multi-arch support, I will also be dropping the old package names in the near future.

Also does the appimage even contain libopus now? Eden was updated to rely on ffmpeg for this. IIRC ffmpeg only needs libopus to encode, not decode.

@crueter

crueter commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

that PR was reverted for now... it might be better to just use a bundled static opus?

@Samueru-sama

Copy link
Copy Markdown
Contributor Author

it might be better to just use a bundled static opus?

Nah

@crueter

crueter commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

why? all other platforms use static opus

@xinitrcn1

Copy link
Copy Markdown
Contributor

Both are the same package opus-nano has been kept around for backwards compat.

However since we are working on adding multi-arch support, I will also be dropping the old package names in the near future.

Also does the appimage even contain libopus now? Eden was updated to rely on ffmpeg for this. IIRC ffmpeg only needs libopus to encode, not decode.

ffmpeg needs libopus to decode as well

@crueter

crueter commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

No it doesn't

@xinitrcn1

xinitrcn1 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

@crueter

crueter commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

We won't use the libopus decoder. That would defeat the whole point because then we'd still depend on Opus itself.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants