Bug 243048

Summary: net-mgmt/net-snmp: fix conditional for OpenSSL 1.1.1
Product: Ports & Packages Reporter: Franco Fichtner <franco>
Component: Individual Port(s)Assignee: Ryan Steinmetz <zi>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (zi)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
only match openssl itself or proper FreeBSD none

Description Franco Fichtner 2020-01-03 08:47:50 UTC 
Created attachment 210407 [details]
only match openssl itself or proper FreeBSD
Comment 1 Ryan Steinmetz freebsd_committer freebsd_triage 2020-01-03 14:01:29 UTC 
Why is this patch necessary?  Are you using the FreeBSD ports tree on non-FreeBSD systems?
Comment 2 Franco Fichtner 2020-01-03 14:05:18 UTC 
As per general rule OSVERSION check should be prefixed with OPSYS check since the OS may not be FreeBSD and the check only tries to apply to FreeBSD.
Comment 3 Ryan Steinmetz freebsd_committer freebsd_triage 2020-01-03 14:12:41 UTC 
Sorry to be ignorant here, but, can you give me an example?

When would non-FreeBSD systems use the FreeBSD ports tree?
Comment 4 Franco Fichtner 2020-01-03 14:14:18 UTC 
E.g. DragonFlyBSD DPorts https://www.dragonflybsd.org/docs/howtos/HowToDPorts/#index2h1
Comment 5 commit-hook freebsd_committer freebsd_triage 2020-01-03 14:23:32 UTC 
A commit references this bug:

Author: zi
Date: Fri Jan  3 14:22:48 UTC 2020
New revision: 521941
URL: https://svnweb.freebsd.org/changeset/ports/521941

Log:
  - Add OPSYS to conditional

  PR:		243048

Changes:
  head/net-mgmt/net-snmp/Makefile
Comment 6 Franco Fichtner 2020-01-03 14:24:24 UTC 
Thanks a lot! :)