Subject: pkg/34943: removing compiler warnings in pkgsrc/pkgtools/pkg_install
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: Eric Schnoebelen <eric@cirr.com>
List: pkgsrc-bugs
Date: 10/29/2006 22:15:00
>Number: 34943
>Category: pkg
>Synopsis: remove compiler warnings in pkgsrc/pkgtools/pkg_install
>Confidential: no
>Severity: non-critical
>Priority: low
>Responsible: pkg-manager
>State: open
>Class: sw-bug
>Submitter-Id: net
>Arrival-Date: Sun Oct 29 22:15:00 +0000 2006
>Originator: Eric Schnoebelen
>Release: HP-UX B.11.11
>Organization:
Eric Schnoebelen eric@cirr.com http://www.cirr.com
"The nice thing about Windows is - It does not just crash, it
displays a dialog box and lets you press 'OK' first." - Arno Schaefer
>Environment:
HP-UX peake B.11.11 A 9000/785 2006709503 two-user license
B3899BA B.11.11.06 HP C/ANSI C Developer's Bundle for HP-UX 11.i (S700)
B3911DB C.03.37.01 HP aC++ Compiler (S700)
FEATURE11-11 B.11.11.0209.5 Feature Enablement Patches for HP-UX 11i, Sept 2002
HPUX11i-TCOE B.11.11.0306 HP-UX Technical Computing OE Component
HPUXBase64 B.11.11 HP-UX 64-bit Base OS
HPUXBaseAux B.11.11.0312 HP-UX Base OS Auxiliary
HWEnable11i B.11.11.0312.4 Hardware Enablement Patches for HP-UX 11i v1, December 2003
[ok, the above is only really useful/interesting to an HP-UX weenie, or
the HP WTEC.]
>Description:
Several source files declare function prototypes as static
at the top, and then fail to repeat the static keyword when declaring
the actual function.
>How-To-Repeat:
compile with a very picky compiler, such as HP ANSI C.
>Fix:
Index: files/delete/perform.c
===================================================================
RCS file: /cvsroot/pkgsrc/pkgtools/pkg_install/files/delete/perform.c,v
retrieving revision 1.16
diff -b -u -w -r1.16 perform.c
--- files/delete/perform.c 30 May 2005 13:23:31 -0000 1.16
+++ files/delete/perform.c 29 Oct 2006 22:10:32 -0000
@@ -263,7 +263,7 @@
* Delete from directory 'home' all packages on lpkg_list.
* If tryall is set, ignore errors from pkg_delete(1).
*/
-int
+static int
require_delete(char *home, int tryall)
{
lpkg_t *lpp;
@@ -362,7 +362,7 @@
* Recursively find all packages "up" the tree (follow +REQUIRED_BY).
* Return 1 on errors
*/
-int
+static int
require_find_recursive_up(lpkg_t *thislpp)
{
lpkg_head_t reqq;
@@ -441,7 +441,7 @@
* Recursively find all packages "down" the tree (follow @pkgdep).
* Return 1 on errors
*/
-int
+static int
require_find_recursive_down(lpkg_t *thislpp, package_t *plist)
{
plist_t *p;
@@ -562,7 +562,7 @@
/*
* Start recursion in the one or other direction.
*/
-int
+static int
require_find(char *pkg, rec_find_t updown)
{
lpkg_t *lpp;
@@ -585,7 +585,7 @@
return (rv);
}
-void
+static void
require_print(void)
{
lpkg_t *lpp;