From owner-cvs-lib Sun Mar 30 13:30:11 1997 Return-Path: Received: (from root@localhost) by freefall.freebsd.org (8.8.5/8.8.5) id NAA14028 for cvs-lib-outgoing; Sun, 30 Mar 1997 13:30:11 -0800 (PST) Received: from sovcom.kiae.su (sovcom.kiae.su [193.125.152.1]) by freefall.freebsd.org (8.8.5/8.8.5) with SMTP id NAA13963; Sun, 30 Mar 1997 13:29:59 -0800 (PST) Received: by sovcom.kiae.su id AA12768 (5.65.kiae-1 ); Mon, 31 Mar 1997 00:20:37 +0300 Received: by sovcom.KIAE.su (UUMAIL/2.0); Mon, 31 Mar 97 00:20:36 +0300 Received: (from ache@localhost) by nagual.ru (8.8.5/8.8.5) id BAA00348; Mon, 31 Mar 1997 01:18:17 +0400 (MSD) Date: Mon, 31 Mar 1997 01:18:09 +0400 (MSD) From: =?KOI8-R?B?4c7E0sXKIP7F0s7P1w==?= To: Brian Somers Cc: CVS-committers@freefall.freebsd.org, cvs-all@freefall.freebsd.org, cvs-lib@freefall.freebsd.org, cvs-usrbin@freefall.freebsd.org, cvs-sbin@freefall.freebsd.org, cvs-usrsbin@freefall.freebsd.org Subject: Re: cvs commit: src/sbin/slattach Makefile src/usr.bin/tip/tip Makefile src/usr.sbin/ppp Makefile src/lib/libutil uucplock.3 uucplock.c Makefile libutil.h src/sbin/startslip Makefile uucplock.c In-Reply-To: <199703301212.EAA21994@freefall.freebsd.org> Message-Id: Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-lib@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk On Sun, 30 Mar 1997, Brian Somers wrote: > Added: lib/libutil uucplock.3 uucplock.c > Removed: sbin/startslip uucplock.c > Log: > Move uucplock into libutil and create a manual page. What you do with USE_PERROR define which switch uucplock error loging from perror to syslog? Startslip uses syslog and ppp not use it. It seems that you just broke ppp since you left USE_PERROR define in its Makefile which unused now. Moreover, having library function with syslog report is bad idea in any case. To fix it properly you need to change return from uucplock functions from int to char *, i.e. return NULL for no errors or static error string if error happens. Then you need to change upper level programs to pass this error string to perror/syslog/whatever. -- Andrey A. Chernov http://www.nagual.ru/~ache/