Initial commit of netcdf
- Group Reviewers
make check is ran after make install
No Linters Available
No Unit Test Coverage
Apart from the many errors in package.yml, and your insufficient test plan, there has been no package request for netcdf yet. Why did you push this? In fact, as per our policy:
Prior to submitting a patch, please ensure you are checking the following:
- There is not an existing patch already provided by others.
- If you are submitting a patch for the inclusion of software in the repo, that the package your patch pertains to has a corresponding package request that is accepted for inclusion or is a dependency of a package that has been accepted into the repository.
It is a prerequisite for Generic Mapping Tools package. Netcdf needs to be installed first along with Gdal package which already exists. It is my first packaging attempt. Could you please help me fix errors?
Regarding your test plan: units test are cool, but they shouldn't be the whole test plan. Just manually perform some operations with your package or try to build Generic Mapping Tools against netcdf-c and make sure it works.
And finally, patch's title should match the package name, so in in this case it misses -c.
License must be a SPDX-3.0 identifier. Here's a list: https://spdx.org/licenses. You don't the full license body.
The summary is longish. Just write a simple phrase: that's what users get first when they read at a package search list with eopkg.
Use 4-space indentation.
There's a static library and we usually don't allow them in packages. In general, passing --disable-static to %configure solves it.