From owner-freebsd-ports-bugs@FreeBSD.ORG Thu Mar 22 18:30:02 2012 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 0875E106566C for ; Thu, 22 Mar 2012 18:30:02 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id CFC9A8FC1B for ; Thu, 22 Mar 2012 18:30:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q2MIU1tq063009 for ; Thu, 22 Mar 2012 18:30:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q2MIU1YY063008; Thu, 22 Mar 2012 18:30:01 GMT (envelope-from gnats) Resent-Date: Thu, 22 Mar 2012 18:30:01 GMT Resent-Message-Id: <201203221830.q2MIU1YY063008@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Christer Edwards Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C352A106564A for ; Thu, 22 Mar 2012 18:24:23 +0000 (UTC) (envelope-from nobody@FreeBSD.org) Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22]) by mx1.freebsd.org (Postfix) with ESMTP id ADE048FC08 for ; Thu, 22 Mar 2012 18:24:23 +0000 (UTC) Received: from red.freebsd.org (localhost [127.0.0.1]) by red.freebsd.org (8.14.4/8.14.4) with ESMTP id q2MIONB9045009 for ; Thu, 22 Mar 2012 18:24:23 GMT (envelope-from nobody@red.freebsd.org) Received: (from nobody@localhost) by red.freebsd.org (8.14.4/8.14.4/Submit) id q2MIONj2044993; Thu, 22 Mar 2012 18:24:23 GMT (envelope-from nobody) Message-Id: <201203221824.q2MIONj2044993@red.freebsd.org> Date: Thu, 22 Mar 2012 18:24:23 GMT From: Christer Edwards To: freebsd-gnats-submit@FreeBSD.org X-Send-Pr-Version: www-3.1 Cc: Subject: ports/166322: sysutils/salt - critical patch - loss of functionality X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Mar 2012 18:30:02 -0000 >Number: 166322 >Category: ports >Synopsis: sysutils/salt - critical patch - loss of functionality >Confidential: no >Severity: critical >Priority: high >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: maintainer-update >Submitter-Id: current-users >Arrival-Date: Thu Mar 22 18:30:01 UTC 2012 >Closed-Date: >Last-Modified: >Originator: Christer Edwards >Release: 9.0-RELEASE >Organization: >Environment: FreeBSD isabel 9.0-RELEASE FreeBSD 9.0-RELEASE #1: Sun Mar 18 13:42:35 MDT 2012 root@isabel:/usr/obj/usr/src/sys/ISABEL amd64 >Description: A critical bug was missed in testing the sysutils/salt 0.9.8 release. The current port is critically broken. The attached diff suggests two patches to regain functionality and address a python warning message. >How-To-Repeat: >Fix: In the even the attached .diff is not clear, a file needs to be created at files/patch-kvm_hyper.py to address a FreeBSD specific issue: --- salt-0.9.8/kvm_hyper.py 2012-03-22 10:57:13.019199302 -0600 +++ salt/modules/kvm_hyper.py 2012-03-22 10:57:45.441204588 -0600 @@ -46,6 +46,8 @@ return False if __grains__['virtual'] != 'physical': return False + if __grains__['kernel'] != 'Linux': + return False if 'kvm_' not in open('/proc/modules').read(): return False if not has_libvirt: Without this addition the FreeBSD salt minion chokes trying to read '/proc/modules' and exits. This means salt is essentially non-functioning on FreeBSD at this time. The second part of the attached diff is a minor update to the Python setup.py, which simply bypasses a warning message during build. It is non-critical. Patch attached with submission follows: diff -ruN salt-0.9.8/files/patch-kvm_hyper.py salt-0.9.8_1/files/patch-kvm_hyper.py --- salt-0.9.8/files/patch-kvm_hyper.py 1969-12-31 17:00:00.000000000 -0700 +++ salt-0.9.8_1/files/patch-kvm_hyper.py 2012-03-22 11:41:04.998199701 -0600 @@ -0,0 +1,11 @@ +--- salt-0.9.8/kvm_hyper.py 2012-03-22 10:57:13.019199302 -0600 ++++ salt/modules/kvm_hyper.py 2012-03-22 10:57:45.441204588 -0600 +@@ -46,6 +46,8 @@ + return False + if __grains__['virtual'] != 'physical': + return False ++ if __grains__['kernel'] != 'Linux': ++ return False + if 'kvm_' not in open('/proc/modules').read(): + return False + if not has_libvirt: diff -ruN salt-0.9.8/files/patch-setup.py salt-0.9.8_1/files/patch-setup.py --- salt-0.9.8/files/patch-setup.py 2012-01-25 00:16:18.000000000 -0700 +++ salt-0.9.8_1/files/patch-setup.py 2012-03-22 12:09:05.477195875 -0600 @@ -1,6 +1,6 @@ ---- salt-0.9.6/setup.py 2012-01-21 02:18:08.000000000 +0000 -+++ setup.py 2012-01-22 19:21:35.158004570 +0000 -@@ -44,7 +44,7 @@ +--- salt-0.9.8/setup.py 2012-03-21 11:46:26.000000000 -0600 ++++ setup.py 2012-03-22 11:57:51.926196304 -0600 +@@ -47,7 +47,7 @@ if 'SYSCONFDIR' in os.environ: etc_path = os.environ['SYSCONFDIR'] else: @@ -9,7 +9,7 @@ libraries = ['ws2_32'] if sys.platform == 'win32' else [] -@@ -91,10 +91,10 @@ +@@ -98,10 +98,10 @@ 'scripts/salt-run', 'scripts/salt'], data_files=[(os.path.join(etc_path, 'salt'), @@ -23,7 +23,7 @@ ['doc/man/salt-master.1', 'doc/man/salt-key.1', 'doc/man/salt.1', -@@ -104,7 +104,7 @@ +@@ -111,9 +111,8 @@ 'doc/man/salt-run.1', 'doc/man/salt-minion.1', ]), @@ -32,3 +32,5 @@ ['doc/man/salt.7', ]), ], +- install_requires=requirements, + ) >Release-Note: >Audit-Trail: >Unformatted: