Jeff Trawick wrote:
> 
> This is the same fix which was applied to APR prior to Apache 2.0.40's
> release.  With 2.0, the problem was first reported on HP-UX/Itanium.
> With 1.3, the problem was first reported on Linux/zSeries.
> 
> I also removed a bogus comment.  In fact, it is normal for union semun
> to not be defined in the header files for user applications.
> 
> Before I committed Madhu's fix to APR, I tried in vain to find any
> doc which said that the first field should be long instead of int.
> With this change to Apache 1.3, SysV sems still work and as far as I
> can tell the world did not come to an end, so I'd like to commit this
> before 1.3.27 is rolled.

+1...this seems safe.  

I don't understand why the OSes make us roll our own header, but maybe there was
a good reason for it once upon a time.

Greg

Reply via email to