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

Test to see if it gets labaled #12

Closed
wants to merge 170 commits into from

Conversation

prince-chrismc
Copy link
Owner

No description provided.

prince-chrismc and others added 30 commits February 3, 2021 15:01
* Update reviewing.md

* Typo

* Update docs/reviewing.md

Co-authored-by: Yoann Potinet <intelligide@hotmail.fr>

Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Yoann Potinet <intelligide@hotmail.fr>
Signed-off-by: SSE4 <tomskside@gmail.com>
* bump cpprestsdk/2.10.18

* missed config.yml
Signed-off-by: SSE4 <tomskside@gmail.com>
* - don't set visibility compiler flags for iOS

Signed-off-by: SSE4 <tomskside@gmail.com>

* - use b2 visibility feature, add option visibility

Signed-off-by: SSE4 <tomskside@gmail.com>

* boost: Update Conan conventions

Automatically created by bincrafters-conventions 0.30.2

Co-authored-by: bincrafters-user <bincrafters@gmail.com>
…mentation in public headers

* create a lib and disable macro implementation

C header-only lib with macro implementation are dangerous in a complex dependency graph.
You can never be sure that multiple third party static libs have not defined macro implementation themselves, leading to ODR violation.
Therefore we provide a binary instead, and prevent implementation by consumers in public headers.

* requires C99

* replace all occurrences of  #ifdef IMPL_MACRO

* completly remove implementation from headers
Signed-off-by: SSE4 <tomskside@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
* - add vaapi/system

Signed-off-by: SSE4 <tomskside@gmail.com>

* Update recipes/vaapi/all/conanfile.py

Co-authored-by: ericLemanissier <ericLemanissier@users.noreply.github.com>

* Update recipes/vaapi/all/conanfile.py

Co-authored-by: Uilian Ries <uilianries@gmail.com>

Co-authored-by: ericLemanissier <ericLemanissier@users.noreply.github.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
* - remove workaround for iOS

Signed-off-by: SSE4 <tomskside@gmail.com>

* libxml2: Update Conan conventions

Automatically created by bincrafters-conventions 0.30.2

Co-authored-by: bincrafters-user <bincrafters@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
* new wolfssl 4.6 version

* new wolfssl 4.6 version
* remove sqlite3 from build_require

sqlite3 is both requirement and build requirement.
But conan doesn't properly support to have the same dependency in requirement and build requirement, so it's removed for the moment.

* bump sqlite3 & libtiff

* allow libtiff > 4.1.0 for proj 7.0.0
* glib 2.67.3

* glib: Update Conan conventions

Automatically created by bincrafters-conventions 0.30.2

Co-authored-by: bincrafters-user <bincrafters@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
they are not part of qt per-se, but tools used when developing qt
* Copy in files from szip as a starting point

* Update versions for cpputest

* Copy in values for CppUTest

* Fix error:

ERROR: cpputest/4.0: Error in source() method, line 48
        tools.get(**self.conan_data["sources"][self.version])
        TypeError: get() argument after ** must be a mapping, not list

* Remove references to patches

* Remove reference to CMAKE_POSITION_INDEPENDENT_CODE

* Copy in test code from https://github.com/cpputest/conan-cpputest/blob/c4cf311e7b03de0ec8dfc89f2626b7ef172b7367/test_package/example.cpp

* Fix building of cpputest test package

* Remove remnant references to szip

* Fix inconsistent quoting of version numbers

* Second attempt at fixing quoting of version numbers

* Fix PC-FILES (KB-H020) by deleting lib/pkgconfig directory

* Fix CMAKE-MODULES-CONFIG-FILES (KB-H016) by deleting lib/CppUTest/cmake

* Add options

Copied in from:
https://github.com/cpputest/conan-cpputest/blob/c4cf311e7b03de0ec8dfc89f2626b7ef172b7367/conanfile.py

* Turn tests off, by default

* Update recipes/cpputest/all/conanfile.py

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Remove coverage and tests options - and force them to always be off.

Addressing code review feedback.

* Code review feedback: Fix use of find_package()

* Correct homepage URL to use https

* Remove unnecessary quotes in add_subdirectory() call

* Whitespace change: Fix inconsistent indentation

* Make default fPIC value consistent with majority of other recipes

* Code review feedback: Set pkg_config name

* Code review feedback: Remove unnecessary setting of self.cpp_info.filenames

* Attempt to fix Windows linker errors

LNK2019: unresolved external symbol __imp_timeGetTime

* Further attempt to fix Windows linker errors

LNK2019: unresolved external symbol __imp_timeGetTime

* Code review feedback: conditionally remove C++ settings

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Further attempt to fix Windows linker errors

LNK2019: unresolved external symbol __imp_timeGetTime

Fix taken from iimachines/webrtc-build#2 (comment)

Thanks to @prince-chrismc for pointing it out.

* Fix typo in configure()

* Remove ".lib" from library names, in further attempt to fix
missing __imp_timeGetTime symbol

