NetBSD Problem Report #49724

From jdwhite@iastate.edu  Thu Mar  5 23:00:38 2015
Return-Path: <jdwhite@iastate.edu>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id D4BF9A6553
	for <gnats-bugs@gnats.netbsd.org>; Thu,  5 Mar 2015 23:00:38 +0000 (UTC)
Message-Id: <20150305215958.A809C143B6911@starbug.its.iastate.edu>
Date: Thu,  5 Mar 2015 15:59:57 -0600 (CST)
From: jdwhite@iastate.edu
Reply-To:
To: gnats-bugs@gnats.netbsd.org
Cc:
Subject: pkgtools/rc.subr needs tweak to support Darwin's echo
X-Send-Pr-Version: 3.113.1
X-GNATS-Notify:

>Number:         49724
>Notify-List:    sbd@netbsd.org, macos-pkg-people@netbsd.org
>Category:       pkg
>Synopsis:       pkgtools/rc.subr needs tweak to support Darwin's echo
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bsiegert
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Mar 05 23:05:00 +0000 2015
>Closed-Date:    Mon May 11 12:55:57 +0000 2015
>Last-Modified:  Mon May 11 12:55:57 +0000 2015
>Originator:     Jason White
>Release:        Darwin 14.1.0
>Organization:
>Environment:
System: Darwin starbug.its.iastate.edu 14.1.0 Darwin Kernel Version 14.1.0: Mon Dec 22 23:10:38 PST 2014; root:xnu-2782.10.72~2/RELEASE_X86_64 x86_64
>Description:
The built-in echo in the stock darwin shell doesn not support the '-n' flag to echo. This causes certain pkgsrc commands to fail or contain visual artifacts.

# /etc/rc.d/xymonclient start
Starting xymonclient.
# /etc/rc.d/xymonclient status
xymonclient is running as pid -n 1275.
# /etc/rc.d/xymonclient stop
Stopping xymonclient.
kill: illegal process id: -n

>How-To-Repeat:
Install a package under Darwin that's controlled with an /etc/rc.d/ script and try it with 'status' and 'stop' arguments.

>Fix:
Force rc.subr to use /bin/echo binary.

--- pkgtools/rc.subr/files/rc.subr.orig 2015-03-05 15:37:52.000000000 -0600
+++ pkgtools/rc.subr/files/rc.subr      2015-03-05 15:39:39.000000000 -0600
@@ -67,6 +67,9 @@
 _osname=$(uname -s)
 _osrelease=$(uname -r)
 case $_osname in
+       Darwin)
+               _RCCMD_echo="/bin/echo"
+               ;;
        SunOS)
                _RCCMD_chown="/bin/chown"
 #              _RCCMD_ci="/usr/bin/ci"                 # not in Solaris 9

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->bsiegert
Responsible-Changed-By: bsiegert@NetBSD.org
Responsible-Changed-When: Sun, 10 May 2015 19:14:34 +0000
Responsible-Changed-Why:
Taking this.


From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/49724 CVS commit: pkgsrc/pkgtools/rc.subr
Date: Sun, 10 May 2015 19:20:09 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Sun May 10 19:20:09 UTC 2015

 Modified Files:
 	pkgsrc/pkgtools/rc.subr: Makefile
 	pkgsrc/pkgtools/rc.subr/files: rc.subr

 Log Message:
 Use /bin/echo on Darwin, as the builtin does not have the -n option.
 Bump version.
 Patch from Jason White in PR pkg/49724.


 To generate a diff of this commit:
 cvs rdiff -u -r1.28 -r1.29 pkgsrc/pkgtools/rc.subr/Makefile
 cvs rdiff -u -r1.11 -r1.12 pkgsrc/pkgtools/rc.subr/files/rc.subr

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/49724 CVS commit: pkgsrc/doc
Date: Sun, 10 May 2015 19:21:38 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Sun May 10 19:21:38 UTC 2015

 Modified Files:
 	pkgsrc/doc: CHANGES-2015

 Log Message:
 rc.subr update, PR pkg/49724


 To generate a diff of this commit:
 cvs rdiff -u -r1.2202 -r1.2203 pkgsrc/doc/CHANGES-2015

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->closed
State-Changed-By: bsiegert@NetBSD.org
State-Changed-When: Mon, 11 May 2015 12:55:57 +0000
State-Changed-Why:
Patch committed, thanks!


>Unformatted:

Home	PR Database Search