You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Blair Zajac <bl...@orcaware.com> on 2010/05/19 23:06:05 UTC
Re: svn commit: r946475 - /subversion/trunk/tools/dev/unix-build/Makefile.svn
On 05/19/2010 03:58 PM, stsp@apache.org wrote:
> Author: stsp
> Date: Wed May 19 22:58:37 2010
> New Revision: 946475
>
> URL: http://svn.apache.org/viewvc?rev=946475&view=rev
> Log:
> * tools/dev/unix-build/Makefile.svn: Fix a comment typo. Set LD_LIBRARY_PATH
> when running tests for perl and python bindings.
>
> svn-check-swig-pl:
> - cd $(svn_builddir)&& make check-swig-pl
> + cd $(svn_builddir)&& \
> + env LD_LIBRARY_PATH=$(LD_LIBRARY_PATH) make check-swig-pl
I don't think you need the "env" here, isn't using
LD_LIBRARY_PATH=$(LD_LIBRARY_PATH) sufficient, since our Makefile's use
Bourne shell by default.
Regards,
Blair
Re: svn commit: r946475 -
/subversion/trunk/tools/dev/unix-build/Makefile.svn
Posted by Stefan Sperling <st...@elego.de>.
On Wed, May 19, 2010 at 04:06:05PM -0700, Blair Zajac wrote:
> On 05/19/2010 03:58 PM, stsp@apache.org wrote:
> >Author: stsp
> >Date: Wed May 19 22:58:37 2010
> >New Revision: 946475
> >
> >URL: http://svn.apache.org/viewvc?rev=946475&view=rev
> >Log:
> >* tools/dev/unix-build/Makefile.svn: Fix a comment typo. Set LD_LIBRARY_PATH
> > when running tests for perl and python bindings.
> >
>
> > svn-check-swig-pl:
> >- cd $(svn_builddir)&& make check-swig-pl
> >+ cd $(svn_builddir)&& \
> >+ env LD_LIBRARY_PATH=$(LD_LIBRARY_PATH) make check-swig-pl
>
> I don't think you need the "env" here, isn't using
> LD_LIBRARY_PATH=$(LD_LIBRARY_PATH) sufficient, since our Makefile's
> use Bourne shell by default.
It possibly is. I'm using env in a couple of other places in this
file though. I don't think it hurts.
Note that this isn't hooked up to our standard makefiles at all.
It's just a wrapper that fetches dependencies, builds svn, can
run multi-way tests and some other things. It may be worth lifting
some functionality from it into our standard Makefile eventually.
Basically I use it to build my own development builds, that's all.
Thanks,
Stefan