Explicitly depend on zlib in conda recipes

[wence-]

Description

We were previously obtaining zlib transitively through our cmake dependency, but since the 3.27.4 conda package, this dependency no longer exists. Therefore we must depend on zlib ourselves.

• Closes [BUG] libcudf CI builds failing due to lack of zlib package #14021

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[bdice]

@wence- Can you comment on where/how we use zlib? Does it need to be in the conda package dependencies (meta.yml for libcudf, or cudf, …) as well?

[bdice]

Is there an issue documenting the problems you see in builds?

[wence-]

There is now: #14021

[vyasr]

We support zlib compression/decompression in ORC reading/writing, so we should be including it in our recipe.

[bdice]

This is what I was missing: #14021 (comment)

[bdice]

Thanks for the fix, this looks right to me.

[vyasr]

@vuule do you know if there's a specific version of zlib that we require for our use cases? Wondering if we need a lower bound here.

[ajschmidt8]

Approving ops-codeowner file changes

[vuule]

@vuule do you know if there's a specific version of zlib that we require for our use cases? Wondering if we need a lower bound here.

I'm pretty sure we go back quite a bit and have the same API available. I'm looking at their changelog, and there are bug fixes in every release. Based on this, should we make 1.2.13 the lower bound? It was released Oct 2022.

[wence-]

@vuule do you know if there's a specific version of zlib that we require for our use cases? Wondering if we need a lower bound here.

I'm pretty sure we go back quite a bit and have the same API available. I'm looking at their changelog, and there are bug fixes in every release. Based on this, should we make 1.2.13 the lower bound? It was released Oct 2022.

Done.

[vyasr]

Lower bound looks reasonable to me.

[vyasr]

/merge