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

Set minimum criteria for system package #3

Merged
merged 2 commits into from
Jul 20, 2020
Merged

Set minimum criteria for system package #3

merged 2 commits into from
Jul 20, 2020

Conversation

cottsay
Copy link
Contributor

@cottsay cottsay commented Jul 17, 2020

There are some features in 1.5.X that I think we'll want to leverage.

Also work around what I believe is a packaging bug in Ubuntu Eoan and newer. I'm not 100% certain, but I believe that libbenchmark_main doesn't work correctly when it's not a shared library.

https://bugs.launchpad.net/ubuntu/+source/benchmark/+bug/1887872

This means that right now, none of the supported platforms have an acceptable version of benchmark available as a system package.

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

There are some features in 1.5.X that I think we'll want to leverage.

Also work around what I believe is a packaging bug in Ubuntu Eoan and
newer. I'm not 100% certain, but I believe that libbenchmark_main
doesn't work correctly when it's not a shared library.

This means that right now, none of the supported platforms have an
acceptable version of `benchmark` available as a system package.
However, Fedora's package seems to work OK and I was able to confirm
that this package behaves correctly in the presence of the system
package.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay added the bug Something isn't working label Jul 17, 2020
@cottsay cottsay self-assigned this Jul 17, 2020
@cottsay
Copy link
Contributor Author

cottsay commented Jul 17, 2020

Another packaging bug, this time in Fedora: https://bugzilla.redhat.com/show_bug.cgi?id=1858127

Copy link
Contributor

@hidmic hidmic left a comment

Choose a reason for hiding this comment

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

LGTM

CMakeLists.txt Show resolved Hide resolved
Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay merged commit 96e59bd into main Jul 20, 2020
@delete-merged-branch delete-merged-branch bot deleted the min_version branch July 20, 2020 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants