<div dir="ltr">Symbol visibility reduction is definitely a good thing. I wouldn't bother too much removing the versioning, but it's going to be more work to keep it up to date. If you don't plan on keeping ABI retrocompatibility on the long term I would suggest to roll this back and go with just <a href="https://www.flameeyes.eu/autotools-mythbuster/libtool/symbols.html">symbol visibility through libtool</a>.</div>
<div class="gmail_extra"><br clear="all"><div>Diego Elio Pettenò — Flameeyes<br><a href="mailto:flameeyes@flameeyes.eu" target="_blank">flameeyes@flameeyes.eu</a> — <a href="http://blog.flameeyes.eu/" target="_blank">http://blog.flameeyes.eu/</a></div>

<br><br><div class="gmail_quote">On 3 September 2014 23:34, Sean McGovern <span dir="ltr"><<a href="mailto:gseanmcg@gmail.com" target="_blank">gseanmcg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi Petri & Diego,<div><div class="h5"><br><div class="gmail_extra"><div class="gmail_quote"><br></div><div class="gmail_quote">On Wed, Sep 3, 2014 at 3:48 PM, Diego Elio Pettenò <span dir="ltr"><<a href="mailto:flameeyes@flameeyes.eu" target="_blank">flameeyes@flameeyes.eu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">Please note that symbol versioning only makes sense if you plan to *replace the ABI of a function in a later version*. If you do that, then no, building against a new version and trying to run it against an old version will not work.</div>


<div class="gmail_extra"><br></div><div class="gmail_extra">If you don't plan to make changes like that, symbol versioning is totally useless.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also, please don't make it optional, but only conditional on the OS, because mixing builds with and without versioning across distributions would jsut be a huge headache for no gain.</div>


<div class="gmail_extra"><br></div></div>
</blockquote></div><br></div></div></div><div class="gmail_extra">With these concerns, maybe instead of symbol versioning I could/should just convert this to a symbol visibility reduction script? Or should I just discard my patch entirely?</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">-- Sean McG.</div></div>
<br>_______________________________________________<br>
libbluray-devel mailing list<br>
<a href="mailto:libbluray-devel@videolan.org">libbluray-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/libbluray-devel" target="_blank">https://mailman.videolan.org/listinfo/libbluray-devel</a><br>
<br></blockquote></div><br></div>