Bug 94055

Porttools Cc:s freebsd-ports@ for unmaintained ports.  This simply creates
duplicate postings.

Fix: From: Diane Bruce <db@db.net>
To: Mark Linimon <linimon@lonesome.com>
Cc: sergei@FreeBSD.org

On Sun, Feb 26, 2006 at 01:13:36AM -0600, Mark Linimon wrote:
> Please don't Cc: ports@ on PRs.  It just creates duplicate messages.  Thanks.

It's a bug in the port submit tool.
If one changes both the maintainer from ports@FreeBSD.org and updates
the port, it cc's ports@FreeBSD.org anyway.

diff attached as well as here (I know my proxy munges patches)

--
- db@db.net http://www.db.net/~db

--HlL+5n6rz5pIUxbD
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="cmd_submit.diff"



--HlL+5n6rz5pIUxbD----JiTLZu6zJSbU6xQEdVq8hHq8Cnj66cDKI0Ejgd3W2vtRBNUl
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- /usr/local/share/porttools/cmd_submit.ORIG	Sun Feb 26 10:36:38 2006
+++ /usr/local/share/porttools/cmd_submit	Sun Feb 26 11:06:18 2006
@@ -233,7 +233,11 @@
 		NEW_MAINTAINER="`awk '/^\+MAINTAINER/ { print $2; }' ${PATCH}`"
 		if [ "${NEW_MAINTAINER}" != "${OLD_MAINTAINER}" ]
 		then
-			CC="${OLD_MAINTAINER}"
+			if [ "${OLD_MAINTAINER}" != "ports@FreeBSD.org" ]
+			then
+				CC="${OLD_MAINTAINER}"
+                        fi
+
 			case ${NEW_MAINTAINER} in
 			# New maintainer is you
 			${EMAIL})