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

prefer ~/.local/share/ as the data directory (WIP) #828

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

grawity
Copy link
Contributor

@grawity grawity commented Mar 12, 2023

For newly created DBs, following the general practice on Linux (and tpm2-tss, which already uses ~/.local/share/tpm2-tss/ for FAPI).

The old ~/.tpm_pkcs11/ path is supposed to be still used if it exists, assuming I got the logic right, but I'm really not sure if the db.c part of it is correct.

@codecov
Copy link

codecov bot commented Mar 12, 2023

Codecov Report

Merging #828 (a6f124c) into master (3987372) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #828      +/-   ##
==========================================
- Coverage   72.32%   72.27%   -0.05%     
==========================================
  Files          33       34       +1     
  Lines        9579     9794     +215     
==========================================
+ Hits         6928     7079     +151     
- Misses       2651     2715      +64     
Impacted Files Coverage Δ
src/lib/db.c 66.38% <0.00%> (+0.34%) ⬆️

... and 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Dash is used as the separator for similarity with the path already used
by FAPI (~/.local/share/tpm2-tss/).

Signed-off-by: Mantas Mikulėnas <[email protected]>
@williamcroberts williamcroberts added this to the 1.9.1 milestone Sep 5, 2024
@williamcroberts williamcroberts self-assigned this Sep 5, 2024
@williamcroberts williamcroberts merged commit d5bc3d3 into tpm2-software:master Sep 6, 2024
13 checks passed
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.

2 participants