Hey, On 06/05/2014 08:38 AM, Trond Myklebust wrote: > On Thu, Jun 5, 2014 at 4:14 AM, Ditang Chen <chendt.fnst@xxxxxxxxxxxxxx> wrote: >> When configure the nfs-utils source code with "--with-statdpath=/foo" parameter， >> #define NSM_DEFAULT_STATEDIR "/foo" will be generated in support/include/config.h file, >> but Makefile will not create "/foo" dir. >> >> Signed-off-by: chendt.fnst@xxxxxxxxxxxxxx >> --- >> Makefile.am | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/Makefile.am b/Makefile.am >> index ae7cd16..5c9ef44 100644 >> --- a/Makefile.am >> +++ b/Makefile.am >> @@ -54,10 +54,10 @@ install-data-hook: >> touch $(DESTDIR)$(statedir)/xtab; chmod 644 $(DESTDIR)$(statedir)/xtab >> touch $(DESTDIR)$(statedir)/etab; chmod 644 $(DESTDIR)$(statedir)/etab >> touch $(DESTDIR)$(statedir)/rmtab; chmod 644 $(DESTDIR)$(statedir)/rmtab >> - mkdir -p $(DESTDIR)$(statedir)/sm $(DESTDIR)$(statedir)/sm.bak >> touch $(DESTDIR)$(statedir)/state >> - chmod go-rwx $(DESTDIR)$(statedir)/sm $(DESTDIR)$(statedir)/sm.bak $(DESTDIR)$(statedir)/state >> - -chown $(statduser) $(DESTDIR)$(statedir)/sm $(DESTDIR)$(statedir)/sm.bak $(DESTDIR)$(statedir)/state >> + mkdir -p $(DESTDIR)$(statdpath)/sm $(DESTDIR)$(statdpath)/sm.bak >> + chmod go-rwx $(DESTDIR)$(statdpath)/sm $(DESTDIR)$(statdpath)/sm.bak $(DESTDIR)$(statedir)/state >> + -chown $(statduser) $(DESTDIR)$(statdpath)/sm $(DESTDIR)$(statdpath)/sm.bak $(DESTDIR)$(statedir)/state >> > > The file 'state' is owned by rpc.statd. Isn't it therefore also > subject to the --with-statdpath modifier? --with-statedir and --with-statdpath are basically the same thing. When --with-statdpath is not specified, its set to the --with-statedir value, that's why this these mkdir,chmod and chown were not needed. But I guess if you do want split out the statdpath from the statedir those mkdir,chmod and chown need to happen. > > At one point, the Fedora RPM packager even used to put it in > /var/lib/nfs/statd (as the manpage says it should) although that seems > to be broken now. > It still does... So does RHEL... We have being going that for years... How is that broken? steved. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
