Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[poco] fix PocoNetSSL and PocoNetSSLWin simultaneous usage under Windows #4205

Merged
merged 18 commits into from
Feb 15, 2021

Conversation

mathbunnyru
Copy link
Contributor

Specify library name and version: lib/1.0

  • 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.

@mathbunnyru mathbunnyru mentioned this pull request Jan 10, 2021
4 tasks
@mathbunnyru
Copy link
Contributor Author

Should also fix in CCI:
#4184
#4095

@paulo-coutinho could you please check, that this change fixed poco for you?

@paulocoutinhox
Copy link
Contributor

paulocoutinhox commented Jan 10, 2021

Hi,

The pull request?

Thanks

@paulocoutinhox
Copy link
Contributor

With this, i only need "enable_net" correct?

@paulocoutinhox
Copy link
Contributor

@mathbunnyru is this that im thinking in do after. Thanks for you time.

@mathbunnyru
Copy link
Contributor Author

With this, i only need "enable_net" correct?

Yes. Or you could try default options as well if you don't need any components which are not turned on by default.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@mathbunnyru mathbunnyru reopened this Jan 10, 2021
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

"PocoNetSSL": _PocoComponent("enable_netssl", True, ("PocoCrypto", "PocoUtil", "PocoNet", ), True), # also external openssl
"PocoNetSSLWin": _PocoComponent("enable_netssl_win", True, ("PocoNet", "PocoUtil", ), True),
"PocoNetSSL": _PocoComponent("enable_netssl", not tools.os_info.is_windows, ("PocoCrypto", "PocoUtil", "PocoNet", ), True), # also external openssl
"PocoNetSSLWin": _PocoComponent("enable_netssl_win", tools.os_info.is_windows, ("PocoNet", "PocoUtil", ), True),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should it be self.settings.os == Windows instead? my doubt that it may break cross-building scenarios (e.g. Windows -> Linux or Linux -> Windows)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can I use self in class namespace?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SSE4 could you elaborate on this, please?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it's possible. probably, you need to add both components and then process it in config_options or somewhere else.
the issue is running on Linux with self.settings.os == Windows, tools.os_info.is_windows will return False.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would try replacing tools.os_info.is_windows with self.settings.os == Windows as suggested by @SSE4.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can leave this option to True, since it is removed in other OSs different from windows


def build(self):
cmake = CMake(self)
cmake.definitions["TEST_CRYPTO"] = self.options["poco"].enable_crypto == True
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@conan-center-bot

This comment has been minimized.

@danimtb danimtb added the infrastructure Waiting on tools or services belonging to the infra label Feb 3, 2021
@jgsogo jgsogo mentioned this pull request Feb 4, 2021
4 tasks
@conan-center-bot
Copy link
Collaborator

Failure in build 21 (4f268faff9493d50ea46696fd8e957556ed3a4ee):

  • poco/1.9.4@:
    CI failed to create some packages (All logs)

    Logs for packageID d46c5bc18c7ac937bdb191768aeb1945aeda2709:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Release
    compiler=Visual Studio
    compiler.runtime=MT
    compiler.version=16
    os=Windows
    os_build=Windows
    [options]
    poco:shared=False
    
    [env]
    
    poco/1.9.4 (test package): Installing package
    Requirements
        bzip2/1.0.8 from 'conan-center' - Cache
        expat/2.2.10 from 'conan-center' - Cache
        openssl/1.1.1i from 'conan-center' - Cache
        pcre/8.44 from 'conan-center' - Cache
        poco/1.9.4 from local cache - Cache
        sqlite3/3.34.0 from 'conan-center' - Cache
        zlib/1.2.11 from 'conan-center' - Cache
    Packages
        bzip2/1.0.8:8feb221a01dcb7f80d1d66f9676d4e44215f5e08 - Cache
        expat/2.2.10:a79a557254fabcb77849dd623fed97c9c5ab7651 - Cache
        openssl/1.1.1i:a79a557254fabcb77849dd623fed97c9c5ab7651 - Cache
        pcre/8.44:963f27b1d1bc3c49f3ad48297e5f6f8de7d96cb7 - Cache
        poco/1.9.4:417dc951eef84d92197ce50f3f5dbfcd8765f923 - Cache
        sqlite3/3.34.0:358a0fdc421664bc2e72248ae2c8a47ba57272ee - Cache
        zlib/1.2.11:a79a557254fabcb77849dd623fed97c9c5ab7651 - Cache
    
    Installing (downloading, building) binaries...
    bzip2/1.0.8: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    expat/2.2.10: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    openssl/1.1.1i: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    sqlite3/3.34.0: Already installed!
    sqlite3/3.34.0: Appending PATH env var with : C:\J\w\BuildSingleReference@5\.conan\data\sqlite3\3.34.0\_\_\package\358a0fdc421664bc2e72248ae2c8a47ba57272ee\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    zlib/1.2.11: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    pcre/8.44: Already installed!
    pcre/8.44: Appending PATH environment variable: C:\J\w\BuildSingleReference@5\.conan\data\pcre\8.44\_\_\package\963f27b1d1bc3c49f3ad48297e5f6f8de7d96cb7\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.9.4: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.9.4 (test package): Generator txt created conanbuildinfo.txt
    poco/1.9.4 (test package): Generator cmake created conanbuildinfo.cmake
    poco/1.9.4 (test package): Generated conaninfo.txt
    poco/1.9.4 (test package): Generated graphinfo
    Using lockfile: 'C:\J\w\BuildSingleReference@5\conan-center-index\recipes\poco\all\test_package\build\d85a4aab099651c589a871b1bb6738ab106b193c/conan.lock'
    Using cached profile from lockfile
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] 'fPIC' option not found
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] OK
    poco/1.9.4 (test package): Calling build()
    ERROR: poco/1.9.4 (test package): Error in build() method, line 22
    	cmake.definitions["TEST_NETSSL"] = self.options["poco"].enable_netssl == True
    	ConanException: option 'enable_netssl' doesn't exist
    Possible options are ['shared', 'enable_apacheconnector', 'enable_cppparser', 'enable_crypto', 'enable_data', 'enable_data_odbc', 'enable_data_sqlite', 'enable_encodings', 'enable_json', 'enable_mongodb', 'enable_net', 'enable_netssl_win', 'enable_pagecompiler', 'enable_pagecompiler_file2page', 'enable_pdf', 'enable_pocodoc', 'enable_redis', 'enable_sevenzip', 'enable_util', 'enable_xml', 'enable_zip']
    
  • poco/1.10.1@:
    CI failed to create some packages (All logs)

    Logs for packageID 6affa419bd4c8c4f073e012613e896789c5186e0:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Release
    compiler=Visual Studio
    compiler.runtime=MT
    compiler.version=15
    os=Windows
    os_build=Windows
    [options]
    poco:shared=False
    
        libmysqlclient/8.0.17:4f07e2a55a61c6bea81b783ce58c7dfbce6217e6 - Cache
        libpq/13.1:fbf483db528c7f9fef8875f5611df78cc54ccef2 - Cache
        openssl/1.1.1i:7bd6f2c3d5c4e48a75805376b58cde753392f711 - Cache
        pcre/8.44:f78c1d1574a81c067f21d31523b2626bbe1ff224 - Cache
        poco/1.10.1:27aa1b302b24609b0ec880751b303f30a83bf2b5 - Cache
        sqlite3/3.34.0:c1eda157197f20f0d98c0aafb16ca15676ccd30b - Cache
        zlib/1.2.11:7bd6f2c3d5c4e48a75805376b58cde753392f711 - Cache
    
    Installing (downloading, building) binaries...
    apr/1.7.0: Already installed!
    apr/1.7.0: Settings APR_ROOT environment var: /c/j/w/buildsinglereference@6/.conan/data/apr/1.7.0/_/_/package/91ffae9f5431df04ca7c2c807aca17e301127ceb
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    bzip2/1.0.8: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    expat/2.2.10: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    openssl/1.1.1i: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    sqlite3/3.34.0: Already installed!
    sqlite3/3.34.0: Appending PATH env var with : C:\J\w\BuildSingleReference@6\.conan\data\sqlite3\3.34.0\_\_\package\c1eda157197f20f0d98c0aafb16ca15676ccd30b\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    zlib/1.2.11: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    apr-util/1.6.1: Already installed!
    apr-util/1.6.1: Appending PATH env var : C:\J\w\BuildSingleReference@6\.conan\data\apr-util\1.6.1\_\_\package\c739eb92ae15d5045cb96fa3e73f062f94532ace\bin
    apr-util/1.6.1: Settings APR_UTIL_ROOT environment var: /c/j/w/buildsinglereference@6/.conan/data/apr-util/1.6.1/_/_/package/c739eb92ae15d5045cb96fa3e73f062f94532ace
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    libmysqlclient/8.0.17: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    libpq/13.1: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    pcre/8.44: Already installed!
    pcre/8.44: Appending PATH environment variable: C:\J\w\BuildSingleReference@6\.conan\data\pcre\8.44\_\_\package\f78c1d1574a81c067f21d31523b2626bbe1ff224\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.10.1: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.10.1 (test package): Generator txt created conanbuildinfo.txt
    poco/1.10.1 (test package): Generator cmake created conanbuildinfo.cmake
    poco/1.10.1 (test package): Generated conaninfo.txt
    poco/1.10.1 (test package): Generated graphinfo
    Using lockfile: 'C:\J\w\BuildSingleReference@6\conan-center-index\recipes\poco\all\test_package\build\abf0cd6c33fb66ab006e917034f7a8e66057fa7f/conan.lock'
    Using cached profile from lockfile
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] 'fPIC' option not found
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] OK
    poco/1.10.1 (test package): Calling build()
    WARN: apr-util/1.6.1: requirement expat/2.2.9 overridden by poco/1.10.1 to expat/2.2.10 
    ERROR: poco/1.10.1 (test package): Error in build() method, line 22
    	cmake.definitions["TEST_NETSSL"] = self.options["poco"].enable_netssl == True
    	ConanException: option 'enable_netssl' doesn't exist
    Possible options are ['shared', 'enable_apacheconnector', 'enable_cppparser', 'enable_crypto', 'enable_data', 'enable_data_mysql', 'enable_data_odbc', 'enable_data_postgresql', 'enable_data_sqlite', 'enable_encodings', 'enable_json', 'enable_jwt', 'enable_mongodb', 'enable_net', 'enable_netssl_win', 'enable_pagecompiler', 'enable_pagecompiler_file2page', 'enable_pdf', 'enable_pocodoc', 'enable_redis', 'enable_sevenzip', 'enable_util', 'enable_xml', 'enable_zip']
    
  • poco/1.10.0@:
    CI failed to create some packages (All logs)

    Logs for packageID 6affa419bd4c8c4f073e012613e896789c5186e0:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Release
    compiler=Visual Studio
    compiler.runtime=MT
    compiler.version=15
    os=Windows
    os_build=Windows
    [options]
    poco:shared=False
    
        libmysqlclient/8.0.17:4f07e2a55a61c6bea81b783ce58c7dfbce6217e6 - Cache
        libpq/13.1:fbf483db528c7f9fef8875f5611df78cc54ccef2 - Cache
        openssl/1.1.1i:7bd6f2c3d5c4e48a75805376b58cde753392f711 - Cache
        pcre/8.44:f78c1d1574a81c067f21d31523b2626bbe1ff224 - Cache
        poco/1.10.0:27aa1b302b24609b0ec880751b303f30a83bf2b5 - Cache
        sqlite3/3.34.0:c1eda157197f20f0d98c0aafb16ca15676ccd30b - Cache
        zlib/1.2.11:7bd6f2c3d5c4e48a75805376b58cde753392f711 - Cache
    
    Installing (downloading, building) binaries...
    apr/1.7.0: Already installed!
    apr/1.7.0: Settings APR_ROOT environment var: /c/j/w/buildsinglereference@4/.conan/data/apr/1.7.0/_/_/package/91ffae9f5431df04ca7c2c807aca17e301127ceb
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    bzip2/1.0.8: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    expat/2.2.10: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    openssl/1.1.1i: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    sqlite3/3.34.0: Already installed!
    sqlite3/3.34.0: Appending PATH env var with : C:\J\w\BuildSingleReference@4\.conan\data\sqlite3\3.34.0\_\_\package\c1eda157197f20f0d98c0aafb16ca15676ccd30b\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    zlib/1.2.11: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    apr-util/1.6.1: Already installed!
    apr-util/1.6.1: Appending PATH env var : C:\J\w\BuildSingleReference@4\.conan\data\apr-util\1.6.1\_\_\package\c739eb92ae15d5045cb96fa3e73f062f94532ace\bin
    apr-util/1.6.1: Settings APR_UTIL_ROOT environment var: /c/j/w/buildsinglereference@4/.conan/data/apr-util/1.6.1/_/_/package/c739eb92ae15d5045cb96fa3e73f062f94532ace
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    libmysqlclient/8.0.17: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    libpq/13.1: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    pcre/8.44: Already installed!
    pcre/8.44: Appending PATH environment variable: C:\J\w\BuildSingleReference@4\.conan\data\pcre\8.44\_\_\package\f78c1d1574a81c067f21d31523b2626bbe1ff224\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.10.0: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.10.0 (test package): Generator txt created conanbuildinfo.txt
    poco/1.10.0 (test package): Generator cmake created conanbuildinfo.cmake
    poco/1.10.0 (test package): Generated conaninfo.txt
    poco/1.10.0 (test package): Generated graphinfo
    Using lockfile: 'C:\J\w\BuildSingleReference@4\conan-center-index\recipes\poco\all\test_package\build\abf0cd6c33fb66ab006e917034f7a8e66057fa7f/conan.lock'
    Using cached profile from lockfile
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] 'fPIC' option not found
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] OK
    poco/1.10.0 (test package): Calling build()
    WARN: apr-util/1.6.1: requirement expat/2.2.9 overridden by poco/1.10.0 to expat/2.2.10 
    ERROR: poco/1.10.0 (test package): Error in build() method, line 22
    	cmake.definitions["TEST_NETSSL"] = self.options["poco"].enable_netssl == True
    	ConanException: option 'enable_netssl' doesn't exist
    Possible options are ['shared', 'enable_apacheconnector', 'enable_cppparser', 'enable_crypto', 'enable_data', 'enable_data_mysql', 'enable_data_odbc', 'enable_data_postgresql', 'enable_data_sqlite', 'enable_encodings', 'enable_json', 'enable_jwt', 'enable_mongodb', 'enable_net', 'enable_netssl_win', 'enable_pagecompiler', 'enable_pagecompiler_file2page', 'enable_pdf', 'enable_pocodoc', 'enable_redis', 'enable_sevenzip', 'enable_util', 'enable_xml', 'enable_zip']
    
  • poco/1.9.3@:
    CI failed to create some packages (All logs)

    Logs for packageID 5d71150932dcea9c52bd63bb2c17f47b17f49e38:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Release
    compiler=Visual Studio
    compiler.runtime=MT
    compiler.version=14
    os=Windows
    os_build=Windows
    [options]
    poco:shared=False
    
    [env]
    
    poco/1.9.3 (test package): Installing package
    Requirements
        bzip2/1.0.8 from 'conan-center' - Cache
        expat/2.2.10 from 'conan-center' - Cache
        openssl/1.1.1i from 'conan-center' - Cache
        pcre/8.44 from 'conan-center' - Cache
        poco/1.9.3 from local cache - Cache
        sqlite3/3.34.0 from 'conan-center' - Cache
        zlib/1.2.11 from 'conan-center' - Cache
    Packages
        bzip2/1.0.8:5dc7229f3a8cc533e83775b90555927da566880b - Cache
        expat/2.2.10:df81ad20137149d7a51276fd3e24009b45e5964a - Cache
        openssl/1.1.1i:df81ad20137149d7a51276fd3e24009b45e5964a - Cache
        pcre/8.44:b89c104ef718b173f54077c16cedfce2df3f5b45 - Cache
        poco/1.9.3:addd031a7f07ff7df7bd9cf8b063fce56086941b - Cache
        sqlite3/3.34.0:6f82a010d2f9eb7d5e1591c958d4c7faa7859e76 - Cache
        zlib/1.2.11:df81ad20137149d7a51276fd3e24009b45e5964a - Cache
    
    Installing (downloading, building) binaries...
    bzip2/1.0.8: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    expat/2.2.10: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    openssl/1.1.1i: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    sqlite3/3.34.0: Already installed!
    sqlite3/3.34.0: Appending PATH env var with : C:\J\w\BuildSingleReference@4\.conan\data\sqlite3\3.34.0\_\_\package\6f82a010d2f9eb7d5e1591c958d4c7faa7859e76\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    zlib/1.2.11: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    pcre/8.44: Already installed!
    pcre/8.44: Appending PATH environment variable: C:\J\w\BuildSingleReference@4\.conan\data\pcre\8.44\_\_\package\b89c104ef718b173f54077c16cedfce2df3f5b45\bin
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.9.3: Already installed!
    [HOOK - conan-center.py] post_package_info(): [CMAKE FILE NOT IN BUILD FOLDERS (KB-H019)] OK
    poco/1.9.3 (test package): Generator cmake created conanbuildinfo.cmake
    poco/1.9.3 (test package): Generator txt created conanbuildinfo.txt
    poco/1.9.3 (test package): Generated conaninfo.txt
    poco/1.9.3 (test package): Generated graphinfo
    Using lockfile: 'C:\J\w\BuildSingleReference@4\conan-center-index\recipes\poco\all\test_package\build\2c66e8e8cf729420cba8bf278c34c79f92e3a67d/conan.lock'
    Using cached profile from lockfile
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] 'fPIC' option not found
    [HOOK - conan-center.py] pre_build(): [FPIC MANAGEMENT (KB-H007)] OK
    poco/1.9.3 (test package): Calling build()
    ERROR: poco/1.9.3 (test package): Error in build() method, line 22
    	cmake.definitions["TEST_NETSSL"] = self.options["poco"].enable_netssl == True
    	ConanException: option 'enable_netssl' doesn't exist
    Possible options are ['shared', 'enable_apacheconnector', 'enable_cppparser', 'enable_crypto', 'enable_data', 'enable_data_odbc', 'enable_data_sqlite', 'enable_encodings', 'enable_json', 'enable_mongodb', 'enable_net', 'enable_netssl_win', 'enable_pagecompiler', 'enable_pagecompiler_file2page', 'enable_pdf', 'enable_pocodoc', 'enable_redis', 'enable_sevenzip', 'enable_util', 'enable_xml', 'enable_zip']
    
  • poco/1.8.1@:
    Didn't run or was cancelled before finishing


Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@mathbunnyru
Copy link
Contributor Author

Investigating together with @danimtb 👍

@dmn-star
Copy link
Contributor

dmn-star commented Feb 6, 2021

I would try replacing tools.os_info.is_windows with self.settings.os == Windows as suggested by @SSE4.

@mathbunnyru
Copy link
Contributor Author

Sorry, I'm on a one week vacation, so I won't be able to respond here for a while.

@danimtb
Copy link
Member

danimtb commented Feb 11, 2021

We need to add the changes of test_package/conanfile.py to avoid the error of the consumer in the CI. The issue with the failed CI status is due to Jenkins blocking the test_package executable due to networking. I'd suggest to comment the lines and add the reason like:

        // FIXME: CCI Jenkins blocks the request making the build fail
        // s.sendRequest(request);	
        // HTTPResponse response;	
        // std::istream& rs = s.receiveResponse(response);	
        // StreamCopier::copyStream(rs, std::cout);

@dmn-star
Copy link
Contributor

@danimtb
We have already turned off the net and net_2 tests for trial, which unfortunately did not help. Are there any more tests going online here?

#4205 (comment)
c53e6a7

@danimtb
Copy link
Member

danimtb commented Feb 11, 2021

@dmn-star I see. I have tested with the commented lines above on our staging environment and it is working. I will try to push the changes here and see what happens

