From owner-svn-ports-head@freebsd.org Wed Jun 15 04:40:38 2016 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id DA7F6B72735; Wed, 15 Jun 2016 04:40:38 +0000 (UTC) (envelope-from pi@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 8896F2EFE; Wed, 15 Jun 2016 04:40:38 +0000 (UTC) (envelope-from pi@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id u5F4ebBS081053; Wed, 15 Jun 2016 04:40:37 GMT (envelope-from pi@FreeBSD.org) Received: (from pi@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id u5F4eb7f081047; Wed, 15 Jun 2016 04:40:37 GMT (envelope-from pi@FreeBSD.org) Message-Id: <201606150440.u5F4eb7f081047@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: pi set sender to pi@FreeBSD.org using -f From: Kurt Jaeger Date: Wed, 15 Jun 2016 04:40:37 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r416914 - in head/sysutils/py-salt: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Jun 2016 04:40:39 -0000 Author: pi Date: Wed Jun 15 04:40:36 2016 New Revision: 416914 URL: https://svnweb.freebsd.org/changeset/ports/416914 Log: sysutils/py-salt: Fix upstream issues #33554, #33578, #33608 and #33529 The patch fixes some issues that are new with 2016.3.0 - https://github.com/saltstack/salt/issues/33529 - https://github.com/saltstack/salt/issues/33554 - https://github.com/saltstack/salt/issues/33578 - https://github.com/saltstack/salt/issues/33608 PR: 209829 Submitted by: ohauer, Andreas Montalban Approved by: christer.edwards@gmail.com (maintainer timeout) Added: head/sysutils/py-salt/files/patch-salt_grains_core.py (contents, props changed) head/sysutils/py-salt/files/patch-salt_grains_disks.py (contents, props changed) head/sysutils/py-salt/files/patch-salt_modules_pkgng.py (contents, props changed) head/sysutils/py-salt/files/patch-salt_returners_local__cache.py (contents, props changed) Modified: head/sysutils/py-salt/Makefile head/sysutils/py-salt/files/patch-setup.py Modified: head/sysutils/py-salt/Makefile ============================================================================== --- head/sysutils/py-salt/Makefile Wed Jun 15 02:20:47 2016 (r416913) +++ head/sysutils/py-salt/Makefile Wed Jun 15 04:40:36 2016 (r416914) @@ -3,6 +3,7 @@ PORTNAME= salt PORTVERSION= 2016.3.0 +PORTREVISION= 1 CATEGORIES= sysutils python MASTER_SITES= CHEESESHOP PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX} Added: head/sysutils/py-salt/files/patch-salt_grains_core.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/sysutils/py-salt/files/patch-salt_grains_core.py Wed Jun 15 04:40:36 2016 (r416914) @@ -0,0 +1,12 @@ +# Issue #33608 +# https://github.com/saltstack/salt/issues/33608 +--- salt/grains/core.py.orig 2016-05-25 22:30:31 UTC ++++ salt/grains/core.py +@@ -1422,6 +1422,7 @@ def os_data(): + else: + grains['os'] = grains['kernel'] + if grains['kernel'] == 'FreeBSD': ++ grains.update(_bsd_cpudata(grains)) + try: + grains['osrelease'] = __salt__['cmd.run']('freebsd-version -u').split('-')[0] + except salt.exceptions.CommandExecutionError: Added: head/sysutils/py-salt/files/patch-salt_grains_disks.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/sysutils/py-salt/files/patch-salt_grains_disks.py Wed Jun 15 04:40:36 2016 (r416914) @@ -0,0 +1,51 @@ +--- salt/grains/disks.py.orig 2016-05-25 22:30:31 UTC ++++ salt/grains/disks.py +@@ -57,18 +57,27 @@ class _geomconsts(object): + + _datatypes = { + MEDIASIZE: ('re_int', r'(\d+)'), +- SECTORSIZE: 'int', +- STRIPESIZE: 'int', +- STRIPEOFFSET: 'int', +- ROTATIONRATE: 'int', ++ SECTORSIZE: 'try_int', ++ STRIPESIZE: 'try_int', ++ STRIPEOFFSET: 'try_int', ++ ROTATIONRATE: 'try_int', + } + + + def _datavalue(datatype, data): +- if datatype == 'int': +- return int(data) +- elif datatype and datatype[0] == 're_int': +- return int(re.search(datatype[1], data).group(1)) ++ if datatype == 'try_int': ++ try: ++ return int(data) ++ except ValueError: ++ return None ++ elif datatype is tuple and datatype[0] == 're_int': ++ search = re.search(datatype[1], data) ++ if search: ++ try: ++ return int(search.group(1)) ++ except ValueError: ++ return None ++ return None + else: + return data + +@@ -97,9 +106,11 @@ def _freebsd_geom(): + tmp[_geomconsts._aliases[attrib]] = value + + name = tmp.pop(_geomconsts.GEOMNAME) ++ if name.startswith('cd'): ++ return + + ret['disks'][name] = tmp +- if tmp[_geomconsts.ROTATIONRATE] == 0: ++ if tmp.get(_geomconsts.ROTATIONRATE) == 0: + log.trace('Device {0} reports itself as an SSD'.format(device)) + ret['SSDs'].append(name) + Added: head/sysutils/py-salt/files/patch-salt_modules_pkgng.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/sysutils/py-salt/files/patch-salt_modules_pkgng.py Wed Jun 15 04:40:36 2016 (r416914) @@ -0,0 +1,24 @@ +--- salt/modules/pkgng.py.orig 2016-05-25 22:30:31 UTC ++++ salt/modules/pkgng.py +@@ -97,7 +97,8 @@ def _get_pkgng_version(jail=None, chroot + ''' + return the version of 'pkg' + ''' +- return __salt__['cmd.run']([_pkg(jail, chroot), '--version']).strip() ++ cmd = _pkg(jail, chroot) + ['--version'] ++ return __salt__['cmd.run'](cmd).strip() + + + def _get_version(name, results): +@@ -287,9 +288,9 @@ def latest_version(*names, **kwargs): + for name in names: + # FreeBSD supports packages in format java/openjdk7 + if '/' in name: +- cmd = [_pkg(jail, chroot), 'search'] ++ cmd = _pkg(jail, chroot) + ['search'] + else: +- cmd = [_pkg(jail, chroot), 'search', '-S', 'name', '-Q', 'version', '-e'] ++ cmd = _pkg(jail, chroot) + ['search', '-S', 'name', '-Q', 'version', '-e'] + if quiet: + cmd.append('-q') + cmd.append(name) Added: head/sysutils/py-salt/files/patch-salt_returners_local__cache.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/sysutils/py-salt/files/patch-salt_returners_local__cache.py Wed Jun 15 04:40:36 2016 (r416914) @@ -0,0 +1,22 @@ +# Issue #33554 +# https://github.com/saltstack/salt/issues/33554 +--- salt/returners/local_cache.py.orig 2016-05-25 22:30:31 UTC ++++ salt/returners/local_cache.py +@@ -407,14 +407,14 @@ def clean_old_jobs(): + for final in t_path_dirs: + f_path = os.path.join(t_path, final) + jid_file = os.path.join(f_path, 'jid') +- if not os.path.isfile(jid_file): ++ if not os.path.isfile(jid_file) and os.path.exists(t_path): + # No jid file means corrupted cache entry, scrub it + # by removing the entire t_path directory + shutil.rmtree(t_path) +- else: ++ elif os.path.isfile(jid_file): + jid_ctime = os.stat(jid_file).st_ctime + hours_difference = (cur - jid_ctime) / 3600.0 +- if hours_difference > __opts__['keep_jobs']: ++ if hours_difference > __opts__['keep_jobs'] and os.path.exists(t_path): + # Remove the entire t_path from the original JID dir + shutil.rmtree(t_path) + Modified: head/sysutils/py-salt/files/patch-setup.py ============================================================================== --- head/sysutils/py-salt/files/patch-setup.py Wed Jun 15 02:20:47 2016 (r416913) +++ head/sysutils/py-salt/files/patch-setup.py Wed Jun 15 04:40:36 2016 (r416914) @@ -1,6 +1,6 @@ ---- setup.py.orig 2015-04-17 18:44:50 UTC +--- setup.py.orig 2016-05-25 22:30:31 UTC +++ setup.py -@@ -623,8 +623,8 @@ class SaltDistribution(distutils.dist.Di +@@ -986,8 +986,8 @@ class SaltDistribution(distutils.dist.Di def _property_data_files(self): # Data files common to all scenarios data_files = [