Package Base Details: monero

Git Clone URL: https://aur.archlinux.org/monero.git (read-only)
Submitter: anonimal
Maintainer: anonimal
Last Packager: anonimal
Votes: 83
Popularity: 1.386100
First Submitted: 2016-09-21 06:24
Last Updated: 2019-07-19 21:22

Packages (2)

Latest Comments

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

vasily commented on 2017-10-28 08:17

Hello,

The systemd unit file in v.0.11.1.0 does not work, as it tells the
daemon to create its PID file in /var/run, which is owned by root.

This commit:

https://github.com/monero-project/monero/commit/4c58b7edb862a96623651d0387f150417ada92ad

tells systemd to create a directory in /run for monerod.service with the
correct ownership and that fixes the problem.
But it isn't in a release yet.

Another solution is to run monerod in foreground mode (not daemon) with
--non-interactive and drop the whole PID file business. Systemd already
tracks whether the service is running and additional side mechanism for
the same thing (the PID file) is not necessary.

Like this:


[Unit]
Description=Monero Full Node
After=network.target

[Service]
User=monero
Group=monero
WorkingDirectory=~

ExecStart=/usr/bin/monerod --config-file /etc/monerod.conf --non-interactive

[Install]
WantedBy=multi-user.target

anonimal commented on 2017-09-26 06:21

@redfish

1. You really should lay of the name calling; it only makes you look childish. Accusing me of being "emotional" and "snarky" is both a baseless accusation and unwarranted opinion. I had merely asked a question.

2. More importantly, the "point" was obviously made but upstream was only merged 15 hours ago - so there was no possibility of using an upstream service file that had not yet been approved. And even without this very *minor* service-file patch, nothing is broken, so ultimately there is no real reason to justify your attack.

3. The conf is still identical. Updating the install source directory is not unwarranted though but doing so *now* is - because we are not pulling from master but from a tagged release. In the next release, the PKGBUILD should install from source directory.

redfish commented on 2017-09-24 15:38

@majewsky: running fine in foreground mode *via* systemd? IIRC, that did not work for me, because without --detach, monerod tried to open console and failed. The current way (forking with PIDfile) does work, and is the right way, although I think there is a race between grandchild process creating the pid file and systemd reading it (child needs to wait for child before exiting).

redfish commented on 2017-09-24 15:35

@anonimal: The point is that the package should not maintain a forked .service and .conf, not that the package is not in sync with upstream. The package should use as much of upstream as possible, so please remove the .conf and .service from here and in package() install those files from upstream.

No need to get emotional. Read what RuntimeDirectory= does, before getting snarky. https://github.com/radfish/monero/blob/df322ddde5b4550194317c1ea2837f70c8c2f034/utils/systemd/monerod.service#L9
Yes, I did test (on Arch).

majewsky commented on 2017-09-16 17:06

Why does the service file use Type=forking? monerod is running fine on my box without the --pidfile and --detach switches, without PIDFile= and with Type=simple.

anonimal commented on 2017-09-12 04:51

@redfish:

1. I updated the service file 2 days before you opened your PR. See https://aur.archlinux.org/cgit/aur.git/commit/?h=monero&id=14732d23196b9d53b17e3909867943e60231956e. Do you even test your patches? monerod can't write to /run because it does not have permission. I'll comment in your PR.

2. The conf is identical to the repo's conf.

@Xavion: thanks, that's a typo inherited from the previous packager. I missed it, will correct.

Xavion commented on 2017-09-10 00:50

FYI, there's a little spelling mistake in the description. The first 'e' in the word 'deaemon' should be removed.

redfish commented on 2017-09-09 17:28

@anonimal: we should drop .conf and .service from pkgs and use upstream's, because upstream updates them. I just did that for the git pkg: https://aur.archlinux.org/cgit/aur.git/commit/?h=bitmonero-git&id=ba740afd3e07a2a9ed5cad0bfd1db8c9016db005

anonimal commented on 2017-09-08 21:10

>need to install rhash, please update the dependencies.

Run `pacman -Syu` before installing this package. rhash is a new CMake dependency, not a monero dependency.

acheloos commented on 2017-09-08 11:50

> cmake: error while loading shared libraries: librhash.so.0: cannot open shared object file: No such file or directory

need to install rhash, please update the dependencies.