Comment # 17 on bug 4293 from Ben Rosser

> 2) rpmlint incorrect-fsf-address errors.

tilem is definitely quiet upstream. I have filed a bug on their issue tracker
about this though: https://sourceforge.net/p/tilem/bugs/11/. My apologies for
not noticing this before; I missed it because the warning only appears in the
debuginfo package. My bad!

> 1) mv, mkdir, & cp are part of coreutils, so we should buildrequres coreutils

Coreutils is one of those packages that it is safe to assume is always
installed. I just asked about this in #fedora-devel to confirm that this is
still the case (as I know this policy has changed in the last few years). As
was pointed out by nirik, rpm itself depends on coreutils so it must always be
installed on a builder: check "rpm -q rpm --requires". :)

https://fedoraproject.org/wiki/Packaging:Guidelines#Build-Time_Dependencies_.28BuildRequires.29
also says that the tools to run "simple shell scripts" should be assumed to be
available, although that's kind of a vague guideline.

Anyway, I don't think it hurts to BR coreutils-- I can add it if you think it's
best. But I do not believe it should be necessary.

> 2) mv ,mkdir, & cp should be referred to by their macro names: %{__mv}, %{__mkidr}, and %{__cp}. Touch does not have a macro.

As Andrea pointed out, this is something that shouldn't be done according to
https://fedoraproject.org/wiki/Packaging:Guidelines#Macros, unless there is a
reason to assume that the location of the coreutils binaries should be
configurable.