Skip to content

vpath build: use top_srcdir#2263

Merged
WardF merged 1 commit intoUnidata:mainfrom
wkliao:vpath
Apr 5, 2022
Merged

vpath build: use top_srcdir#2263
WardF merged 1 commit intoUnidata:mainfrom
wkliao:vpath

Conversation

@wkliao
Copy link
Copy Markdown
Contributor

@wkliao wkliao commented Mar 27, 2022

No description provided.

@wkliao wkliao requested a review from WardF as a code owner March 27, 2022 22:38
Copy link
Copy Markdown
Collaborator

@DennisHeimbigner DennisHeimbigner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks right to me.

@WardF
Copy link
Copy Markdown
Member

WardF commented Mar 29, 2022

Is there an equivalent change that needs to be made in CMakeLists.txt? I'll try to sort this out for myself before merging. Thanks!

@WardF WardF added this to the 4.9.1 milestone Mar 29, 2022
@wkliao
Copy link
Copy Markdown
Contributor Author

wkliao commented Mar 29, 2022

cmake always builds out of source.
So, github should have already captured the error if cmake is used in the workflow.

@wkliao
Copy link
Copy Markdown
Contributor Author

wkliao commented Apr 5, 2022

It will be the best to move this PR to 4.9.0 milestone, as the latest main branch failed at make time when running a VPATH build. Below shows the configure command used and the error messages.

../netcdf-c/configure CC=mpicc \
      CPPFLAGS="-I$HDF5_DIR/include" \
      LDFLAGS="-L$HDF5_DIR/lib"

make
...
Making all in nczarr_test
cat: ../nc_test4/tst_quantize.c: No such file or directory
cat: ../nc_test4/tst_filter_avail.c: No such file or directory
make[2]: *** [Makefile:1730: test_quantize.c] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [Makefile:1735: test_filter_avail.c] Error 1
cat: ../nc_test4/tst_specific_filters.sh: No such file or directory
make[2]: *** [Makefile:1741: run_specific_filters.sh] Error 1
make[1]: *** [Makefile:744: all-recursive] Error 1
make: *** [Makefile:589: all] Error 2

@WardF WardF modified the milestones: 4.9.1, 4.9.0 Apr 5, 2022
@WardF WardF merged commit 127c139 into Unidata:main Apr 5, 2022
@wkliao wkliao deleted the vpath branch April 6, 2022 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants