Arch Linux User Repository

There's a bigger issue than just convenience when removing system users. The id of a user should not be reused because if there are leftover files owned by the old user, they will now be owned by the one that replaces it. So you actually want to keep system users indefinitely for security reasons!

EDIT: I misread the post_remove script as deleting the user, when instead it deletes the group. I think the same issue applies if files are created with the group set as mock.

IMHO it makes sense not to remove mock group in post_remove(). Otherwise it makes harder to re-install this package

post_remove() {
  getent group mock >/dev/null && groupdel mock
  echo 'Make sure to empty out /var/lib/mock and /var/cache/mock'
}

@anatolik

It would be great if the package installed a default config file.

Which config should be used as default? Fedora? CentOS? ...? IMHO, none of them make sense as default on Arch Linux systems.

I installed the package and at the first run it says:

$ mock
ERROR: Could not find required config file: /etc/mock/default.cfg
ERROR:   Did you forget to specify the chroot to use with '-r'?
ERROR:   If you're trying to specify a path, include the .cfg extension, e.g. -r ./target.cfg

It would be great if the package installed a default config file.

@pnorcks: should be OK now, thanks for the report

When trying to use mock 1.4.7-1, I run into an exception:

Traceback (most recent call last):
File "/usr/bin/mock", line 79, in <module>
from mockbuild import util
File "/usr/lib/python3.6/site-packages/mockbuild/util.py", line 41, in <module>
from pyroute2 import IPRoute
ModuleNotFoundError: No module named 'pyroute2'

@Duologic: I'm not sure what you mean by "collaborate". Can you clarify?

I really prefer to run mock with 'sudo', both for reason I mentioned in my previous comment, and also because it makes clear that I'm running the program as root. Running mock with 'usermode' blurs that distinction...

Can you collaborate on getting it running in 'usermode'?

I adopted the package.

Note that I removed the 'usermode' dependency from this package, since I prefer running mock with 'sudo', and it simplifies packaging.

@roheim: Thanks!

I'm now seeing an install failure. Error output is below.

error: failed to commit transaction (conflicting files)
mock: /usr/sbin exists in filesystem
mock: /usr/sbin/mock exists in filesystem
Errors occurred, no packages were upgraded.