Package Details: yandex-disk 0.1.6.1074-1

Git Clone URL: https://aur.archlinux.org/yandex-disk.git (read-only, click to copy)
Package Base: yandex-disk
Description: Yandex.Disk keeps your files with you at all times.
Upstream URL: http://disk.yandex.ru/
Licenses: custom
Submitter: ziggi
Maintainer: sistematico
Last Packager: sistematico
Votes: 126
Popularity: 1.74
First Submitted: 2013-08-27 11:59
Last Updated: 2021-06-10 22:08

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 Next › Last »

ziggi commented on 2017-05-03 20:55

> In the systemd service file I've found the following line:
> ExecReload=/usr/bin/killall -qw yandex-disk
>
> I really appreciate your work and I beg your pardon, ziggi, but are
> you kidding me? :) Why bother implementing a "reload" command if it
> merely kills ALL the processes that have "yandex-disk" in their names?
>
> There is a wonderful systemctl command named 'reload-or-restart' that
> restarts a service if it doesn't have the "reload" command
> implemented, so what you have done is certainly a misuse of the
> ExecReload parameter. If you can prove me wrong please let me know.

I kill the process and him will be started automatically soon.

> Also in the docs they mention that the daemon has to be stopped with
> the command "yandex-disk stop": https://yandex.com/support/disk/cli-
> clients.html#cli-daemon__cli-commands . There is the "ExecStop"
> parameter (man 5 systemd.service) which one can utilize to achieve the
> desired behaviour. Well I guess nothing bad will happen if the daemon
> is stopped with the SIGTERM signal, but nevertheless.

"yandex-disk stop" doesn't kill the systemd daemon.

> And I can't keep myself from mentioning the "RestartSec" parameter
> which is set to 60. Do you have any special reason to override the
> default systemd behaviour? I mean, is anything bad going to happen if
> it is restarted with, let's say, a second timeout?

Yes, without some delay daemon won't start. And I don't know exactly how much him need.

> The last but not the least. You have that weird shell script that
> apparently parses the configuration file and appends the parsed values
> to the yandex-disk command line. Well, to begin with, it happily
> parses all the commented lines producing stuff like `--#proxy=socks4`.
> And why did you even bother with that? The daemon reads that file by
> default (unless you specify the "-c"/"--config" option).

Because "-c" option is not supported with "--no-daemon" mode. But I found the way how it can be fixed.

P.S. This service works good but if you want to suggest any changes - just show me your service configuration file. Because any program has specific behavior and any decision has specific causes. Anyway, thanks for reporting.

mexus commented on 2017-05-03 19:17

Hi ziggi!

In the systemd service file I've found the following line:
ExecReload=/usr/bin/killall -qw yandex-disk

I really appreciate your work and I beg your pardon, ziggi, but are you kidding me? :) Why bother implementing a "reload" command if it merely kills ALL the processes that have "yandex-disk" in their names?

There is a wonderful systemctl command named 'reload-or-restart' that restarts a service if it doesn't have the "reload" command implemented, so what you have done is certainly a misuse of the ExecReload parameter. If you can prove me wrong please let me know.

Also in the docs they mention that the daemon has to be stopped with the command "yandex-disk stop": https://yandex.com/support/disk/cli-clients.html#cli-daemon__cli-commands . There is the "ExecStop" parameter (man 5 systemd.service) which one can utilize to achieve the desired behaviour. Well I guess nothing bad will happen if the daemon is stopped with the SIGTERM signal, but nevertheless.

And I can't keep myself from mentioning the "RestartSec" parameter which is set to 60. Do you have any special reason to override the default systemd behaviour? I mean, is anything bad going to happen if it is restarted with, let's say, a second timeout?

The last but not the least. You have that weird shell script that apparently parses the configuration file and appends the parsed values to the yandex-disk command line. Well, to begin with, it happily parses all the commented lines producing stuff like `--#proxy=socks4`. And why did you even bother with that? The daemon reads that file by default (unless you specify the "-c"/"--config" option).

Thanks for your attention.

mamantoha commented on 2016-07-27 22:54

ziggi, I just checked sha512sum of yandex-disk.install. And it`s correct. Probably something wrong with my system.

ziggi commented on 2016-07-27 22:45

mamantoha, everything works, I have tested it.

mamantoha commented on 2016-07-27 22:09

Can't install the latest version:

==> Validating source files with sha512sums...
yandex-disk_0.1.5.948_amd64.deb ... Passed
yandex-disk.install ... FAILED
yandex-disk.service ... Passed
==> ERROR: One or more files did not pass the validity check!
==> ERROR: Makepkg was unable to build yandex-disk.
==> Restart building yandex-disk ? [y/N]
==> ------------------------------------

mdeni commented on 2016-01-15 21:16

kfr2359, thank you for this information. I will wait for new kernel in stable repo. So I hope this update will solve this issue on my system too.

kfr2359 commented on 2016-01-15 13:23

Installing 4.4.0-2 kernel (available on testing repository) fixes the problem mdeni and I have faced.

kfr2359 commented on 2016-01-05 15:41

Have got exactly the same problem mdeni had described. Maybe it's somehow connected to the recent kernel update?

mdeni commented on 2016-01-02 20:44

I've got a bug. I start the daemon and it works fine about 1 minute then there becomes a lot of processes and two of them using 100% cpu each. And nothing works as log doesn't change. Also I cannot kill or stop processes and then reboot system. Only hard reboot with power button pressed helps. Could you help me to solve this problem or at least investigate and report a bug to devs (I'm newbie to Arch). Thank you.

salikovpro commented on 2015-10-11 09:10

Please.

in PkBuild:

update:

pkgver=0.1.5.940 //not 905
md5sums = ('f4f3b9093ff30faa16a9df1504025947')