FreeBSD 5.3 and Adaptec raidutils (again)

[Scott Long]

Matt Rowley wrote:
>>> Yes; you can work around it by declaring a temp variable, assigning it
>>> the value of attachedTo, making whatever modification is necessary,
>>> then assigning attachedTo=temp.  Do this every time you get that error.
>>> You /might/ also be able to just remove the PACKed attribute from the
>>> attachedTo field, but that will cause havoc if the struct is supposed
>>> to line up with something generated by the card.
>>>
>>
>> I'd highly recommend against removing the packed attribute.
> 
> 
> :)  It does compile, when you remove packed.  After commenting out the 
> unneeded semaphore union struct in basic.hh, the whole thing compiles.  
> The resulting raidutil binary spews out the same error as the one from 
> the current binary port about "Engine connect failed: COMPATIBILITY 
> number"... but that's to be expected.
> 
> Scott, is Dan's suggestion about using a temp variable and to stop 
> passing by reference the right way to go?
> 
> --Matt

I agree with Dan's temp variable solution.  I'm also open to discussion
on the compatibility number thing.  If you can make it work with the
value provided by the current driver, then please do.  Otherwise we can
discuss reverting the value.

Scott