* Remove verbose option - it doesn't affect the output library

* Make tests pass if extensions are disabled

* Invert if condition, to reduce nesting of code

* Further attempt at fixing missing symbols on Windows

* Another attempt at fixing missing symbols on Windows

* Revert back to the way of specifying winmm in b83b8da

As this matches the

* Fix windows linker errors

* Delete fPIC option if shared

* Shared libraries are not supported, so remove that option

* Completely remove shared option, as shared builds not supported

* Code-review feedback: Remove commented-out lines

* Code-review feedback: Add pthread dependency on some platforms

* Add target CppUTest::CppUTestExt

* Make CppUTest::CppUTestExt depend on CppUTest::CppUTest

* Port remaining package_info() code to components

* Rename option extensions to with_extensions

* Rename option longlong to with_longlong

* Rename option detect_mem_leaks to detect_memory_leaks

* Remove all options except fPIC and with_extensions

* Remove empty method configure()

* Code review suggestion: "with_extensions" now uses [True, False]

Co-authored-by: Uilian Ries <uilianries@gmail.com>

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
* fix output name collisions

* add octomap/1.9.6

* no -Werror

* shared and MTd runtime not supported
* Add aom/2.0.1

* Restrict compiler versions

* Always require perl on Windows

Co-authored-by: Uilian Ries <uilianries@gmail.com>

* Rename to libaom-av1

* Force build

Co-authored-by: Uilian Ries <uilianries@gmail.com>
* Add recipe for CERN ROOT data analysis framework.

* Minimal changes to fix CI failure.

Fixes two classes of CI failure:

(1) ROOT6 requires C++ standard >= 11.
This is now checked with tools.check_min_cppstd.

(2) Seg fault when building with clang dues LLVM library loading issues.
See: https://root-forum.cern.ch/t/root-with-geant4-llvm-problems/21960

* Implement improvements for PR #3732

Implement improvements suggested by @prince-chrismc including:

(1) Remove default options for dependent packages.

(2) Remove all f-strings for python 2 compatibility.

(3) Cache CMake object.

(4) Use dependencies from CCI where available.

(5) Reduce size of test_package.

(6) Apply version check boiler plate from conan-io/conan#8002.

* Set CMAKE_CXX_STANDARD in root/all/test_package.

* Implement PR #3732 review improvements

(1) Remove unused variable.

(2) Prefer class variables over instance variables.

(3) Move source patching from source method to build method.

(4) String formatting style changes.

(5) Use tools to remove files over in-recipe implementation.

* Fix typo in 66718b6.

This should fix the failing CI clang-5 build.

* Specify `required_conan_version`

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Update openssl version number for ROOT

Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>

* Fix backslashes in paths that might cause problems on Windows

Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>

* On Windows, backslashes in paths might cause problems (escaping of variables)

Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>

* Adding FIXME comment for dependencies where there is currently no Conan Center Index dependency available.

* Include conanbuildinfo.cmake in ROOT CMake as this is needed for MSVC.

Following recommendation in comments for #3732 by @madebr:
"This cmake script sets the current C runtime on MSVC (MT vs MD vd MTd vs MDd).
It also sets libstdc++11 vs libstdc++."

* Fix typo in 8b41c8a

* Fix typo in cmake.

* Fix path to source folder, solves issue when it is not current working directory.

* Fix failing build causing by changes for MSVC (introduced in 9181355)

* Rename package from root to cern-root.

* Apply suggestions from code review

Use self._source_subfolder everywhere
use os.path.join(...) instead of os.sep.join((...))

Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>

* Fix typo in previous commit.

* Fix build failure due to moving build directory in 13df472

* Fix LICENSE file copying to output package.

* Apply suggestions from code review

Update dependency version numbers.
Change name of SQLite3 target.

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>

* Run black on cern-root conanfile.py.

* Update ROOT version to latest patch (v6-22-06).

* Rename TODO to FIXME as FIXME is CCI convention.

* Add FIXME to warn about multiple CMake files as discussed in #3732 (comment)

* Fix issue with the wrong path when conan is called with --source-folder argument.

* Remove dead code.

* Reinstate conan_basic_setup.

