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

Implemented rewrite to real path in dump_path #315

Closed
wants to merge 1 commit into from
Closed

Conversation

Mic92
Copy link
Member

@Mic92 Mic92 commented Apr 18, 2024

This fixes #254 by adding a function, get_real_store_dir, to libnixstore to support non-root nix stores, where the physical and logical paths differ.

@Mic92 Mic92 force-pushed the virtual-store branch 3 times, most recently from df50080 to d255e31 Compare April 18, 2024 09:21
@Mic92 Mic92 force-pushed the virtual-store branch 3 times, most recently from fa3ba82 to 38550fc Compare April 18, 2024 10:44
@Thesola10
Copy link

LGTM, mind if I re-sign this and push it into my original PR?

@Mic92
Copy link
Member Author

Mic92 commented Apr 18, 2024

@Thesola10 sure.

This fixes #254 by adding a function, get_real_store_dir, to libnixstore
to support non-root nix stores, where the physical and logical paths
differ.

prettier code

Applied formatter and clippy recommendation

Fixed string comparison

Added test cases, fixed web serve paths
@Mic92 Mic92 closed this Apr 18, 2024
@Mic92 Mic92 deleted the virtual-store branch April 23, 2024 10:23
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.

Allow specifying alternative store root
2 participants