Skip to content
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

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
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.

1 participant