The previous issues with conan_basic_setup (discussed in #3732 (comment))
were caused by conan_output_dirs_setup(). We now disable this step by
calling "conan_basic_setup(NO_OUTPUT_DIRS)".

Remove explicit setting of CMAKE_CXX_STANDARD as conan_basic_setup should set CMAKE_CXX_STANDARD for us.

* Revert change to CXX standard in 6548864f

Previous commit was incorrect and caused failures in clang <= 4.
conan_basic_setup will only set the standard under certain conditions
and this is experimental.
See: https://docs.conan.io/en/1.7/howtos/manage_cpp_standard.html

Revert to the original method of setting this.

* Implement style suggestions from @SpaceIm's  code review.

* Require libstdc++11. ROOT will not build with out the new ABI.

* Fix issue with the wrong path when conan is called with --source-folder argument.

* Try an alternative solution to the libcxx problem.

* Remove version number from file.

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Adding code review suggestions by @madebr

This should solve build issues on gcc and clang when libcxx != libstdc++11.

See:
<#3732 (comment)>

* Raise error on libc++ as it doesn't build due to portability issues in ROOT.

* Fix test_package CMake "install FILES given no DESTINATION" error.

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
* Add safeclib/3.6.0

* Use tools.remove_files_by_mask

* Use libsafec for alignment with existing distribution packages

* Fail test_package if not working constraint handler

* Fix licenses

* Remove "make check" due to build complaints for test sources

* Strip too explicit version from resulting library file name

* Limit recipe GCC version to >= 5

* Update recipes/libsafec/all/conanfile.py

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* Disable Werror to allow warnings

* Avoid extra patching, use version in the library name

* Fix exported includedirs to match original pkg-config package

* Update recipes/libsafec/all/test_package/conanfile.py

Co-authored-by: Yoann Potinet <intelligide@hotmail.fr>

* Update recipes/libsafec/all/conanfile.py

Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>

* Fix includedirs.append

* Isolate autotools.configure for better --keep-build support

* Add explicit name for pkg_config

* Minor name fix

* Keep autoreconf in build method

* Enforce standard compatibility for exported include path

* Revert "Enforce standard compatibility for exported include path"

This reverts commit dfda98c.

* Disable build for apply-clang with libc++

* Revert "Disable build for apply-clang with libc++"

This reverts commit b65111d.

* Update recipes/libsafec/all/conanfile.py

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: Yoann Potinet <intelligide@hotmail.fr>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
…e imported target

* bump libtiff & proj

* delete fPIC option if shared

* fix CMake imported target
* - libvpx 1.9.0

Signed-off-by: SSE4 <tomskside@gmail.com>

* Update recipes/libvpx/all/test_package/conanfile.py

* - 1.9.0 has c++ sources

Signed-off-by: SSE4 <tomskside@gmail.com>

* - target is needed for cross-building only

Signed-off-by: SSE4 <tomskside@gmail.com>

* - MSVC needs target as well

Signed-off-by: SSE4 <tomskside@gmail.com>

* - add x86 micro-architecture options

Signed-off-by: SSE4 <tomskside@gmail.com>

* - move replacements to patch

Signed-off-by: SSE4 <tomskside@gmail.com>

* Update recipes/libvpx/all/conanfile.py
* bump dependencies

* decrease build path length

* Revert "decrease build path length"

This reverts commit 6f8648d.

* use short_paths to make CI of CCI happy

* cosmetic change
Signed-off-by: Uilian Ries <uilianries@gmail.com>
* use external glfw

* explicitly set several CMake options

* explicit cpp_info.libs

* explicit pkg_config

* fix CMake imported target

* use C test_package

* define USE_LIBTYPE_SHARED if msvc and shared

* remove transitive system dependencies

removed system libs were coming from glfw and are not direct dependencies of raylib

* remove CUSTOMIZE_BUILD CMake option

not available in raylib 3.5.0

* fix windows and external glfw
ngrodzitski and others added 16 commits February 24, 2021 16:31
* add glfw/3.3.3

* fix cpp_info.builddirs

* download .tar.gz archive instead of .zip
* Add tsl-hat-trie/0.6.0

* Update recipes/tsl-hat-trie/all/conanfile.py

Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com>

Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com>
* bump eastl/3.17.06

* Fix eastl/3.17.03 sha256
…d + fix system libs

* reorder methods and attributes

* delete fPIC option if shared

* patch in build() only

a patch in _configure_autotools() with be executed twice with conan create -kb

* pkgconf required for autotools build

* pkgconf only required if zlib, lzma or icu

* finer grained cleanup in package()

* fix system_libs in package_info()

* provide official CMake variables for find_package

* fix typo

* remove unrelated CMake variable

* fix typo

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* check defined variables in CMake module file

* fix builddirs

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
* replace patch with conanfile.py

* add required boost parts

* remove patch

* remove workaround because it is not needed

* Bump openssl version

* remove cmake generator

* Revert "remove cmake generator"

This reverts commit a59402e.

* use patch

* Shorten patch code

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
* fix sprintf for Visual Studio >= 2015

* bump msys2

* fix typo in patch name

* add -FS flag for Visual Studio

to avoid errors with concurrent .pdb files access during build

* use https url
* fix cpp_info.builddirs

don't remove package_folder path

* add cmake variables to cmake_find_package only
* gtsam/4.0.3 native fix

* Update recipes/gtsam/all/conanfile.py

* Apply suggestions from code review

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
ericriff and others added 3 commits February 27, 2021 09:56
* libsmacker: add libsmacker recipe

* libsmacker: fix homepage

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>

* libsmacker: add -FS flag for msvc debug build

* smacker: add required_conan_version + fix license + use AUTORECONF env var

Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
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.