libtiff: allow using zlib-ng instead of zlib

[Nekto89]

Specify library name and version: libtiff/all

zlib-ng provides better performance.

---

• I've read the contributing guidelines.
• I've used a recent Conan client version close to the currently deployed.
• I've tried at least one configuration locally with the conan-center hook activated.

[memsharded]

This change will break users that had zlib=True option defined. I'd try to leave it True with meaning "zlib" and just add the new one.

[conan-center-bot]

Conan v1 pipeline ✔️

All green in build 2 (24b47af412e983b7518526ce393108db67b08e45):

• libtiff/4.5.1@:
  All packages built successfully! (All logs)

• libtiff/4.3.0@:
  All packages built successfully! (All logs)

• libtiff/4.4.0@:
  All packages built successfully! (All logs)

• libtiff/4.5.0@:
  All packages built successfully! (All logs)

---

Conan v2 pipeline ✔️

Note: Conan v2 builds may be required once they are on the v2 ready list

All green in build 2 (24b47af412e983b7518526ce393108db67b08e45):

• libtiff/4.4.0@:
  All packages built successfully! (All logs)

• libtiff/4.5.0@:
  All packages built successfully! (All logs)

• libtiff/4.5.1@:
  All packages built successfully! (All logs)

• libtiff/4.3.0@:
  All packages built successfully! (All logs)

[AbrilRBS]

Hi @Nekto89 thanks a lot for your patience - These PRs got put on hold while we could think about ways to have this be used more generally, supported within the Conan client itself.

There's now a PR conan-io/conan#14871 (We might or might not end up merging it, we're still in the thinking and discussion phase) that would introduce the possibility for users to do these kinds of substitutions in a global way, without the need to modify the recipes.

If the feature does not go forward we'll then revisit the PRs, so either way, thanks a lot for your contribution :)

[valgur]

LGTM, except for the backwards-compatibility concern mentioned by memsharded.

[Nekto89]

Hi @Nekto89 thanks a lot for your patience - These PRs got put on hold while we could think about ways to have this be used more generally, supported within the Conan client itself.

There's now a PR conan-io/conan#14871 (We might or might not end up merging it, we're still in the thinking and discussion phase) that would introduce the possibility for users to do these kinds of substitutions in a global way, without the need to modify the recipes.

If the feature does not go forward we'll then revisit the PRs, so either way, thanks a lot for your contribution :)

Hi @RubenRBS

https://blog.conan.io/2024/02/20/Conan-2-graph-features.html

Should I close PRs with zlib-ng option? Is it something that should be completely on user side with new [replace_requires]? What about check of zlib_ng.options.zlib_compat option? How to implement it in case of [replace_requires] usage?

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions.