From owner-freebsd-ports-bugs@FreeBSD.ORG Sun Aug 3 17:50:51 2014 Return-Path: Delivered-To: freebsd-ports-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id A5646758 for ; Sun, 3 Aug 2014 17:50:51 +0000 (UTC) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 8C75529A6 for ; Sun, 3 Aug 2014 17:50:51 +0000 (UTC) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.14.8/8.14.8) with ESMTP id s73HopK9024851 for ; Sun, 3 Aug 2014 17:50:51 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 192347] [maintainer update] multimedia/universal-media-server: Update to 4.0.1 + FIXES Date: Sun, 03 Aug 2014 17:50:51 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports Tree X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: marino@FreeBSD.org X-Bugzilla-Status: Open X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: robak@freebsd.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_status assigned_to Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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.18 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 03 Aug 2014 17:50:51 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D192347 John Marino changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Open Assignee|freebsd-ports-bugs@FreeBSD. |robak@freebsd.org |org | --- Comment #19 from John Marino --- (In reply to dreamcat4 from comment #17) > (In reply to John Marino from comment #14) > > It still doesn't answer the question: Why mess with the pkg-plist inste= ad of > > just fixing the script or reporting the script broken on the previous b= ug > > report? >=20 > Yeah, agree here this question is not answered by me. But I think you can= 't > be satisfied with my explanation, which is this: To be utterly safe in te= rms > of FUNCTIONAL behaviour. To stay on the side of safety. Because the known > tested FUNCTIONAL configuration (by me) is the old ways. (you say isn't > correct). So in other words, there was no identified problem. As far as anybody knows the committed pkg-plist worked fine. > I still don't fully understand (myself) commiter #1's ALL of his 'pkg-pli= st' > changes in their entirety. And this is because he didn't bother to explain > it (to me). He just said "take a look". And immediately closed / committe= d. He should explain his logic. > * Allows people involved to move on to other higher-priority issues becau= se > many other things in Ports tree are more seriously broken than this. Only the script was broken, so only the script needs to be fixed. > * the guy who committed them made a boo-boo and didn't really do it > properly.=20 He didn't see that you were using PLIST_SUBS in the RC script. Simple oversight. > Then I can't seriously be expected to say "just continue to do it HIS way > and I assume it'll all be fine," >=20 > "well probably, since that was some new configuration i had never seen > before and had never functionally tested myself"=20 >=20 > OR i could instead say: >=20 > "Here, take this last known - good patch, which I tested last week and all > was working 100% fine with the log file, pid file, rc.d script, etc, etc= =E2=80=A6" >=20 > So that's what I say / said. Please don't give me extra grief about it wh= en > I wasn't the guy who fluffed it up. Part of the issue is you are now mixing bugs. It would have been better to reopen the PR (yes, you can do that), announce that the script was busted a= nd ask what the reason for the pkg-plist change was. Once that was resolved, then you should have opened this PR to a simple ver= sion upgrade. In fact, you can still do that. I am going to assign this PR to the guy that did that last commit, with the recommendation that he process this PR. Assuming that he has good rationale for the pkg-plist change and it works, then I would recommend that this PR = only fix the script and do the update. --=20 You are receiving this mail because: You are the assignee for the bug.=