graphics/zathura-*: Update MASTER_SITES

[rigoletto]

Summary

PR: 242029

• graphics/zathura: update to 0.4.3
• graphics/zathura-djvu: update to 0.2.9
• graphics/zathura-pdf-poppler: update to 0.3.0

[rigoletto]

NOT tested!

[madpilot]

Hi,

Why the change in MASTER_SITE? The project home page, when clicking on downloads still links to the https://pwmt.org/projects/girara/download/girara-0.3.4.tar.xz URL, so why not follow what upstream does?

I completely agree on removing the www.madpilot.net "mirror" which I added some time ago to fix a temporary problem.

[rigoletto]

In D23908#525723, @madpilot wrote:

Hi,

Why the change in MASTER_SITE? The project home page, when clicking on downloads still links to the https://pwmt.org/projects/girara/download/girara-0.3.4.tar.xz URL, so why not follow what upstream does?

I completely agree on removing the www.madpilot.net "mirror" which I added some time ago to fix a temporary problem.

Hello,

While working in some of the zathura-* ports the source files were missing (I can't recall which ones now) and that is why I discovered they moved to a GitLab instance. Then I assumed they will move completely at some point, and IMO this is a better idea to keep MASTER_SITES consistent among all zathura-* instead of be changing one by one when necessary.

Cheers! :-)

[uzsolt]

In D23908#525724, @rigoletto wrote:

While working in some of the zathura-* ports the source files were missing (I can't recall which ones now) and that is why I discovered they moved to a GitLab instance. Then I assumed they will move completely at some point, and IMO this is a better idea to keep MASTER_SITES consistent among all zathura-* instead of be changing one by one when necessary.

I think would the best to use multiple MASTER_SITES.

[madpilot]

In D23908#525749, @uzsolt_uzsolt.hu wrote:

In D23908#525724, @rigoletto wrote:

While working in some of the zathura-* ports the source files were missing (I can't recall which ones now) and that is why I discovered they moved to a GitLab instance. Then I assumed they will move completely at some point, and IMO this is a better idea to keep MASTER_SITES consistent among all zathura-* instead of be changing one by one when necessary.

I think would the best to use multiple MASTER_SITES.

I second this, it looks like a good idea, if the distfiles are the same.

[rigoletto]

In D23908#528010, @madpilot wrote:

In D23908#525749, @uzsolt_uzsolt.hu wrote:

In D23908#525724, @rigoletto wrote:

While working in some of the zathura-* ports the source files were missing (I can't recall which ones now) and that is why I discovered they moved to a GitLab instance. Then I assumed they will move completely at some point, and IMO this is a better idea to keep MASTER_SITES consistent among all zathura-* instead of be changing one by one when necessary.

I think would the best to use multiple MASTER_SITES.

I second this, it looks like a good idea, if the distfiles are the same.

The distfiles are different.

EDIT: zathura-pdf-mupdf and girara are keep the same distfile.

[rigoletto]

Latest changes, including some portlint fixes.

[madpilot]

I have no problems for the parts the I am maintainer of.

Make sure you have contacted the other maintainers and have permission or they are actually timed out.

Not all maintainers are subscribed to phabricator, so I don't think the review emails are enough to make sure maintainers are informed.

[rigoletto]

In D23908#550315, @madpilot wrote:

I have no problems for the parts the I am maintainer of.

Make sure you have contacted the other maintainers and have permission or they are actually timed out.

Not all maintainers are subscribed to phabricator, so I don't think the review emails are enough to make sure maintainers are informed.

They have already approved on bugs.

Thanks!

[rigoletto]

@uzsolt_uzsolt.hu are you fine with it?

[uzsolt]

In D23908#565841, @rigoletto wrote:

@uzsolt_uzsolt.hu are you fine with it?

Sorry for late.
Why did you remove own mirror sites? Sometimes the pwmt.org is unavailable. If the source files (zathura-pdf-mupdf) are different I can update it on my host.

[mat]

I do not think all the USE_LDCONFIG bits are needed.

The USE_LDCONFIG knob is there when the port provides a shared library that other ports will need to link with, here, the .so are plugins, and they will be dlopen'ed on request.

[rigoletto]

In D23908#565853, @mat wrote:

I do not think all the USE_LDCONFIG bits are needed.

The USE_LDCONFIG knob is there when the port provides a shared library that other ports will need to link with, here, the .so are plugins, and they will be dlopen'ed on request.

Mostly were to shut portlint.

[rigoletto]

In D23908#565842, @uzsolt_uzsolt.hu wrote:

In D23908#565841, @rigoletto wrote:

@uzsolt_uzsolt.hu are you fine with it?

Sorry for late.
Why did you remove own mirror sites? Sometimes the pwmt.org is unavailable. If the source files (zathura-pdf-mupdf) are different I can update it on my host.

The distfiles were not matching and when the upstream mirror is not available the port system grab from the copy in FreeBSD own infrastructure.

[uzsolt]

In D23908#565903, @rigoletto wrote:

In D23908#565842, @uzsolt_uzsolt.hu wrote:

In D23908#565841, @rigoletto wrote:

@uzsolt_uzsolt.hu are you fine with it?

Sorry for late.
Why did you remove own mirror sites? Sometimes the pwmt.org is unavailable. If the source files (zathura-pdf-mupdf) are different I can update it on my host.

The distfiles were not matching and when the upstream mirror is not available the port system grab from the copy in FreeBSD own infrastructure.

Some version ago I could fetch the source, created a bug report (update report) but the committer couldn't fetch it: see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216067

[mat]

In D23908#565901, @rigoletto wrote:

In D23908#565853, @mat wrote:

I do not think all the USE_LDCONFIG bits are needed.

The USE_LDCONFIG knob is there when the port provides a shared library that other ports will need to link with, here, the .so are plugins, and they will be dlopen'ed on request.

Mostly were to shut portlint.

It is about the worst reason possible to make a change. portlint is a static lint tool, it has no clue about what you are doing, it just see stuff and tells you what it thinks you should do, but it very often gets it wrong.

[rigoletto]

Sync with HEAD.
I removed all LD_CONFIG entries and it apparently is fine but I will still need to review it.

[rigoletto]

I think this is fine now. @uzsolt_uzsolt.hu is this good for you?

Thanks!

[uzsolt]

In D23908#570003, @rigoletto wrote:

I think this is fine now. @uzsolt_uzsolt.hu is this good for you?

Thanks!

Yes, thanks!