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

Add libbacktrace package #174

Merged
merged 13 commits into from
Apr 2, 2020
Merged

Add libbacktrace package #174

merged 13 commits into from
Apr 2, 2020

Conversation

Bjoe
Copy link

@Bjoe Bjoe commented Mar 18, 2020

Copy link

@NeroBurner NeroBurner left a comment

Choose a reason for hiding this comment

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

Small comments and some things that are not clear to me

And please squasch commits before merge

# )
#endif()

hunter_configuration_types(libbacktrace CONFIGURATION_TYPES Release)

Choose a reason for hiding this comment

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

Why hard code the configutation types. If debug is not supportet please add a comment about the reasons

Copy link
Author

Choose a reason for hiding this comment

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

Because it uses the autotools scheme and this can only build one build type. If you remove this it will show an error:

[hunter ** FATAL ERROR **] Autotools PACKAGE_CONFIGURATION_TYPES has 2 elements: Release                                                                                                     
[hunter ** FATAL ERROR **] Debug. Only 1 is allowed                                                                                                                                          
[hunter ** FATAL ERROR **] [Directory:/source/hunter/_testing/Hunter/_Base/4ed7980/0fca6e7/2f6b703/Build/libbacktrace]                                                                       
                                                                                                                                                                                             
------------------------------ ERROR -----------------------------                                                                                                                           
    https://hunter.readthedocs.io/en/latest/reference/errors/autools.package.configuration.types.html                                                                                        
------------------------------------------------------------------                                                                                                                           
                                                                                                                                                                                             
CMake Error at /source/hunter/_testing/Hunter/_Base/Download/Hunter/unknown/4ed7980/Unpacked/cmake/modules/hunter_error_page.cmake:12 (message):                                             
Call Stack (most recent call first):                                                                                                                                                         
  /source/hunter/_testing/Hunter/_Base/Download/Hunter/unknown/4ed7980/Unpacked/cmake/modules/hunter_fatal_error.cmake:20 (hunter_error_page)                                                
  /source/hunter/_testing/Hunter/_Base/Download/Hunter/unknown/4ed7980/Unpacked/cmake/modules/hunter_autotools_configure_command.cmake:164 (hunter_fatal_error)                              
  /source/hunter/_testing/Hunter/_Base/Download/Hunter/unknown/4ed7980/Unpacked/cmake/modules/hunter_autotools_project.cmake:158 (hunter_autotools_configure_command)                        
  CMakeLists.txt:40 (hunter_autotools_project)

libbacktrace

.. index::
single: unsorted ; libbacktrace

Choose a reason for hiding this comment

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

Can this be sorted into a category?

Copy link
Author

Choose a reason for hiding this comment

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

Ok, I add Logging as category ....

)

if(WIN32)
set(CMAKE_FIND_LIBRARY_SUFFIXES ${_libbacktrace_ORIG_CMAKE_FIND_LIBRARY_SUFFIXES})

Choose a reason for hiding this comment

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

Why is the variable reset again. Could you explain?

Copy link
Author

Choose a reason for hiding this comment

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

It should be "reset" because other find_library() in other modules etc. shouldn't be affected ! See also FindBoost and other find modules. They all do it that way ... I added a comment.

7c7db4d096d96f329c453fb97b7a456fe823e943
)

#if (ANDROID OR IOS)

Choose a reason for hiding this comment

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

Remove if unused

Copy link
Author

Choose a reason for hiding this comment

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

Arg, I always forget this ... thanks for the hint 👍 ... I removed it.

# * https://github.com/cpp-pm/gate
include("../common.cmake")

project(libbacktrace)

Choose a reason for hiding this comment

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

I think previous examples are called download-foo
in your case download-libbacktrace

Copy link
Author

Choose a reason for hiding this comment

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

Ok fixed.

Copy link

@NeroBurner NeroBurner left a comment

Choose a reason for hiding this comment

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

LGTM

@rbsheth rbsheth merged commit ed320d8 into cpp-pm:master Apr 2, 2020
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.

3 participants