Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 22 Apr 2003 22:06:50 -0500
From:      David Leimbach <leimy2k@mac.com>
To:        "Matthew Emmerton" <matt@gsicomp.on.ca>
Cc:        freebsd-current@freebsd.org
Subject:   Re: Makefile in /usr/src/lib/libdisk WARNS change? PLEASE READ!
Message-ID:  <A091285D-7538-11D7-BA46-0003937E39E0@mac.com>
In-Reply-To: <007301c30944$c050e920$1200a8c0@gsicomp.on.ca>

next in thread | previous in thread | raw e-mail | index | archive | help

On Tuesday, April 22, 2003, at 10:02 PM, Matthew Emmerton wrote:

>>> Hi,
>>>
>>> I posted the libdisk patch that was heavily ignored two days ago :).
>>> Without it I
>>> couldn't build libdisk and subsequently world.  If I lower the WARNS
>>> value
>>> in the libdisk directory Makefiel the source builds without warning 
>>> and
>>> world
>>> completes.
>>>
>>> The problem is rules.c having a switch/case that handles only 2 of 7
>>> cases for the
>>> platform value.  Adding a simple "default:" [as my patch did] will
>>> repair this problem.
>>>
>>> I should tell you that I clean-installed FBSD 5.0-RELEASE on Saturday
>>> 4/19/2003
>>> and started playing with CURRENT again the very next day at which 
>>> point
>>> I realized
>>> world wouldn't build.  I didn't do much else so I can only assume 
>>> that
>>> somehow this
>>> could happen to anyone else starting from RELEASE-5 today and
>>> attempting to build
>>> world.
>>>
>>> I have the latest version of the rules.c file [1.29]. I checked 
>>> CVSWEB
>>> and it has
>>> the exact same code that fails for me in HEAD so I know its not my
>>> source.  The
>>> only data involved in the switch is an enum type from libdisk.h 
>>> which I
>>> also
>>> have the latest version of [1.55].
>>>
>>> The error is
>>> /usr/src/lib/libdisk/rules.c:264: warning: enumeration value `p_any'
>>> not handled in switch
>>> /usr/src/lib/libdisk/rules.c:264: warning: enumeration value 
>>> `p_alpha'
>>> not handled in switch
>>> /usr/src/lib/libdisk/rules.c:264: warning: enumeration value
>>> `p_sparc64' not handled in switch
>>> /usr/src/lib/libdisk/rules.c:264: warning: enumeration value `p_ia64'
>>> not handled in switch
>>> /usr/src/lib/libdisk/rules.c:264: warning: enumeration value `p_ppc'
>>> not handled in switch
>>> *** Error code 1.
>>>
>>> I know this isn't a bug in the VM system.  Its not solving a kernel
>>> panic or
>>> some other serious commit but it does pose a barrier to those who 
>>> want
>>> to test CURRENT.
>>>
>>> Should I file a problem report?
>>>
>>> Dave Leimbach
>>
>> I know how you feel, boss. I just started a thread about how my 
>> computer
>> dies if I run out of memory, and all I got was a "me too" from another
>> fellow user. No developers jumped in. It seems like unless you have a 
>> LOR
> or
>> a laptop, you get no attention. And no, don't bother filing a bug 
>> report
>> because it will stagnate further.
>
> Please ignore the above comment.  The poster does not have the best
> interests of FreeBSD in mind.
> Remember that most (if not all) FreeBSD contributors are unpaid 
> volunteers,
> and their day jobs take precendence.
> It just happens that the people looking after LOR and ACPI issues have 
> more
> time on their hands than others.
>
> I've cc'd the committer of the last change to libdisk.h; this should 
> capture
> their attention and get this looked into more expediently.
>

I sent to PHK who dealt with rules.c so please don't annoy him more 
than I might
just have with my slightly desperate sounding email :).

I didn't want to get dramatic about this since its such a tiny issue... 
I just didn't
want to fork "LeimyBSD" over this simple fix :).

I am not worried about it... None of the FreeBSD developers owe me a 
goddamned thing.
In fact the only reason I report the bug is because I want to make sure 
it doesn't
happen to someone else :).

Dave


> --
> Matt Emmerton
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?A091285D-7538-11D7-BA46-0003937E39E0>