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.

Sign up for GitHub

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

Jump to bottom

Incorrect podman-connections.json path in Windows #2025

Closed
vvreutskiy opened this issue May 30, 2024 · 2 comments · Fixed by #2027
Closed

Incorrect podman-connections.json path in Windows #2025

vvreutskiy opened this issue May 30, 2024 · 2 comments · Fixed by #2027
Assignees
@Luap99

Comments

@vvreutskiy
Copy link

After updating podman to v.5.1.0 on windows 10 it can't find saved connecitons in C:\Users\user\AppData\Roaming\containers\podman-connections.json, but tries to find them in C:\Users\user\AppData\Roamingcontainers\podman-connections.json (note the Roamingcontainers incorrect path)

As a workaround I've copied "correct" podman-connections.json to incorrect path, but it is definetly wrong.
@vvreutskiy
Copy link
Author

Looks like missing path separator in pkg/config/config_windows.go

@rhatdan
Copy link
Member

rhatdan commented May 30, 2024

Please open a PR to fix?

@Luap99 Luap99 self-assigned this May 30, 2024
Luap99 added a commit to Luap99/common that referenced this issue May 30, 2024
@Luap99
fix windows config path regression
0093c7b 
The path no longer included the path seperator between the env and our
config dir. The regession was added in commit 6c651df.

Fixes containers#2025

Signed-off-by: Paul Holzinger <[email protected]>
Luap99 added a commit to Luap99/common that referenced this issue May 30, 2024
@Luap99
fix windows config path regression
7490d3d 
The path no longer included the path separator between the env and our
config dir. The regression was added in commit 6c651df.

Fixes containers#2025

Signed-off-by: Paul Holzinger <[email protected]>
@Luap99 Luap99 mentioned this issue May 30, 2024
Merged
Luap99 added a commit to Luap99/common that referenced this issue May 31, 2024
@Luap99
fix windows config path regression
3e57a6a 
The path no longer included the path separator between the env and our
config dir. The regression was added in commit 6c651df.

Fixes containers#2025

Signed-off-by: Paul Holzinger <[email protected]>
hswong3i pushed a commit to alvistack/containers-common that referenced this issue Jun 5, 2024
@Luap99 @hswong3i
fix windows config path regression
a113b49 
The path no longer included the path separator between the env and our
config dir. The regression was added in commit 6c651df.

Fixes containers#2025

Signed-off-by: Paul Holzinger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Luap99 Luap99

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix windows config path regression Luap99/common
3 participants
@rhatdan @vvreutskiy @Luap99