[Rivet] Problem with --disable-analyses --> Solution included!

[Andy Buckley]

Thanks David, I'll incorporate this into the next v2 release. Should be
soon.

On 05/12/13 17:40, David Hall wrote:
> Hello,
> 
> I found that the rivet build failed when configured with
> `—disable-analyses`, producing the error:
> (the solution can be viewed with nicer formatting
> at https://github.com/davidchall/homebrew-hep/issues/13)
> 
> |No rule to make target `Analyses/libRivetAnalysisTools.la', needed by `libRivet.la'.|
> 
> 
> To fix this, I had to edit |src/Makefile.am|
> 
> |libRivet_la_LIBADD = \
>   Core/libRivetCore.la \
>   Projections/libRivetProjections.la \
>   Tools/libRivetTools.la \
>   Analyses/libRivetAnalysisTools.la \
>   -ldl -lm -lYODA -lHepMC \
>   $(GSL_LDFLAGS) $(FASTJETCONFIGLIBADD)
> |
> 
> to become
> 
> |libRivet_la_LIBADD = \
>   Core/libRivetCore.la \
>   Projections/libRivetProjections.la \
>   Tools/libRivetTools.la
> if ENABLE_ANALYSES
> libRivet_la_LIBADD += Analyses/libRivetAnalysisTools.la
> endif
> libRivet_la_LIBADD += \
>   -ldl -lm -lYODA -lHepMC \
>   $(GSL_LDFLAGS) $(FASTJETCONFIGLIBADD)
> |
> 
> This then meant that one of the tests in |make check| failed since it
> relied on an analysis. So I had to edit|test/Makefile.am|
> 
> |TESTS = testMath testMatVec testBoost testCmp testApi testNaN testCmdLine.sh
> 
> EXTRA_DIST = testApi.hepmc testCmdLine.sh
> |
> 
> to become
> 
> |TESTS = testMath testMatVec testBoost testCmp testApi testNaN
> if ENABLE_ANALYSES
> TESTS += testCmdLine.sh
> endif
> 
> EXTRA_DIST = testApi.hepmc
> if ENABLE_ANALYSES
> EXTRA_DIST += testCmdLine.sh
> endif|
> 
> 
> 
> I hope this is helpful to you, and that I haven’t misunderstood something.
> 
> Cheers,
> David
> 
> 
> 
> _______________________________________________
> Rivet mailing list
> Rivet at projects.hepforge.org
> http://www.hepforge.org/lists/listinfo/rivet
> 


-- 
Dr Andy Buckley, Royal Society University Research Fellow
Particle Physics Expt Group, University of Glasgow / PH Dept, CERN