<html><body><div style="color:#000; background-color:#fff; font-family:Courier New, courier, monaco, monospace, sans-serif;font-size:10pt">Alexander,<div><span><br></span></div><div>I did an initial review of your code. Looks good overall.<br></div><div><br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;">One thing:<br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;">1. Bump PHP to 5.4.17</div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;"><br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color:
 transparent; font-style: normal;">This can help improve the reported issues with connectivity issues between the SQL Server 2008/MySQL/and PostGresSQL database components. </div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;"><br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;">~ Ken Mays<br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;"><br></div><div style="color: rgb(0, 0, 0); font-size: 13.3333px; font-family: Courier New,courier,monaco,monospace,sans-serif; background-color: transparent; font-style: normal;"><br></div>  <div style="font-family: Courier New, courier, monaco,
 monospace, sans-serif; font-size: 10pt;"> <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <div dir="ltr"> <hr size="1">  <font face="Arial" size="2"> <b><span style="font-weight:bold;">From:</span></b> Alexander Pyhalov <alp@rsu.ru><br> <b><span style="font-weight: bold;">To:</span></b> OpenIndiana Developer mailing list <oi-dev@openindiana.org> <br> <b><span style="font-weight: bold;">Sent:</span></b> Monday, July 8, 2013 4:10 AM<br> <b><span style="font-weight: bold;">Subject:</span></b> [oi-dev] Please review php 54<br> </font> </div> <div class="y_msg_container"><br>Hello.<br>I have the following problem.<br>I prepared php-54 component: <br><a href="https://github.com/pyhalov/oi-userland/commit/5ca1d7cd32cead580ad5d0a5c1b3eef082803390" target="_blank">https://github.com/pyhalov/oi-userland/commit/5ca1d7cd32cead580ad5d0a5c1b3eef082803390</a><br>and need your review.<br>It is not complete yet. At least
 it misses MySQL/PostgreSQL connectivity <br>modules.<br>However, to build modules I need php-config available on build machine.<br><br>One more trouble - it is incompatible with php 5.2, because of <br>/usr/php/* links. php 52 delivers them unconditionally, new module has a <br>mediator on them.<br>Does build machine need php 5.2 to function correctly? Can it be <br>uninstalled? Or, better, can we resolve this conflict making php 5.2 <br>links mediator-based?<br>-- <br>Best regards,<br>Alexander Pyhalov,<br>system administrator of Computer Center of Southern Federal University<br><br>_______________________________________________<br>oi-dev mailing list<br><a ymailto="mailto:oi-dev@openindiana.org" href="mailto:oi-dev@openindiana.org">oi-dev@openindiana.org</a><br><a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/mailman/listinfo/oi-dev</a><br><br><br></div> </div> </div>  </div></body></html>