Garrett D'Amore wrote:
> This looks like a potentially handy feature, and the architecture seems 
> fairly straight-forward.  However, I'm not sure it qualifies for 
> self-review.  (Although the bar for self review seems to be moving 
> somewhat...)  If only because it introduces new interfaces and behavior 
> for consumers.
> 
> I'd suggest that this maybe should be a fast-track instead of self 
> review, unless there is some particular rush to get this case through?  
> Please consider promoting it to a fast track.  In any case, I'll happily 
> give it a +1.
> 
> Btw, you failed to specify a release binding.  Is this patch or minor 
> (or something else)?  (It looks like this could be Patch binding fairly 
> easily...)
> 
>    - Garrett

I'm not opposed to making it a fast track. However, I'd like to ask you to
reconsider. It's a trivial option being implemented purely for Linux
compatibility. Software we care about (Xorg) uses it, which is the sole
reason for introducing it. It seems well below the line to me...

I did specify the release binding, in the middle:

 >> -----
 >>
 >> Release Binding:                Patch/Micro
 >>
 >> New ld options:
 >>     -z wrap                    Committed
 >>     -wrap                    Committed
 >>     --wrap                    Committed
 >>
 >> -----

Thanks...

- Ali

Reply via email to