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

Added NanoRange to the index #157

Open
wants to merge 5 commits into
base: master
from

Conversation

@paulbendixen
Copy link

commented Oct 1, 2019

Specify library name and version: NanoRange/20191001

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes even though they are WRONG and everybody should use tabs and camelCase.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.
@CLAassistant

This comment has been minimized.

Copy link

commented Oct 1, 2019

CLA assistant check
All committers have signed the CLA.

@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 1, 2019

Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue.

Co-Authored-By: Michael "Croydon" Keck <git@cr0ydon.com>
@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 1, 2019

Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue.

@@ -0,0 +1,3 @@
versions:
"20191001":

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member
Suggested change
"20191001":
"20191001":
@@ -0,0 +1,3 @@
versions:
"20191001":
folder: all

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member
Suggested change
folder: all
folder: all
recipes/nanorange/all/test_package/conanfile.py Outdated Show resolved Hide resolved
recipes/nanorange/all/test_package/conanfile.py Outdated Show resolved Hide resolved
recipes/nanorange/all/test_package/conanfile.py Outdated Show resolved Hide resolved
recipes/nanorange/all/test_package/conanfile.py Outdated Show resolved Hide resolved

add_executable(example example.cpp)
target_link_libraries(example ${CONAN_LIBS})
#set_property( TARGET example PROPERTY CXX_STANDARD 17)

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member

Remove these comments

This comment has been minimized.

Copy link
@paulbendixen

paulbendixen Oct 1, 2019

Author

I need another way of saying that this library must be compiled with -s compiler.cppstd=(>= c++17)

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member

please, take a look: #54
anyway, if your package requires c++17, uncomment this line to force the compiler compatibility.

@@ -0,0 +1,4 @@
sources:
"20191001":

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member
Suggested change
"20191001":
"20191001":
@@ -0,0 +1,4 @@
sources:
"20191001":
url: "https://github.com/tcbrindle/NanoRange/archive/abc34279d71369527e4da43cc4d02cf77446f79a.zip"

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member
Suggested change
url: "https://github.com/tcbrindle/NanoRange/archive/abc34279d71369527e4da43cc4d02cf77446f79a.zip"
url: "https://github.com/tcbrindle/NanoRange/archive/abc34279d71369527e4da43cc4d02cf77446f79a.tar.gz"
sources:
"20191001":
url: "https://github.com/tcbrindle/NanoRange/archive/abc34279d71369527e4da43cc4d02cf77446f79a.zip"
sha256: "c7f1223491560090369c0ba558193e925a8005a7e909c86d11403880b53cae00"

This comment has been minimized.

Copy link
@uilianries

uilianries Oct 1, 2019

Member
Suggested change
sha256: "c7f1223491560090369c0ba558193e925a8005a7e909c86d11403880b53cae00"
sha256: "799431094ab784eb3cdbcd6de6681ff5d8b2b28e1a1e26e0a3a94fbfbfcd7e75"
@uilianries

This comment has been minimized.

Copy link
Member

commented Oct 1, 2019

Hi @paulbendixen !

Thanks for contributing to Conan Center Index, could you please send an inclusion request on #4 .

Co-Authored-By: Uilian Ries <uilianries@gmail.com>
@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 1, 2019

Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue.

@paulbendixen

This comment has been minimized.

Copy link
Author

commented Oct 1, 2019

Hi @paulbendixen !

Thanks for contributing to Conan Center Index, could you please send an inclusion request on #4 .

#4 (comment)

@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 2, 2019

Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue.

@lasote lasote removed the No Beta user label Oct 10, 2019
@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 10, 2019

nanorange/20191001:

  • Error processing recipe: Windows x86_64, Debug, Visual Studio 14, MTd
    Unexpected error happened:
ERROR: nanorange/20191001: Error in configure() method, line 21
	if not self.settings.compiler.cppstd in ["17", "20", "gnu17", "gnu20"]:
	ConanException: Invalid setting 'gnu17' is not a valid 'settings.compiler.cppstd' value.
Possible values are ['14', '17', '20', 'None']
Read "http://docs.conan.io/en/latest/faq/troubleshooting.html#error-invalid-setting"

Change the detection of the standard used, so that the code will hopefully run on msvc
@conan-center-bot

This comment has been minimized.

Copy link
Collaborator

commented Oct 18, 2019

Sorry, the system is under maintenance and it doesn't accept builds right now. Thanks for your understanding and help with the EAP program!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.