-
Notifications
You must be signed in to change notification settings - Fork 546
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
Build fixes #1328
Build fixes #1328
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1328 +/- ##
=========================================
- Coverage 92.28% 91.9% -0.39%
=========================================
Files 466 466
Lines 43109 43109
Branches 5235 5235
=========================================
- Hits 39785 39618 -167
- Misses 3292 3459 +167
Partials 32 32
Continue to review full report at Codecov.
|
0aab321
to
e760329
Compare
It's been dropped from GCC, appears OpenACC is the new hotness for this kind of thing.
Add a test in CI to cover this
Also remove a duplicated block (rebase error)
e760329
to
1891809
Compare
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.
Thanks!
Working as expected.
clean: | ||
$(PYTHON_EXE) $(SCRIPTS_DIR)/cleanup.py --build-dir="%{build_dir}" | ||
|
||
distclean: | ||
$(PYTHON_EXE) $(SCRIPTS_DIR)/cleanup.py --build-dir="%{build_dir}" --distclean | ||
|
||
install: $(CLI) docs | ||
install: libs cli docs |
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.
I suggest to:
install: all
as you always end up require all
@@ -40,18 +41,18 @@ libs: $(LIBRARIES) | |||
|
|||
# Misc targets | |||
|
|||
docs: | |||
%{build_doc_commands} |
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.
I suggest to have docs.stamp or any other file created, so that at make install the sphinx will not run again
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.
Good idea thanks
Fixes install target when static lib disabled (#1327)
Accept CXXFLAGS and LDFLAGS from environment, and adds docs to the
all
target (#1237)Plus some cleanups in generating the template variables.