Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

changes to partially fix make distcheck #2342

Closed
wants to merge 1 commit into from

Conversation

edwardhartnett
Copy link
Contributor

Part of #2338

This PR contains many Makefile.am fixes.

There are still some issues to get distcheck working, after this PR is merged.

@edwardhartnett edwardhartnett requested a review from WardF as a code owner May 5, 2022 14:06
@edwardhartnett edwardhartnett changed the title changes to fix make distcheck changes to partially fix make distcheck May 5, 2022
@edwardhartnett
Copy link
Contributor Author

I will take this down as @DennisHeimbigner has a PR that fixes some of these issues. When his PR is merged, I'll put a corrected one back up with the remaining changes, if necessary.

DennisHeimbigner added a commit to DennisHeimbigner/netcdf-c that referenced this pull request May 9, 2022
re: Unidata#2342
This PR replaces PR Unidata#2342

This PR extends the distcheck corrections in PR
Unidata#2342.  That original PR
exposed some errors in the file naming in the plugins and
nczarr_test directories.  This PR corrects those problems and
should be used instead of Unidata#2342

Ed Hartnett's suggestion about how to install the plugins in the
user specified directory will be addressed in a subsequent PR.
@edwardhartnett edwardhartnett deleted the ejh_distcheck_2 branch June 12, 2022 17:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant