docs icon indicating copy to clipboard operation
docs copied to clipboard

[bug] `self.dependencies[lib].cpp_info.resdirs` empty in test package

Open SpaceIm opened this issue 3 years ago • 3 comments

Environment Details (include every applicable attribute)

  • Operating System+version: Windows 10
  • Compiler+version: Visual Studio 2022
  • Conan version: 1.52.0
  • Python version: 3.10.2

Steps to reproduce (Include if Applicable)

In test_package:

    def generate(self):
        self.output.info(f"bindirs: {self.dependencies[<recipename>].cpp_info.bindirs}")
        self.output.info(f"includedirs: {self.dependencies[<recipename>].cpp_info.includedirs}")
        self.output.info(f"libdirs: {self.dependencies[<recipename>].cpp_info.libdirs}")
        self.output.info(f"resdirs: {self.dependencies[<recipename>].cpp_info.resdirs}")

run conan create

Logs (Executed commands with output) (Include/Attach if Applicable)

libjpeg/9e (test package): bindirs: ['C:\\Users\\spaceim\\.conan\\data\\libjpeg\\9e\\_\\_\\package\\e9a552ebe8f994398de9ceee972f0ad207df0658\\bin']
libjpeg/9e (test package): includedirs: ['C:\\Users\\spaceim\\.conan\\data\\libjpeg\\9e\\_\\_\\package\\e9a552ebe8f994398de9ceee972f0ad207df0658\\include']
libjpeg/9e (test package): libdirs: ['C:\\Users\\spaceim\\.conan\\data\\libjpeg\\9e\\_\\_\\package\\e9a552ebe8f994398de9ceee972f0ad207df0658\\lib']
libjpeg/9e (test package): resdirs: []

(In libjpeg recipe, there is a cmake_layout, and resdirs is not modified in package_info(), see https://github.com/conan-io/conan-center-index/pull/13123/commits/30e4ab1c0a00a2f03d98ddc137d7dda7cfb54c56)

SpaceIm avatar Sep 23 '22 21:09 SpaceIm

Actually it seems that layout() set resdirs to empty

/cc @uilianries @SSE4 @prince-chrismc for documentation in CCI

SpaceIm avatar Sep 23 '22 22:09 SpaceIm

Yes, it is true, layout() changes the defaults, as resdirs was not been used very much, we decided that the default should be empty.

Moving this issue to docs.

memsharded avatar Sep 24 '22 10:09 memsharded

Good to know, so we don't need to update it in recipes.

uilianries avatar Sep 26 '22 11:09 uilianries