Bug 215959

Summary: [patch] sysutils/htop does not need python to build
Product: Ports & Packages Reporter: Adam Weinberger <adamw>
Component: Individual Port(s)Assignee: Adam Weinberger <adamw>
Status: Closed FIXED    
Severity: Affects Only Me CC: gaod
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (gaod)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Remove python dependency
none
Build log none

Description Adam Weinberger freebsd_committer freebsd_triage 2017-01-11 06:51:44 UTC
Created attachment 178740 [details]
Remove python dependency

htop doesn't seem to need python at all to build. Please see the attached log.

AFAICT we can just remove the python dependency entirely (as well as the shebangfix).
Comment 1 Adam Weinberger freebsd_committer freebsd_triage 2017-01-11 06:52:32 UTC
Created attachment 178741 [details]
Build log

Build log showing successful build without python.
Comment 2 Hung-Yi Chen 2017-01-11 08:01:54 UTC
Hi Adam,

You are right.

There is no need to invoke python.

The patch seems fine, please commit it, thanks.
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-01-11 14:15:27 UTC
A commit references this bug:

Author: adamw
Date: Wed Jan 11 14:14:55 UTC 2017
New revision: 431183
URL: https://svnweb.freebsd.org/changeset/ports/431183

Log:
  Python is never actually used in the build here, so remove the dependency
  and the associated shebangfix.

  PR:		215959
  Approved by:	maintainer (Hung-Yi Chen)

Changes:
  head/sysutils/htop/Makefile
Comment 4 Adam Weinberger freebsd_committer freebsd_triage 2017-01-11 14:16:20 UTC
Done. Thanks for reviewing so quickly!