Skip to content

Conversation

@marcinmajsc
Copy link
Contributor

@marcinmajsc marcinmajsc commented Nov 7, 2025

Link: #27607
Link: openwrt/openwrt@1b48ebd

📦 Package Details

Maintainer: Ted Hess [email protected]
http://xiph.org/vorbis/

Description:

Vorbis is a general purpose audio and music encoding format contemporary to MPEG-4's AAC and TwinVQ, the next generation beyond MPEG audio layer 3. Unlike the MPEG sponsored formats (and other proprietary formats such as RealAudio G2 and Windows' flavor of the month), the Vorbis CODEC specification belongs to the public domain. All the technical details are published and documented, and any software entity may make full use of the format without license
fee, royalty or patent concerns.


🧪 Run Testing Details

  • OpenWrt Version: master
  • OpenWrt Target/Subtarget:
  • OpenWrt Device:

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/feeds/packages/libvorbis/refresh V=s
  • It is structured in a way that it is potentially upstreamable
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

@BKPepe
Copy link
Member

BKPepe commented Nov 7, 2025

Your PR is the same as you approved this one - #27774 with the same issues.

@marcinmajsc
Copy link
Contributor Author

Your PR is the same as you approved this one - #27774 with the same issues.

Sorry, my bad. PR was updated.

@marcinmajsc marcinmajsc force-pushed the libvorbis-cmake-min-ver branch from e635fe5 to 1ccb232 Compare November 7, 2025 06:53
@marcinmajsc marcinmajsc force-pushed the libvorbis-cmake-min-ver branch from 1ccb232 to df62152 Compare November 8, 2025 14:32
@hnyman hnyman merged commit ca4abc6 into openwrt:master Nov 8, 2025
12 checks passed
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.

3 participants