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

Option --confdir not added to Autostart config #5339

Closed
klada opened this issue Nov 28, 2016 · 4 comments
Closed

Option --confdir not added to Autostart config #5339

klada opened this issue Nov 28, 2016 · 4 comments
Labels
p4-low Low priority type:bug

Comments

@klada
Copy link

klada commented Nov 28, 2016

When launching the OwnCloud client with --confdir it creates the wrong Autostart config. The command line switch should also be appended to the OwnCloud binary when generating the Autostart config.

Expected behaviour

  1. Start OwnCloud with /path/to/owncloud.exe --confdir %APPDATA%\custom.ownCloud\

  2. Configure client and enable Autostart

  3. The full path to the client binary, including the --confdir %APPDATA%/ownCloud/ should be stored in HKCU\Software\Microsoft\Windows\CurrentVersion\Run\ownCloud (or whereever the autostart config is stored on other platforms).

Actual behaviour

When starting the client with --confdir %APPDATA%/ownCloud/ this config is omitted when creating Autostart entries.

@ckamm
Copy link
Contributor

ckamm commented Nov 29, 2016

Note that we want to generate the command line from the arguments that were parsed. Simply forwarding the command line arguments is not going to work if they contain relative paths.

@hodyroff
Copy link

@ckamm sounds like you do suggest this is not a bug, but a different command should be used?

@ckamm
Copy link
Contributor

ckamm commented Mar 10, 2017

@hodyroff No, I'm saying that forwarding the arguments that were passed to the owncloud binary to the autostart entry verbatim is not going to work. Getting this right is not complicated but needs a bunch of effort.

This is a low-severity issue: Users who need the client started with specific command line arguments are probably savy enough to set up autostart for themselves. The client could be nice and not set up autostart if it was started with command line arguments, or at least tell the user that what's happening is probably not what they want.

That could avoid frustrating advanced users without having a large, very rarely used chunk of code in the client that deals with this particular issue.

@felixboehm felixboehm added p4-low Low priority and removed sev4-low labels May 3, 2018
@TheOneRing
Copy link
Contributor

--confdir was removed #11142

@TheOneRing TheOneRing closed this as not planned Won't fix, can't repro, duplicate, stale Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p4-low Low priority type:bug
Projects
None yet
Development

No branches or pull requests

6 participants