[Bug 509739] Review Request: daemonize - daemonize runs a command as a Unix daemon.

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=509739





--- Comment #6 from Jussi Lehtola <jussi.lehtola at iki.fi>  2009-07-05 15:54:26 EDT ---
(In reply to comment #5)
> The eventual goal for this package is EPEL, should I use a combined Fedora/EPEL
> or should I maintain two seperate spec files?  

EPEL is a part of Fedora, not RHEL. And the primary target of Fedora is Fedora,
EPEL is sort of an addon.

Also,
 install -m 0644 daemonize.1 %{buildroot}%{_mandir}/man1
should be
 install -p -m 0644 daemonize.1 %{buildroot}%{_mandir}/man1
to preserve timestamps, see
http://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps

Quite often one needs to add INSTALL="install -p" as an argument to 'make
install' to make it preserve time stamps too.

Is the man page not installed automatically by make install..?

P.S. You don't have to create the directory, instead of 
 mkdir -p %{buildroot}%{_mandir}/man1
 install -p -m 0644 daemonize.1 %{buildroot}%{_mandir}/man1
you can
 install -D -p -m 0644 daemonize.1 %{buildroot}%{_mandir}/man1/daemonize.1

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.