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

Renamed libsexp to libsexpp; integrated an option to use libsexpp provided by the system #2103

Merged
merged 5 commits into from
Jun 28, 2023

Conversation

maxirmx
Copy link
Member

@maxirmx maxirmx commented Jun 23, 2023

  1. Renamed libsexp to libsexpp (name conflict libsexp (rnpgp/sexp and mjsottile/sfsexp) sexpp#45, Rename repository to sexpp sexpp#49)
  2. Integrated an option to use libsexpp provided by the system (add option to build with system sexp library #2102)
  3. Bumped install-nix-action version (macOS Nix builld is broken #2104)

@maxirmx maxirmx changed the title Added an option to use system-provided libsexp Finalize an option to use libsexp provided by the systen Jun 23, 2023
@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch from 7be61a7 to e65e510 Compare June 23, 2023 14:15
@maxirmx maxirmx changed the title Finalize an option to use libsexp provided by the systen Finalize the option to use libsexp provided by the systen Jun 23, 2023
@maxirmx maxirmx changed the title Finalize the option to use libsexp provided by the systen Finalize the option to use libsexp provided by the system Jun 23, 2023
@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch from 5309cda to c1357ef Compare June 23, 2023 14:44
@codecov
Copy link

codecov bot commented Jun 23, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (152ab7c) 83.82% compared to head (9624962) 83.81%.

❗ Current head 9624962 differs from pull request most recent head 9206d03. Consider uploading reports for the commit 9206d03 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2103      +/-   ##
==========================================
- Coverage   83.82%   83.81%   -0.01%     
==========================================
  Files         161      161              
  Lines       32405    32405              
==========================================
- Hits        27163    27161       -2     
- Misses       5242     5244       +2     
Impacted Files Coverage Δ
src/librekey/g23_sexp.hpp 100.00% <ø> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch 3 times, most recently from 62fb505 to a416ef8 Compare June 26, 2023 19:02
@maxirmx maxirmx changed the title Finalize the option to use libsexp provided by the system Renamed libsexp to libsexpp; integrated an option to use libsexpp provided by the system Jun 26, 2023
@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch 2 times, most recently from 3d3c12b to dcccf5e Compare June 27, 2023 06:42
@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch 4 times, most recently from 1c571cc to 6f110b0 Compare June 27, 2023 21:22
@maxirmx maxirmx force-pushed the maxirmx-system-sexp branch from 6f110b0 to 9206d03 Compare June 28, 2023 06:51
@maxirmx maxirmx marked this pull request as ready for review June 28, 2023 08:09
@ronaldtse ronaldtse merged commit 82d9747 into main Jun 28, 2023
@ronaldtse ronaldtse deleted the maxirmx-system-sexp branch June 28, 2023 10:37
@ronaldtse
Copy link
Contributor

Thank you @maxirmx !

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