Page MenuHomeFreeBSD

graphics/zathura: Update to 4.3
ClosedPublic

Authored by rigoletto on Feb 23 2019, 1:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 6:16 AM
Unknown Object (File)
Fri, Apr 12, 6:14 AM
Unknown Object (File)
Fri, Apr 12, 6:13 AM
Unknown Object (File)
Fri, Apr 12, 6:13 AM
Unknown Object (File)
Fri, Apr 12, 6:13 AM
Unknown Object (File)
Fri, Apr 12, 5:37 AM
Unknown Object (File)
Wed, Apr 10, 5:08 AM
Unknown Object (File)
Tue, Apr 9, 8:17 PM
Subscribers
None

Details

Summary

PR 235952 233727 229935 232902 233021

  • update graphics/zathura to 4.3
  • update graphics/mupdf to 1.14.0
  • update graphics/zathura-pdf-mupdf to 0.3.4

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 22675
Build 21776: arc lint + arc unit

Event Timeline

rigoletto added a reviewer: uzsolt.
graphics/mupdf/Makefile
6–7

^ DISTVERSIONPREFIX comes before DISTVERSION

22

^ that should be handled by USES=jpeg.

graphics/zathura-pdf-mupdf/Makefile
30

^ why +=?

rigoletto marked 3 inline comments as done.

Fixing.

  • remove jpeg-turbo from BUILD_DEPENDS in graphics/mupdf
  • remove + from MESON_ARGS in zathura-pdf-mupdf

Thanks.

Can you view pdf files with zathura?

In D19301#413297, @uzsolt_uzsolt.hu wrote:

Can you view pdf files with zathura?

I usually use just mupdf, but yes they are opening using zathura-pdf-mupdf.

↓ Why is this showing up in here? ↓

graphics/mupdf/Makefile
6–7

But that is DISTVERSIONSUFFIX. :-D

Remove gmake from graphics/zathura-pdf-mupdf. Not needed.

Looks good to me.
If the two maintainers are happy with it too, go for it.

This revision is now accepted and ready to land.Feb 23 2019, 5:54 PM

ah wait, was removing the NLS optiona intentional? :D

This revision now requires changes to proceed.Feb 23 2019, 5:55 PM

Looks good to me.
If the two maintainers are happy with it too, go for it.

If zathura works with mupdf plugin (now I can't test it) for me it's okay.

ah wait, was removing the NLS optiona intentional? :D

Yes, there is no option to actually disable it. The option was no-op. :-)

In D19301#413325, @uzsolt_uzsolt.hu wrote:

Looks good to me.
If the two maintainers are happy with it too, go for it.

If zathura works with mupdf plugin (now I can't test it) for me it's okay.

Is it broken with some other plugin?

ok, in that case it looks fine to me again :D

This revision is now accepted and ready to land.Feb 23 2019, 10:43 PM
This revision was automatically updated to reflect the committed changes.