Page MenuHomeFreeBSD

graphics/mupdf: update to 1.23.11
ClosedPublic

Authored by uzsolt on Feb 14 2024, 4:13 PM.
Tags
None
Referenced Files
F82411094: D43880.diff
Sun, Apr 28, 8:34 AM
Unknown Object (File)
Sat, Apr 27, 7:43 AM
Unknown Object (File)
Fri, Apr 26, 3:25 AM
Unknown Object (File)
Sun, Apr 21, 9:58 PM
Unknown Object (File)
Sun, Apr 14, 5:39 PM
Unknown Object (File)
Mon, Apr 8, 10:36 PM
Unknown Object (File)
Sat, Apr 6, 5:06 AM
Unknown Object (File)
Fri, Apr 5, 4:16 AM
Subscribers
None

Details

Summary

Changelog: https://mupdf.com/releases/history.html

Portlint, portfmt and portclippy checks are ok.
Poudriere builds on 13.2 and 14 (amd64) are ok, only one Q/A test warning:
Warning: you might not need LIB_DEPENDS on libcurl.so

Without curl the build process drops a warning:
Package libcurl was not found in the pkg-config search path.
Perhaps you should add the directory containing `libcurl.pc'
to the PKG_CONFIG_PATH environment variable

I don't know what mupd's feature break because of missed curl (and how can I test it) so I don't want remove the curl dependency.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

uzsolt created this revision.

Update to 1.23.11.

In this commit (https://cgit.freebsd.org/ports/commit/graphics/mupdf?id=537cbffa5984539c50847e71989842f72cb3a118) the cosmetic changes done.

Portlint, portclippy, portfmt passed.
Poudriere testport on 13.2 and 14.0 passed on amd64.
The libcurl.so warning appears.

uzsolt retitled this revision from graphics/mupdf: update to 1.23.10 to graphics/mupdf: update to 1.23.11.Mar 1 2024, 6:58 AM

As mentioned earlier poudriere's lib detection method is not too robust and might not be the correct way to follow. So do not blindly remove those and retest like you have done here. curl should stay if it throws an error while absent.

Macro shipit:

As mentioned earlier poudriere's lib detection method is not too robust and might not be the correct way to follow. So do not blindly remove those and retest like you have done here. curl should stay if it throws an error while absent.

I don't want remove curl dependency.
Can I commit it?

I thought I said Ship It. :D

This revision is now accepted and ready to land.Mar 3 2024, 5:41 PM

I thought I said Ship It. :D

Yes, you said but I didn't see the green check mark and I wasn't sure.

This revision was automatically updated to reflect the committed changes.