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

privoxy: Bugfix, add privoxy-pki-bundle subport that depends on other PKI bundles #14386

Merged
merged 1 commit into from Mar 28, 2022

Conversation

essandess
Copy link
Contributor

Description

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 12.3 21E230 x86_64
Xcode 13.3 13E113

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 -vst install?
  • tested basic functionality of all binary files?

@essandess
Copy link
Contributor Author

essandess commented Mar 28, 2022

Note: I don't believe that the new port privoxy-pki-bundle should be built by a buildbot and provided by MacPorts so that the CAs are guaranteed to be as recent as the latest ones provided by the dependent ports. How should this be reflected in the Portfile?

@mascguy mascguy self-assigned this Mar 28, 2022
@mascguy
Copy link
Member

mascguy commented Mar 28, 2022

Steve, I need to test locally with these changes, once I can take a few minutes away from work.

In the interim, if you're interested in cleaning up the path copy-pasta too, I took a stab at it here. (I only ran a quick test, but seems to work properly.) Feel free to apply my changes to this PR, if you're comfortable with that.

https://github.com/mascguy/macports-ports/tree/mascguy-privoxy-cleanup

@essandess
Copy link
Contributor Author

@mascguy Thank you very much for the suggestions. (I imagined you had a different issue in mind with all the temp file redirections.)

Honestly, I prefer the full references in the commands to be able to follow along directly, rather than cross checking against a list of indirect references in variables, as well not clearly indicating what's in destroot and what's in the final install. Therefore, my preference is for the implementation in the current PR.

@mascguy
Copy link
Member

mascguy commented Mar 28, 2022

Honestly, I prefer the full references in the commands to be able to follow along directly, rather than cross checking against a list of indirect references in variables, as well not clearly indicating what's in destroot and what's in the final install. Therefore, my preference is for the implementation in the current PR.

No worries, I'm fine with that.

I'll merge once I've had a chance to test the fix locally. (Hopefully during lunch today.)

@mascguy
Copy link
Member

mascguy commented Mar 28, 2022

Steve, activation still fails, at least when upgrading.

Ah, just noticed your comment in the ticket, stating that forced activation is still required. I'm guessing this isn't a new scenario though. Have you checked the Portfile Recipes page on our wiki?

EDIT: You can start with this section, though it may not precisely match. Still, might provides some ideas:

https://trac.macports.org/wiki/PortfileRecipes#configfiles

@mascguy mascguy merged commit 8662ecd into macports:master Mar 28, 2022
@essandess essandess deleted the privoxy branch March 28, 2022 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants