From owner-svn-ports-head@freebsd.org Fri Mar 13 04:46:14 2020 Return-Path: Delivered-To: svn-ports-head@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id D7AF8273E47; Fri, 13 Mar 2020 04:46:14 +0000 (UTC) (envelope-from koobs@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 48dtSQ3k9vz4RkJ; Fri, 13 Mar 2020 04:46:14 +0000 (UTC) (envelope-from koobs@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 mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 786648A1; Fri, 13 Mar 2020 04:46:14 +0000 (UTC) (envelope-from koobs@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id 02D4kE2g065715; Fri, 13 Mar 2020 04:46:14 GMT (envelope-from koobs@FreeBSD.org) Received: (from koobs@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id 02D4kEhr065714; Fri, 13 Mar 2020 04:46:14 GMT (envelope-from koobs@FreeBSD.org) Message-Id: <202003130446.02D4kEhr065714@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: koobs set sender to koobs@FreeBSD.org using -f From: Kubilay Kocak Date: Fri, 13 Mar 2020 04:46:14 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r528327 - in head/sysutils/py-diffoscope: . files X-SVN-Group: ports-head X-SVN-Commit-Author: koobs X-SVN-Commit-Paths: in head/sysutils/py-diffoscope: . files X-SVN-Commit-Revision: 528327 X-SVN-Commit-Repository: ports 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.29 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: Fri, 13 Mar 2020 04:46:15 -0000 Author: koobs Date: Fri Mar 13 04:46:13 2020 New Revision: 528327 URL: https://svnweb.freebsd.org/changeset/ports/528327 Log: sysutils/py-diffoscope: Restore portability of zipinfo call /dev/stdin is a non-portable non-POSIX extension having different semantics on different operating systems. zininfo(1) exits with 9 when /dev/stdin is supplied on FreeBSD. In fact, unzip(1) explicitly documents that it does not support reading from stdin. [1] https://lists.reproducible-builds.org/pipermail/diffoscope/2020-March/002632.html PR: 244750 Submitted by: Michael Osipov MFH: 2020Q1 Added: head/sysutils/py-diffoscope/files/ head/sysutils/py-diffoscope/files/patch-diffoscope_comparators_zip.py (contents, props changed) Modified: head/sysutils/py-diffoscope/Makefile Modified: head/sysutils/py-diffoscope/Makefile ============================================================================== --- head/sysutils/py-diffoscope/Makefile Fri Mar 13 04:32:30 2020 (r528326) +++ head/sysutils/py-diffoscope/Makefile Fri Mar 13 04:46:13 2020 (r528327) @@ -3,6 +3,7 @@ PORTNAME= diffoscope PORTVERSION= 136 +PORTREVISION= 1 CATEGORIES= sysutils python MASTER_SITES= CHEESESHOP PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX} Added: head/sysutils/py-diffoscope/files/patch-diffoscope_comparators_zip.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/sysutils/py-diffoscope/files/patch-diffoscope_comparators_zip.py Fri Mar 13 04:46:13 2020 (r528327) @@ -0,0 +1,28 @@ +# Non-portable regression caused by 25fee28c/#879011 +# https://lists.reproducible-builds.org/pipermail/diffoscope/2020-March/002632.html +# TODO: Upstream + +--- diffoscope/comparators/zip.py.orig 2020-03-12 13:23:59 UTC ++++ diffoscope/comparators/zip.py +@@ -38,10 +38,7 @@ from .utils.command import Command + class Zipinfo(Command): + @tool_required('zipinfo') + def cmdline(self): +- # zipinfo (without -v) puts warning messages (some of which contain +- # $path) into stdin when stderr is not a tty, see #879011 for details. +- # to work around it, we run it on /dev/stdin instead, seems to work ok. +- return ['zipinfo', '/dev/stdin'] ++ return ['zipinfo', self.path] + + @property + def returncode(self): +@@ -54,9 +51,6 @@ class Zipinfo(Command): + returncode = 0 + + return returncode +- +- def stdin(self): +- return open(self.path, 'rb') + + def filter(self, line): + # we don't care about the archive file path