Suggestion for 'pkg_add -r'

[Anders Nore]

On Thu, 07 Aug 2008 22:40:35 +0200, Lars Engels <lars.engels at 0x20.net>  
wrote:

> On Thu, Aug 07, 2008 at 03:47:24PM +0200, Anders Nore wrote:
>> >>
>> >>RELENG_7:
>> >>http://home.no.net/andenore/patches/pkg_install_2008-08-06_RELENG_7.diff
>> >>
>> >>CURRENT:
>> >>http://home.no.net/andenore/patches/pkg_install_2008-08-06_CURRENT.diff
>> >
>> >Some comments:
>> >   * I think you have reversed the patch. :-)
>> >   * Build errors:
>> >cc1: warnings being treated as errors
>> >file.c:433: warning: no previous prototype for 'power'
>> >file.c:452: warning: no previous prototype for 'human_readable'
>> >file.c:474: warning: no previous prototype for 'printHumanReadable'
>> >file.c: In function 'printHumanReadable':
>> >file.c:482: warning: comparison between signed and unsigned
>> >
>> >parallels# ./pkg_add -r joe
>> >Fetching 321.2 kB from
>> >ftp://ftp.freebsd.org/pub/FreeBSD/ports/i386/packages-8-current/Latest/joe.tbz...
>> >                                       Downloading: 100%  321.2 kB at   
>> 214.8 kB/s
>> >                                                             Done
>> >Fetching 2.4 MB from
>> >ftp://ftp.freebsd.org/pub/FreeBSD/ports/i386/packages-8-current/All/gettext-0.17_1.tbz...
>> >                                                                              
>> Done
>> >Fetching 2.2 MB from
>> >ftp://ftp.freebsd.org/pub/FreeBSD/ports/i386/packages-8-current/All/aspell-0.60.6_2.tbz...
>> >                                       Downloading: 201%    4.5 MB at   
>> 149.5 kB/s
>> >
>> >Something's wrong :-)
>> >
>>
>> Heh, yes. It's my first patch(es), so it didn't go as well as I had  
>> hoped, I believe they are
>> fixed now.
>
> I just downloaded the patch and still get the warning:
> /usr/src/usr.sbin/pkg_install/lib/file.c: In function
> 'printHumanReadable':
> /usr/src/usr.sbin/pkg_install/lib/file.c:486: warning: comparison
> between signed and unsigned
> *** Error code 1
>
> Stop in /usr/src/usr.sbin/pkg_install/lib.
>

Hey, it should be fixed now. I don't know why this causes a stop in the  
compiling though?


Thanks,
Anders Nore