Bug 269333 - dns/mDNSResponder_nss: Update to 1790.80.10
Summary: dns/mDNSResponder_nss: Update to 1790.80.10
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Nuno Teixeira
URL:
Keywords: needs-patch
Depends on: 256955
Blocks:
  Show dependency treegraph
 
Reported: 2023-02-04 23:39 UTC by DutchBSD
Modified: 2023-06-26 19:26 UTC (History)
3 users (show)

See Also:
eduardo: maintainer-feedback? (mazhe)


Attachments
dns/mDNSResponder_nss : update to 1790.80.10 (1.17 KB, patch)
2023-03-01 13:04 UTC, Matthieu Volat
mazhe: maintainer-approval+
Details | Diff
dns/mDNSResponder_nss : poudriere testport log (22.39 KB, text/plain)
2023-03-01 13:05 UTC, Matthieu Volat
no flags Details
net/mDNSResponder : update to 1790.80.10 (12.36 KB, patch)
2023-03-13 06:05 UTC, Matthieu Volat
mazhe: maintainer-approval? (sunpoet)
Details | Diff
net/mDNSResponder : poudriere testport log (41.72 KB, text/plain)
2023-03-13 06:07 UTC, Matthieu Volat
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description DutchBSD 2023-02-04 23:39:04 UTC
mDNSResponder has released way newer releases than our current port.

see: https://github.com/apple-oss-distributions/mDNSResponder/tags

Maybe one is willing to upgrade the port to mDNSResponder-1790.80.10?

regards
Comment 1 Matthieu Volat 2023-02-06 09:19:18 UTC
Hi, maintainer here,

Yeah, I was kind of hoping to get some feedback on https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=256955 to transform this port in a pseudo-slave port to avoid running behind net/mDNSResponder like that...

If this looks too much to review, let's just bump again the version number, the base mdns code is quite stable, as is FreeBSD nsswitch api, so it's always just PORTVERSION/distinfo update.
Comment 2 Nuno Teixeira freebsd_committer freebsd_triage 2023-02-06 10:04:53 UTC
(In reply to Matthieu Volat from comment #1)

Let's wait for sunpoet@ opinion on slave port #256955

Cheers
Comment 3 Graham Perrin freebsd_committer freebsd_triage 2023-02-18 02:59:35 UTC
dns/mDNSResponder does not exist in the tree, should this report be for 
dns/mDNSResponder_nss or for 
net/mDNSResponder?
Comment 4 Matthieu Volat 2023-02-18 06:00:36 UTC
This is for dns/mDNSResponder_nss, the base port is up to date.

Maybe let's have a timeout for progress on #256955 to 2023/02/26? #256955 can still happen on a more up-to-date port.
Comment 5 Matthieu Volat 2023-03-01 13:04:25 UTC
Ho... Since things went without an update to 256955, I went on to create a patch simply updating dns/mDNSResponder_nss... Which when I finaly realized that net/mDNSResponder is *not* up to date (Apple went on a release frenzy it seems).

Here's my proposal: I can post patches to update dns/mDNSResponder_nss to 1790.80.10. Since this module do not need any newer API than the last one, it even works against current port net/mDNSResponder.

We can enlarge the scope of this PR to include net/mDNSResponder, if sunpoet@ has no time, I can have a pass at proposing also a patch for this one.

When net/mDNSResponder patch is submitted/approved, I can optionaly run another dns/mDNSResponder_nss test, but I'm 99.999% confident it's not needed.

Then both ports can be updated. Slave port issue can wait for a bit.

Would that be a good way to proceed?
Comment 6 Matthieu Volat 2023-03-01 13:04:58 UTC
Created attachment 240500 [details]
dns/mDNSResponder_nss : update to 1790.80.10
Comment 7 Matthieu Volat 2023-03-01 13:05:31 UTC
Created attachment 240501 [details]
dns/mDNSResponder_nss : poudriere testport log
Comment 8 Matthieu Volat 2023-03-13 06:05:57 UTC
Created attachment 240808 [details]
net/mDNSResponder : update to 1790.80.10

Hi,

Sorry this took more than a week, but I rebased the current patches on top of mDNSResponder 1790.80.10 and added a new one to handle a objective-c declaration that slipped in the code.

From my usage of this ports, things works -- both for the base usage and in conjonction with dns/mDNSResponder_nss for direct hostname resolution, for ipv4 and ipv6.

This patch must be accepted by sunpoet@ (maintainer), if done both patchs can be included at the same time.

Regards,
Comment 9 Matthieu Volat 2023-03-13 06:07:27 UTC
Created attachment 240809 [details]
net/mDNSResponder : poudriere testport log

Poudriere testport log file.