sail: new package

[HappySeaFox]

Specify library name and version: sail/0.9.0-pre22

SAIL is a general-purpose image decoding library.

---

• I've read the guidelines for contributing.
• I've followed the PEP8 style guides for Python code in the recipes.
• I've used the latest Conan client version.
• I've tried at least one configuration locally with the conan-center hook activated - NO because of conan-center hook always reports "pre_export(): could not get source code" on WIndows hooks#157

[uilianries]

It looks good, only a small question.

[conan-center-bot]

All green in build 76 (e896ad7b4919ddf9d1b46095d0c6d66b0e739ead):

• sail/0.9.0-pre22@:
  All packages built successfully! (All logs)

[HappySeaFox]

@SpaceIm

I guess it works (maybe not always), but you should really consider using imported target of libavif instead

I totally agree. Please correct me if I'm wrong, the imported target for libavif is avif. It's used in the libavif tests, and I use it too as well.

[SpaceIm]

@SpaceIm

I guess it works (maybe not always), but you should really consider using imported target of libavif instead

I totally agree. Please correct me if I'm wrong, the imported target for libavif is avif. It's used in the libavif tests, and I use it too as well.

Indeed, I see that your patch replace value of sail_avif_libs by avif target.
But for example you don't use PNG::PNG target here: https://github.com/HappySeaFox/sail/blob/bf798da3fbcc003b6f16b7997c969b92e5514b3f/src/sail-codecs/png/png.cmake#L8-L9. I think it's a side effect of your wish to make something generic in sail_codec macro.

[HappySeaFox]

Thanks for the approval and merging! :) How soon it will be available in conan search?

[HappySeaFox]

@SpaceIm

But for example you don't use PNG::PNG target here

Oh, I see. Yes, it uses the Cmake's FindPNG functionality. It exports PNG_LIBRARIES, see https://cmake.org/cmake/help/latest/module/FindPNG.html . Since it doesn't fail in the linking phase, I guess conan generates a valid ${PNG_LIBRARIES} for libpng cmake integration (or just re-uses the CMake's one)? With png.lib and zlib.lib I mean. Same to libtiff which has even more dependencies. Otherwise we could see a linking errors. And libavif indeed needs a special patching since we have no FindAVIF in CMake.