@conan-center-bot
Copy link
Collaborator

All green in build 22 (cb0a9e88ee8ba5325e632c98334352f19d7f8bde):

  • poco/1.10.1@:
    All packages built successfully! (All logs)

  • poco/1.10.0@:
    All packages built successfully! (All logs)

  • poco/1.9.4@:
    All packages built successfully! (All logs)

  • poco/1.9.3@:
    All packages built successfully! (All logs)

  • poco/1.8.1@:
    All packages built successfully! (All logs)

@mathbunnyru
Copy link
Contributor Author

Thanks @danimtb ! 👍

@mathbunnyru
Copy link
Contributor Author

mathbunnyru commented Feb 15, 2021

@SpaceIm @SSE4 @ericLemanissier @danimtb please, approve the PR :)
It's time to get this merged 🎉

@danimtb danimtb removed the infrastructure Waiting on tools or services belonging to the infra label Feb 15, 2021
@conan-center-bot conan-center-bot merged commit d7e0581 into conan-io:master Feb 15, 2021
@prince-chrismc
Copy link
Contributor

@jgsogo @danimtb This PR triggered a build on master and it looks stuck.. it's been about ~10 hrs with master status 'being built'.
Looking at the last build in this PR it should be <4 hrs for the build

It's a PR from hell 🙃

