Bug 605 - make install don't create piddir
Summary: make install don't create piddir
Status: CLOSED WONTFIX
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: Build system (show other bugs)
Version: -current
Hardware: All All
: P2 enhancement
Assignee: OpenSSH Bugzilla mailing list
URL:
Keywords:
Depends on:
Blocks: 606
  Show dependency tree / graph
 
Reported: 2003-06-26 17:46 AEST by Roumen Petrov
Modified: 2004-04-14 12:24 AEST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roumen Petrov 2003-06-26 17:46:11 AEST 
Command "make install" should create piddir.
Comment 1 Damien Miller 2003-06-26 21:48:05 AEST 
no - it is the job of the sysadmin to put piddir somewhere sane.
Comment 2 Roumen Petrov 2003-06-27 00:45:02 AEST 
In that case could we remove
(umask 022 ; $(srcdir)/mkinstalldirs $(DESTDIR)$(PRIVSEP_PATH))
because it is the job of the sysadmin to do this ;-) ?
Comment 3 Ben Lindstrom 2003-06-27 00:50:44 AEST 
I agree with Damien we should not be making the directory for the PID file.
If the admin sets a custom pid file path then they are on their own.
Comment 4 Tim Rice 2003-06-27 02:12:17 AEST 
We DO warn if the specified piddir doesn't exist.
....
if test ! -d $piddir ; then
AC_MSG_WARN([** no $piddir directory on this system **])
fi
....

That's a clue to ethier specify a different piddir or create it.
Comment 5 Roumen Petrov 2003-06-27 21:35:19 AEST 
What about to DO warn out of AC_ARG_WITH(pid-dir....) statement in configure ?
Comment 6 Damien Miller 2004-04-14 12:24:19 AEST 
Mass change of RESOLVED bugs to CLOSED