From owner-freebsd-ports-bugs@FreeBSD.ORG Tue Apr 28 18:44:52 2015 Return-Path: Delivered-To: freebsd-ports-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 9A8B8FAE for ; Tue, 28 Apr 2015 18:44:52 +0000 (UTC) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (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 7D49816EE for ; Tue, 28 Apr 2015 18:44:52 +0000 (UTC) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.14.9/8.14.9) with ESMTP id t3SIiquJ044118 for ; Tue, 28 Apr 2015 18:44:52 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 199069] New USE_GITHUB has inconsistent DISTNAMEs for same GH_TAGNAME Date: Tue, 28 Apr 2015 18:44:52 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Ports Framework X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: commit-hook@freebsd.org X-Bugzilla-Status: Closed X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: portmgr@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Apr 2015 18:44:52 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=199069 --- Comment #1 from commit-hook@freebsd.org --- A commit references this bug: Author: bdrewery Date: Tue Apr 28 18:44:17 UTC 2015 New revision: 384932 URL: https://svnweb.freebsd.org/changeset/ports/384932 Log: USE_GITHUB: For the new support, fix DISTNAME to be more consistent. When using GH_TAGNAME the DISTNAME would have GH_PROJECT and GH_ACCOUNT in it. When not using GH_TAGNAME it would not have this. Now both cases will add in the GH_PROJECT and GH_ACCOUNT. Add special care to ensure that the DISTVERSION is not added in twice. If a port does GH_TAGNAME=v${PORTVERSION} it will be added in twice though. For that case DISTVERSIONPREFIX=v should be set and no GH_TAGNAME should be used. empty() is used rather than (!defined || !${}) to support fmake. The purpose of setting DISTNAME at all in these cases is to make it more clear that the distfile is from *GITHUB* and to avoid collisions if a project were to be renamed or moved. Without adding in GH_PROJECT and GH_ACCOUNT then there are real risks that collisions on filenames would happen on renamed or moved projects, which is fairly common. A GITHUB-generated file may not match a custom-rolled or git-archive-rolled distfile. PR: 199069 With hat: portmgr Testing done: All USE_GITHUB ports without GH_COMMIT were checksum/fetch/extract/WRKSRC tested. Changes: head/Mk/bsd.port.mk head/archivers/liblz4/distinfo head/astro/gpxloggerd/distinfo head/audio/gogglesmm/distinfo head/benchmarks/smhasher/distinfo head/comms/dump1090_mr/distinfo head/databases/php5-redis/distinfo head/devel/editline/distinfo head/devel/fstrm/distinfo head/devel/hub/distinfo head/devel/injeqt/distinfo head/devel/osc/distinfo head/devel/protobuf-c/distinfo head/devel/sdl2pp/distinfo head/graphics/libsixel/distinfo head/irc/insub/distinfo head/lang/chibi-scheme/distinfo head/mail/libetpan/distinfo head/mail/mu/distinfo head/mail/roundcube-contextmenu/distinfo head/math/openblas/distinfo head/multimedia/emby-server/distinfo head/multimedia/libass/distinfo head/multimedia/openh264/distinfo head/multimedia/plexhometheater/distinfo head/net/GeoIP/distinfo head/net/freevrrpd/distinfo head/net/geoipupdate/distinfo head/net/go-cs/distinfo head/net/google-daemon/distinfo head/net/google-startup-scripts/distinfo head/net/libilbc/distinfo head/net/libsrtp/distinfo head/net/shadowsocks-libev/distinfo head/net/syncthing/distinfo head/net-mgmt/icinga2/distinfo head/polish/libgadu/distinfo head/ports-mgmt/poudriere/distinfo head/print/indexinfo/distinfo head/security/libzrtpcppcore/distinfo head/security/passivedns/distinfo -- You are receiving this mail because: You are on the CC list for the bug.