Don't break on empty patch list for version in conandata.yml (#15841)

[iskunk]

Changelog: Bugfix: Avoid TypeError when a version in conandata.yml lists no patches.
Docs: Omit

(#15841)

• Refer to the issue that supports this Pull Request.
• If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
• I've read the Contributing guide.
• I've followed the PEP8 style guides for Python code.
• I've opened another PR in the Conan docs repo to the develop branch, documenting this one.

[AbrilRBS]

Hi! Thanks a lot for taking the time to add the extra checks, really appreciated.
Could you add a test to ensure this behaviour is consistent in the future? Else we can contribute the test ourselves no problem :)

[iskunk]

Goodness, this moved before I even got a chance to respond :-]

Could you add a test to ensure this behaviour is consistent in the future?

I'm afraid I'm barely conversant with the testing framework Conan uses; hope it's not a big deal!

Suggested change

I originally wrote the warning like that, but then I saw output like this:

package/1.2.3: Calling source() in /home/src/package-1.2.3
package/1.2.3: apply_conandata_patches(): No patches defined for version 1.2.3 in conandata

I figured, there's no need to print the version twice.

Oh, nevermind, sorry I understood it incorrectly. A warning makes sense.

Yep, I saw that the functions print a warning in a similar instance, and so presumed that the same should happen here.

Thank you both!