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.

Sign up for GitHub

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

Jump to bottom

cpr: add cmake patch to create install target #2

Merged
merged 2 commits into from
Dec 13, 2019
Merged

cpr: add cmake patch to create install target #2

merged 2 commits into from
Dec 13, 2019

Conversation

madebr
Copy link

@madebr madebr commented Dec 12, 2019

Specify library name and version: cpr/1.3.0

See conan-io#450

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

@madebr madebr mentioned this pull request Dec 12, 2019
Merged
4 tasks
jgsogo
jgsogo reviewed Dec 13, 2019
View reviewed changes
Copy link
Owner

@jgsogo jgsogo left a comment

Choose a reason for hiding this comment

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

I have some questions, then on top of your changes I will modify the generators to use find_package in the test_package.

recipes/cpr/all/conanfile.py Show resolved Hide resolved
recipes/cpr/all/conanfile.py Show resolved Hide resolved
recipes/cpr/all/test_package/conanfile.py Outdated Show resolved Hide resolved
@jgsogo jgsogo merged commit 37e9c78 into jgsogo:recipe/cpr Dec 13, 2019
@jgsogo jgsogo mentioned this pull request Dec 13, 2019
Merged
@madebr madebr deleted the jgsogo_cpr branch December 13, 2019 17:36
jgsogo pushed a commit that referenced this pull request Dec 20, 2019
@uilianries
Check C++17 support (#2)
7fff920 
Signed-off-by: Uilian Ries <uilianries@gmail.com>
jgsogo pushed a commit that referenced this pull request Dec 30, 2019
@ericriff
Merge pull request #2 from uilianries/tinycbor
30f2075 
Fix Windows build for tinycbor
jgsogo pushed a commit that referenced this pull request Feb 12, 2020
Merge pull request #2 from conan-io/master
aa8d768 
merge
jgsogo pushed a commit that referenced this pull request Feb 12, 2020
@dheater
Merge pull request #2 from p-podsiadly/543-add-approvaltests.cpp
cea2ea2 
approvaltests.cpp: added config.yml, updated doctest too 2.3.6
jgsogo pushed a commit that referenced this pull request Mar 10, 2020
@SpaceIm
Merge pull request #2 from uilianries/jasper-2.0.16
d20a569 
Configure cmake module path for jasper
jgsogo pushed a commit that referenced this pull request Mar 23, 2020
@casabre
Merge pull request #2 from madebr/zyre_for_casabre
5882cdf 
zyre: use CONAN_PKG targets
jgsogo pushed a commit that referenced this pull request Apr 3, 2020
@ericriff
Merge pull request #2 from uilianries/blake2/20190723
b94376b 
Blake2/20190723
jgsogo pushed a commit that referenced this pull request Apr 3, 2020
@worldemar
Merge pull request #2 from conan-io/master
d326692 
pull from upstream
jgsogo pushed a commit that referenced this pull request Apr 30, 2020
@prince-chrismc
Merge pull request #2 from uilianries/cpprestsdk
cac3221 
External patches for cpprestsdk
jgsogo pushed a commit that referenced this pull request Aug 10, 2020
Merge pull request #2 from conan-io/master
939fecf 
merge master
jgsogo pushed a commit that referenced this pull request Aug 10, 2020
@intelligide
Merge pull request #2 from madebr/pr_lodepng
4870bc3 
lodepng: change version + fix shared + test_package
jgsogo pushed a commit that referenced this pull request Sep 10, 2020
@Hopobcn
Merge pull request #2 from madebr/pr_elfutils
53d9c56 
libelf: update to 0.180 + no more loadable modules

Works on this setup:
```
[settings]
arch=x86_64
arch_build=x86_64
build_type=Debug
compiler=gcc
compiler.libcxx=libstdc++
compiler.version=9
os=Linux
os_build=Linux
[options]
elfutils:shared=True
[build_requires]
[env]
```
jgsogo pushed a commit that referenced this pull request Jan 17, 2021
@prince-chrismc @madebr
@joxoby @SpaceIm
(conan-io#4045) Add/ignition math
6bd6f71 
* Add ignition-math

* Formatting

* Change version_mejor approach

* Address review comments

* Add import ConanInvalidConfiguration

* del fPIC in Windows

* Add shared and static support

* Add import glob

* Unconditially remove files

* Address review comments

* Remove fPIC option since it's always true

* Change test_package.cpp

* Change test CMakeLists

* Add regex patch

* Add friends function Angle definition patch

* Revert "Add friends function Angle definition patch"

This reverts commit 4e68918.

* Explicitly link libc++

* Revert "Explicitly link libc++"

This reverts commit f659367.

* Don't run git clone + support fPIC + create components (#1)

* Add cmake_wrapper

* Update recipes/ignition-math/all/CMakeLists.txt

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

* ignition-math: use ignition-cmake (#2)

* bump to 6.7.0

* bump 6.7.0

* Apply suggestions from code review

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

* remove pkgconf

* Apply suggestions from code review

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

Co-authored-by: Juan Oxoby <me@jmoxo.by>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Juan Oxoby <49600381+joxoby@users.noreply.github.com>
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
jgsogo pushed a commit that referenced this pull request Sep 6, 2021
@redradist @prince-chrismc
(conan-io#6083) Initial support for wasmtime package
cc01ea9 
* Initial support for wasmtime package

* Fixed build package on Linux

* Updates according the comments in review #0

* Update recipes/wasmtime/all/conanfile.py

Update license type

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

* Updates according the comments in review #1

* Updates according the comments in review #2

* Updates according the comments in review #3

* Updates according the comments in review #4

* Updates according the comments in review #5

* Next iteration of fixes

* Fixed cmake variable C_STANDARD -> CMAKE_C_STANDARD

* Added check on minimal version of conan

* Used copytree instead of copy individual files

* Fixed the build

* Added checking for architechure in validate(...)

* Updated receipt for new comments from @madebr

* Fixed calling method tools.cross_building(...)

* Update def package(self) according the comments in review

* Updated versions of wasmtime

* Updated forgot version ugrade

* Fixed sha256sum for macos

* Fixed SHA256 for Linux

* Updated all SHA256 to proer values

Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
jgsogo pushed a commit that referenced this pull request Sep 24, 2021
@rl-enjoyer @uilianries
@madebr @SSE4
(conan-io#6098) Add gsl/2.7
6bcd837 
* init commit

* adding stuff from https://github.com/Chrismarsh/conan-gsl/blob/stable/2.x/conanfile.py

* adding swp and swo files to gitignore. working on gsl

* commit before test

* gsl 2.7 , trying to get conanfile working

* commit before restart

* seems to work now?

* pep8 compliant

* adding license

* remove share directory..hopefully thats not a game breaking type thing

* no more conan center hook errors

* reverting gitignore

* adding version 2.6 in. (I make this change orthogonal to the conan-center-bot error)

* Build on Linux

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Build on Windows

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* adding lm flag to gsl autools flags due to use of math include

* adding v2.6 to config yml for gsl

* correcting include for printf in gsl test package

* gsl: fix MSVC + mingw@Windows

* working on changes from #2

* fixing mispelling

* adding back in 2.6 version to config yml, maybe will remove if too much of a burden to get working. addressing hook warnining

* addressing review feedback. still need 2 fix package constructon error for ppc darwin

* removing HAVE_DARWIN_IEEE_INTERFACE def when apple os

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

Co-authored-by: SSE4 <tomskside@gmail.com>

* Update recipes/gsl/all/conanfile.py

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

* gsl - using settings_build function in build_requrements

* gsl - fixing bug in windows shared renaming

* Update recipes/gsl/all/conanfile.py

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

Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Anonymous Maarten <anonymous.maarten@gmail.com>
Co-authored-by: SSE4 <tomskside@gmail.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
jgsogo pushed a commit that referenced this pull request Sep 29, 2021
@werto87
Merge pull request #2 from jgsogo/werto87-emsdk
34d4e37 
madebr changes + up version + use nodejs package
jgsogo pushed a commit that referenced this pull request Jan 20, 2022
@chusitoo
(conan-io#8648) Add flatbush lib
3bb9ad0 
* Add flatbush recipe

* Cleanup

* Extra line

* Put back sha256

* Removed unused property

* Fix for PR

* Fix cppstd whit gcc5

* Code review

* Trim trailing spaces

* Looks like apple-clang defaults to C++98

* Logical error

* Update define

* Code review #2

* Remove verbose flag
jgsogo pushed a commit that referenced this pull request Oct 4, 2022
@fpelliccioni @uilianries @prince-chrismc
(conan-io#13037) [Catch2] adapt recipe for Conan v2
361df20 
* [Catch2] adapt recipe for Conan v2

* fixes

* fixes

* fixes

* fixes

* fixes 4

* fixes 5

* fixes 6

* Update recipes/catch2/3.x.x/conanfile.py

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

* Update recipes/catch2/3.x.x/conanfile.py

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

* Update recipes/catch2/3.x.x/conanfile.py

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

* Update recipes/catch2/3.x.x/conanfile.py

* fix patches

* fix

* fix

* remove comments

* Update recipes/catch2/3.x.x/conanfile.py

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

* Update recipes/catch2/3.x.x/conanfile.py

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

* test package changes

* Move test package files (#2)

* move gtest

Signed-off-by: Uilian Ries <uilianries@gmail.com>

* Delete test_all.sh

* Use cmake targets

* Do not use settings

Signed-off-by: Uilian Ries <uilianries@gmail.com>

Signed-off-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jgsogo jgsogo jgsogo left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@madebr @jgsogo