Skip to content

Add command line switch for passing the socket path#42

Open
dragotin wants to merge 1 commit into
mainfrom
addSocketPathSwitch
Open

Add command line switch for passing the socket path#42
dragotin wants to merge 1 commit into
mainfrom
addSocketPathSwitch

Conversation

@dragotin

Copy link
Copy Markdown
Member

Before it was hardcoded

Comment thread src/openvfsfuse/openvfsfuse.cpp Outdated
if (socketPath.empty()) {
// only use for debugging
std::cerr << "Attention: Debug path used for socket!" << std::endl;
socketPath = "/run/user/1000/OpenCloud/socket";

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rather keep the default/fallback value in the declaration of sockerPath attribute in the openVFSfuse_Args structure declaration

@dragotin dragotin force-pushed the addSocketPathSwitch branch from 72d6e6f to 5b59056 Compare June 8, 2026 15:04
@dragotin dragotin force-pushed the addSocketPathSwitch branch from 5b59056 to 782fa54 Compare June 8, 2026 15:05
@dragotin dragotin requested a review from mgallien June 8, 2026 15:05

@mgallien mgallien left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good

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