Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unbound: enable support for dnscrypt, dnstap #8148

Merged
merged 1 commit into from
Aug 31, 2020

Conversation

iefdev
Copy link
Contributor

@iefdev iefdev commented Aug 19, 2020

Description

  • Enable support for dnscrypt, dnstap
  • Add variant for dnstap
  • Disable dsa
Type(s)
  • enhancement
Tested on

macOS 10.7.5 11G63
Xcode 4.6.3 4H1503

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vsd install?

  - Enable support for dnscrypt, dnstap
  - Add variant for dnstap
  - Disable dsa
@macportsbot
Copy link

Notifying maintainers:
@nerdling for port unbound.

@macportsbot macportsbot added maintainer: open Affects an openmaintainer port type: enhancement labels Aug 19, 2020
@iefdev
Copy link
Contributor Author

iefdev commented Aug 19, 2020

I would also really like to add this one to: (but…)

if {${os.major} >= 15} {
    configure.args-append   --enable-tfo-client \
                            --enable-tfo-server
}

This enables the support for TCP Fast Open: »»»

macOS have support for that from 10.11. So, before adding anything - someone with 10.11 would have to test that it installs correctly without the error that follows.

Unbound have taken the approach to error the install if not found, instead of looking for it first and skip it. The function is connectx()

If the if-condition works, I guess one could just add it as is now, &/or put it in a variant and mention the 10.11 requirement in the description?

For me it's not so important since I can't use it, but I think most who can would like to have it.

@macportsbot
Copy link

Travis Build #13487 Passed.

Lint results
--->  Verifying Portfile for unbound
--->  0 errors and 0 warnings found.

Port unbound success on xcode10.3. Log
Port unbound success on xcode9.4. Log
Port unbound success on xcode8.3. Log
Port unbound success on xcode7.3. Log

@nerdling
Copy link
Member

someone with 10.11 would have to test that it installs correctly without the error that follows.

If that's on a branch and the time-to-build isn't egregious, liekly Travis Build here will let you know if it works on 10.11 (xcode 7.3 is run on 10.11 I believe: https://paste.macports.org/6da7256b95d3 )

@nerdling nerdling merged commit be20394 into macports:master Aug 31, 2020
@iefdev
Copy link
Contributor Author

iefdev commented Aug 31, 2020

If that's on a branch and the time-to-build isn't egregious, liekly Travis Build here will let you know if it works on 10.11 (xcode 7.3 is run on 10.11 I believe: https://paste.macports.org/6da7256b95d3 )

Thanks! Great tip. That would maybe be worth a try. Maybe add it without the if condition, and see where at what version it fails. The build time wouldn't increase as it's just adding the support/option. The deps for dnstap though… The protobuf-c took an eon to build. One of the reasons I put it in a variant.

I'll look more into it later. Perhaps for next upgrade. They'll have their DoH thing coming next time (I think/hope). I'm really exited about that one.

@iefdev iefdev deleted the unbound branch August 31, 2020 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer: open Affects an openmaintainer port type: enhancement
3 participants