Created attachment 224797 [details]
Patch for libogg
Backport CMake fixes from upstream and switch to CMake
PORTVERSION --> DISTVERSION as per Porter's Handbook
Replace downloads.xiph.org with ftp.osuosl.org (redirects) in MASTER_SITES
Don't build static lib (couldn't find any users in ports tree)
Compile tested on 13.0-STABLE #0 stable/13-n245283-70a2e9a3d44 (arm64) (make, make check-plist, make test)
Poudriere testport OK 12.2-RELEASE (amd64)
Reported issue to upstream about Makefiles in docs dir
Thank you, but I disagree with all of the proposed changes.
There is no reason to switch to cmake, which is a heavy dependency. The included autotools build framework works out of the box.
PORTVERSION is the more common idiom in the ports tree. Both variants are fine, portlint agrees.
downloads.xiph.org is the canonical URL.
We generally install static libraries to permit static linking.
As I wrote it provides CMake package files which is beneficial as we see more and more projects overall moving away from autotools and many users already depends/pulls in CMake so I don't really see that as an valid argument.
While both are still valid DISTVERSION is preferred (since a few years back) so might as well switch since we're doing changes but if you strongly want PORTVERSION I don't mind keeping it.
Direct links are in general preferred over redirects (same goes for http --> https redirects).
As for static libraries we're slowly moving away, see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=255735#c5 as that was recently discussed.
libogg (and libvorbis) are old and stable projects. Existing software deals with the way they are now. There appears to be no need to install additional cmake files; not even Debian does.
You have strong opinions how the ports tree should be reshaped. Get portsmgr to issue corresponding guidance, and I'll follow suit. As is, I reject such changes.