@jgsogo
Copy link
Contributor

jgsogo commented Feb 16, 2021

OMG 😫

I've stopped and triggered again the build for master. CI restarted tonight and it is something we need to work on, how to recover from that circunstance.

@jngrb
Copy link
Contributor

jngrb commented Jul 15, 2021

@jgsogo @danimtb I think this should be changed again in order to again allow OpenSSL-enabled SSL support under Windows

see my discussion with @mathbunnyru in my issue #6311 und my suggested solution in #6314

@jgsogo
Copy link
Contributor

jgsogo commented Jul 16, 2021

@jgsogo @danimtb I think this should be changed again in order to again allow OpenSSL-enabled SSL support under Windows

see my discussion with @mathbunnyru in my issue #6311 und my suggested solution in #6314

Hi, @jngrb ! This kind of thing will always happen from time to time. I know people are doing their best in this repository, but we are humans sometimes 🙂 I will add another batch of people to the EAP so your PR can run, and people with much more experience than me will notice about it and review.

Anyway, if you are using these recipes in a production environment (cc @Talkless ) I encourage you to use recipe revisions and/or lockfiles. That way, you will be using always the same version of the recipe and you can decide when to update and start using modifyied versions.

@prince-chrismc
Copy link
Contributor

If you are using these recipes in a production environment I encourage you to use recipe revisions and/or lockfiles.

Words to live by!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.