Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

CXXFLAGS in Makefile is not used #384

Closed
ariostas opened this issue Apr 3, 2024 · 1 comment
Closed

CXXFLAGS in Makefile is not used #384

ariostas opened this issue Apr 3, 2024 · 1 comment

Comments

@ariostas
Copy link
Member

ariostas commented Apr 3, 2024

CXXFLAGS and LDFLAGS are defined twice in the makefile, and CXXFLAGS should be used for the .o targets, but it's missing.

TrackLooper/Makefile

Lines 13 to 17 in ad8ed39

CXXFLAGS = -g -O2 -Wall -fPIC -Wshadow -Woverloaded-virtual -lineinfo -fopenmp -lgomp --default-stream per-thread
LDFLAGS = -g -O2 -Wall -fPIC -Wshadow -Woverloaded-virtual
SOFLAGS = -g -shared
CXXFLAGS = -g -O2 -Wall -fPIC -Wshadow -Woverloaded-virtual
LDFLAGS = -g -O2 $(SDLLIB) -L${TRACKLOOPERDIR}/SDL

@ariostas
Copy link
Member Author

ariostas commented Apr 5, 2024

Apart from that, it seems to me like there are outdated things like the bin/doAnalysis target.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant