New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Makefile update. #26
Makefile update. #26
Conversation
Fixes issue google#19: moving the link statements to the end of the flag declarations appears to fix the build issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per offline discussion, lets split the flags apart so the compiler flags are still first. Also, don't we need to flip GL and glfw?
Done. PTAL |
linux/example/Makefile
Outdated
@@ -75,7 +76,7 @@ $(FLUTTER_EXAMPLE_DIR)/build: | |||
--local-engine=$(FLUTTER_ENGINE_BUILD); | |||
|
|||
$(BIN_OUT): $(SOURCES) $(LIBRARIES) | |||
$(CXX) $(SOURCES) $(CXXFLAGS) -o $@ | |||
$(CXX) $(SOURCES) $(CXXFLAGS) $(CPPFLAGS) $(LDFLAGS) -o $@ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep C*FLAGS before the sources list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Closes #19 |
Fixes issue #19: moving the link statements to the end of the flag
declarations appears to fix the build issue.