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

Passage does not recognize git repository in .passage dir #39

Open
mutlusun opened this issue Jan 18, 2023 · 1 comment · May be fixed by #40
Open

Passage does not recognize git repository in .passage dir #39

mutlusun opened this issue Jan 18, 2023 · 1 comment · May be fixed by #40

Comments

@mutlusun
Copy link

mutlusun commented Jan 18, 2023

Hello,

Many thanks for your work on age and passage. I recently switched to both from gpg and pass. Key management is much easier with age! :)

Instead of putting .passage/store/ into a git repository, I track the whole .passage dir (identities are stored on a yubikey). The git subcommand does not recognize that the passage dir is a repository.

I narrowed it down to line 33 in password-store.sh:

[[ $(git -C "$INNER_GIT_DIR" rev-parse --is-inside-work-tree 2>/dev/null) == true ]] || INNER_GIT_DIR=""

This test somehow fails (and resets the variable to "") even though executing this test manually results in the correct true result. I do not understand why this test does not detect correctly that the .passage dir is a git repository in the script but executing it manually gives the correct result.

The GIT_CEILING_DIRECTORIES seems to be correctly set to the home directory.

Can somebody help me out here?

Thanks for your help!

@mutlusun
Copy link
Author

I have mistaken. The GIT_CEILING_DIRECTORIES stops in the store directory. Thus, the .passage directory is not included.

@mutlusun mutlusun linked a pull request Jan 19, 2023 that will close this issue
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 a pull request may close this issue.

1 participant