<div dir="ltr">It seems like the easiest solution to the DESTDIR problem is just to append the directory to the PKG_CONFIG_PATH in Makefile.am, rather than replacing it, as is done in the patch, like so.<br><div><br>if COIN_HAS_PKGCONFIG<br> PKG_CONFIG_PATH="@COIN_PKG_CONFIG_PATH@":$(DESTDIR)$(pkgconfiglibdir) \<br> "$(PKG_CONFIG)" --libs coinutils > $(addlibsdir)/coinutils_addlibs.txt<br>else<br>...<br></div><div>endif<br><br></div><div>This should not break anything that currently works and I think will fix the problem (i.e., patches should no longer be required), but it occurs to me that it may not be the most elegant solution. Any thoughts?<br><br></div><div>Note that this actually does not require any changes to the build system itself. Since it seems DESTDIR is specified as an argument to make, not to configure, this doesn't seem to be an issue with coin.m4 and won't require any new releases of BuildTools. I guess we could have fixed it a long time ago :). Unfortunately, we do have to modify Makefile.am for every project individually,<br><br></div><div>Cheers,<br><br>Ted<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 20, 2015 at 4:42 PM, Ted Ralphs <span dir="ltr"><<a href="mailto:ted@lehigh.edu" target="_blank">ted@lehigh.edu</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">I tend to agree with Stefan about changing the default behavior of --with-pic. If we can get ASL fixed up so it pays attention to this argument, that should be enough for me.<div><br></div><div>I will try to implement something to address the patches on Debian and Fedora in light of Stefan's comments, although the situations in which things might break are not ones we ordinarily encounter in the typical build process so testing will take some creativity. </div><div><br></div><div>Cheers,</div><div><br></div><div>Ted</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 20, 2015 at 12:48 PM, Miles Lubin <span dir="ltr"><<a href="mailto:miles.lubin@gmail.com" target="_blank">miles.lubin@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 Tony,<div><br></div><div>Yes, the debian patches are identical. This is the only issue that we currently have to patch, so I'd be quite happy to not need any patches at all.</div><span><font color="#888888"><div><br></div><div>Miles</div></font></span></div><div class="gmail_extra"><br><div class="gmail_quote"><span>On Mon, Jan 19, 2015 at 1:47 AM, Tony Kelman <span dir="ltr"><<a href="mailto:kelman@berkeley.edu" target="_blank">kelman@berkeley.edu</a>></span> wrote:<br></span><div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">
<div dir="ltr">
<div style="FONT-SIZE:12pt;FONT-FAMILY:'Calibri';COLOR:#000000">
<div>Ted et al,</div>
<div> </div>
<div>Looks like I just missed the boat on the .0 releases of the new stables. I
wish I had remembered slightly earlier, but this would have been a good time to
try to resolve <a href="https://projects.coin-or.org/BuildTools/ticket/104" target="_blank">https://projects.coin-or.org/BuildTools/ticket/104</a>.
Debian and Fedora have to carry local patches to make installing with DESTDIR
work, the patches themselves are pretty simple. Here are Fedora’s, hopefully
Miles can tell us whether or not Debian’s look exactly the same.</div>
<div> </div>
<div><a href="http://pkgs.fedoraproject.org/cgit/coin-or-CoinUtils.git/tree/coin-or-CoinUtils-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-CoinUtils.git/tree/coin-or-CoinUtils-pkgconfig.patch</a></div>
<div><a title="http://pkgs.fedoraproject.org/cgit/coin-or-Osi.git/tree/coin-or-Osi-pkgconfig.patch" href="http://pkgs.fedoraproject.org/cgit/coin-or-Osi.git/tree/coin-or-Osi-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-Osi.git/tree/coin-or-Osi-pkgconfig.patch</a></div>
<div><a title="http://pkgs.fedoraproject.org/cgit/coin-or-Clp.git/tree/coin-or-Clp-pkgconfig.patch" href="http://pkgs.fedoraproject.org/cgit/coin-or-Clp.git/tree/coin-or-Clp-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-Clp.git/tree/coin-or-Clp-pkgconfig.patch</a></div>
<div><a title="http://pkgs.fedoraproject.org/cgit/coin-or-Cgl.git/tree/coin-or-Cgl-pkgconfig.patch" href="http://pkgs.fedoraproject.org/cgit/coin-or-Cgl.git/tree/coin-or-Cgl-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-Cgl.git/tree/coin-or-Cgl-pkgconfig.patch</a></div>
<div><a title="http://pkgs.fedoraproject.org/cgit/coin-or-Cbc.git/tree/coin-or-Cbc-pkgconfig.patch" href="http://pkgs.fedoraproject.org/cgit/coin-or-Cbc.git/tree/coin-or-Cbc-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-Cbc.git/tree/coin-or-Cbc-pkgconfig.patch</a></div>
<div><a title="http://pkgs.fedoraproject.org/cgit/coin-or-Ipopt.git/tree/coin-or-Ipopt-pkgconfig.patch" href="http://pkgs.fedoraproject.org/cgit/coin-or-Ipopt.git/tree/coin-or-Ipopt-pkgconfig.patch" target="_blank">http://pkgs.fedoraproject.org/cgit/coin-or-Ipopt.git/tree/coin-or-Ipopt-pkgconfig.patch</a></div>
<div>I believe Couenne and OS need the same but they aren’t packaged in Fedora
yet.</div>
<div> </div>
<div>Obviously we’d need to apply the same changes to the corresponding
Makefile.am files as well.</div>
<div> </div>
<div>-Tony</div>
<div> </div>
<div style="FONT-SIZE:small;TEXT-DECORATION:none;FONT-FAMILY:"Calibri";FONT-WEIGHT:normal;COLOR:#000000;FONT-STYLE:normal;DISPLAY:inline">
<div style="FONT:10pt tahoma">
<div> </div>
<div style="BACKGROUND:#f5f5f5">
<div><b>From:</b> <a title="ted@lehigh.edu" href="mailto:ted@lehigh.edu" target="_blank">Ted Ralphs</a> </div>
<div><b>Sent:</b> Friday, January 16, 2015 8:17 AM</div>
<div><b>To:</b> <a title="buildtools@list.coin-or.org" href="mailto:buildtools@list.coin-or.org" target="_blank">Buildtools</a> </div>
<div><b>Cc:</b> <a title="coin-tlc@list.coin-or.org" href="mailto:coin-tlc@list.coin-or.org" target="_blank">coin-tlc</a> </div>
<div><b>Subject:</b> [Coin-tlc] Recent changes</div></div></div>
<div> </div></div>
<div style="FONT-SIZE:small;TEXT-DECORATION:none;FONT-FAMILY:"Calibri";FONT-WEIGHT:normal;COLOR:#000000;FONT-STYLE:normal;DISPLAY:inline"><div><div>
<div dir="ltr">Hi all,
<div> </div>
<div>I recently made several changes to the build tools that I wanted to make
you aware of.</div>
<div> </div>
<div>1. I made release 0.7.18 that changes from the use of the vecLib framework
to the use of the Accelerate framework on OS X. The vecLib framework is no
longer included in OS X as of 10.9.</div>
<div> </div>
<div>2. I made a new stable version 0.8 and changed dependency linking to be the
default in all configurations except with the Visual Studio compilers.</div>
<div> </div>
<div>I am planning to make a new release 0.8.0 shortly and then will begin
switching trunks of all projects that use the BuildTools to point to stable/0.8.
Is there anything else that should be changed in stable/0.8 before we make the
first release?</div>
<div> </div>
<div>After switching trunks of all projects, I will begin making new stables and
releases for all projects. This will eventually be followed by a new stable
version 1.8 of CoinAll.</div>
<div> </div>
<div>Cheers,</div>
<div> </div>
<div>Ted<br clear="all">
<div>-- <br></div>
<div>
<div dir="ltr">Dr. Ted Ralphs<br>Professor, Lehigh University<br>(610)
628-1280<br>ted 'at' lehigh 'dot' edu<br><a href="http://coral.ie.lehigh.edu/~ted" target="_blank">coral.ie.lehigh.edu/~ted</a><br></div></div></div></div>
</div></div><p>
</p><hr>
_______________________________________________<br>Coin-tlc mailing
list<br><a href="mailto:Coin-tlc@list.coin-or.org" target="_blank">Coin-tlc@list.coin-or.org</a><br><a href="http://list.coin-or.org/mailman/listinfo/coin-tlc" target="_blank">http://list.coin-or.org/mailman/listinfo/coin-tlc</a><br><p></p></div></div></div></div>
<br>_______________________________________________<br>
Coin-tlc mailing list<br>
<a href="mailto:Coin-tlc@list.coin-or.org" target="_blank">Coin-tlc@list.coin-or.org</a><br>
<a href="http://list.coin-or.org/mailman/listinfo/coin-tlc" target="_blank">http://list.coin-or.org/mailman/listinfo/coin-tlc</a><br>
<br></blockquote></div></div></div><br></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">Dr. Ted Ralphs<br>Professor, Lehigh University<br><a href="tel:%28610%29%20628-1280" value="+16106281280" target="_blank">(610) 628-1280</a><br>ted 'at' lehigh 'dot' edu<br><a href="http://coral.ie.lehigh.edu/~ted" target="_blank">coral.ie.lehigh.edu/~ted</a><br></div></div>
</div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">Dr. Ted Ralphs<br>Professor, Lehigh University<br>(610) 628-1280<br>ted 'at' lehigh 'dot' edu<br><a href="http://coral.ie.lehigh.edu/~ted" target="_blank">coral.ie.lehigh.edu/~ted</a><br></div></div>
</div>