* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf @ 2015-12-22 17:12 Jonathan Ben-Avraham 2015-12-22 17:28 ` Yann E. MORIN 2015-12-22 19:53 ` Thomas Petazzoni 0 siblings, 2 replies; 6+ messages in thread From: Jonathan Ben-Avraham @ 2015-12-22 17:12 UTC (permalink / raw) To: buildroot From: Jonathan Ben Avraham <yba@tkos.co.il> Installs /etc/nfsmount.conf on the target, mainly to be able to configure the the NFS version that mount.nfs uses by default. Without this ability you are likely to encounter mount errors when automounting directories from NFS version 3 servers. Signed-off-by: Jonathan Ben Avraham <yba@tkos.co.il> --- package/nfs-utils/nfs-utils.mk | 2 ++ 1 file changed, 2 insertions(+) diff --git a/package/nfs-utils/nfs-utils.mk b/package/nfs-utils/nfs-utils.mk index 30f12fd..6dd175c 100644 --- a/package/nfs-utils/nfs-utils.mk +++ b/package/nfs-utils/nfs-utils.mk @@ -38,6 +38,8 @@ endif define NFS_UTILS_INSTALL_FIXUP rm -f $(NFS_UTILS_TARGETS_) touch $(TARGET_DIR)/etc/exports + $(INSTALL) -D -m 644 \ + $(@D)/utils/mount/nfsmount.conf $(TARGET_DIR)/etc/nfsmount.conf endef NFS_UTILS_POST_INSTALL_TARGET_HOOKS += NFS_UTILS_INSTALL_FIXUP -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf 2015-12-22 17:12 [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf Jonathan Ben-Avraham @ 2015-12-22 17:28 ` Yann E. MORIN 2015-12-22 17:48 ` Jonathan Ben Avraham 2015-12-22 19:53 ` Thomas Petazzoni 1 sibling, 1 reply; 6+ messages in thread From: Yann E. MORIN @ 2015-12-22 17:28 UTC (permalink / raw) To: buildroot Jonathan, All, On 2015-12-22 19:12 +0200, Jonathan Ben-Avraham spake thusly: > From: Jonathan Ben Avraham <yba@tkos.co.il> > > Installs /etc/nfsmount.conf on the target, mainly to be able to configure the > the NFS version that mount.nfs uses by default. Without this ability you are > likely to encounter mount errors when automounting directories from NFS > version 3 servers. > > Signed-off-by: Jonathan Ben Avraham <yba@tkos.co.il> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> However, see below... > --- > package/nfs-utils/nfs-utils.mk | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/package/nfs-utils/nfs-utils.mk b/package/nfs-utils/nfs-utils.mk > index 30f12fd..6dd175c 100644 > --- a/package/nfs-utils/nfs-utils.mk > +++ b/package/nfs-utils/nfs-utils.mk > @@ -38,6 +38,8 @@ endif > define NFS_UTILS_INSTALL_FIXUP > rm -f $(NFS_UTILS_TARGETS_) > touch $(TARGET_DIR)/etc/exports > + $(INSTALL) -D -m 644 \ > + $(@D)/utils/mount/nfsmount.conf $(TARGET_DIR)/etc/nfsmount.conf This file is only comments, save for a single un-commented line; [ NFSMount_Global_Options ] Would it not be better that we just generate a file with just that line, and a comment to see the sources for the format, like so: # See utils/mount/nfsmount.conf in nfs-utils sources # for the format of this file. [ NFSMount_Global_Options ] Either way, I'm OK. Regards, Yann E. MORIN. > endef > NFS_UTILS_POST_INSTALL_TARGET_HOOKS += NFS_UTILS_INSTALL_FIXUP > > -- > 1.7.9.5 > > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf 2015-12-22 17:28 ` Yann E. MORIN @ 2015-12-22 17:48 ` Jonathan Ben Avraham 2015-12-22 18:51 ` Yann E. MORIN 0 siblings, 1 reply; 6+ messages in thread From: Jonathan Ben Avraham @ 2015-12-22 17:48 UTC (permalink / raw) To: buildroot On Tue, 22 Dec 2015, Yann E. MORIN wrote: > Date: Tue, 22 Dec 2015 18:28:42 +0100 > From: Yann E. MORIN <yann.morin.1998@free.fr> > To: Jonathan Ben-Avraham <yba@tkos.co.il> > Cc: buildroot at busybox.net > Subject: Re: [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf > > Jonathan, All, > > On 2015-12-22 19:12 +0200, Jonathan Ben-Avraham spake thusly: >> From: Jonathan Ben Avraham <yba@tkos.co.il> >> >> Installs /etc/nfsmount.conf on the target, mainly to be able to configure the >> the NFS version that mount.nfs uses by default. Without this ability you are >> likely to encounter mount errors when automounting directories from NFS >> version 3 servers. >> >> Signed-off-by: Jonathan Ben Avraham <yba@tkos.co.il> > > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> > > However, see below... > >> --- >> package/nfs-utils/nfs-utils.mk | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/package/nfs-utils/nfs-utils.mk b/package/nfs-utils/nfs-utils.mk >> index 30f12fd..6dd175c 100644 >> --- a/package/nfs-utils/nfs-utils.mk >> +++ b/package/nfs-utils/nfs-utils.mk >> @@ -38,6 +38,8 @@ endif >> define NFS_UTILS_INSTALL_FIXUP >> rm -f $(NFS_UTILS_TARGETS_) >> touch $(TARGET_DIR)/etc/exports >> + $(INSTALL) -D -m 644 \ >> + $(@D)/utils/mount/nfsmount.conf $(TARGET_DIR)/etc/nfsmount.conf > > This file is only comments, save for a single un-commented line; > [ NFSMount_Global_Options ] > > Would it not be better that we just generate a file with just that line, > and a comment to see the sources for the format, like so: > > # See utils/mount/nfsmount.conf in nfs-utils sources > # for the format of this file. > [ NFSMount_Global_Options ] Hi Yann, When debugging an unexpected NFS mount problem on a target for which you do not have the build system, at a site that might not have Internet access, with a root filesystem that might not have the "strings" utility, it is much easier to have all of the mount options in front of you rather than to have to leave the premises and start searching for the correct nfs-utils sources in order to find the full nfsmount.conf. It's just a much more friendly way of working. The lack of any nfsmount.conf cost me (my client really) several hours of work while I installed "strings" and figured out that there is such a thing as nfsmount.conf. It would be unfortunate for others to have to repeat this. Most people assigned to debug these types of systems are even less experienced than I. - yba > Either way, I'm OK. > > Regards, > Yann E. MORIN. > >> endef >> NFS_UTILS_POST_INSTALL_TARGET_HOOKS += NFS_UTILS_INSTALL_FIXUP >> >> -- >> 1.7.9.5 >> >> _______________________________________________ >> buildroot mailing list >> buildroot at busybox.net >> http://lists.busybox.net/mailman/listinfo/buildroot > > -- 9590 8E58 D30D 1660 C349 673D B205 4FC4 B8F5 B7F9 ~. .~ Tk Open Systems =}-------- Jonathan Ben-Avraham ("yba") ----------ooO--U--Ooo------------{= mailto:yba at tkos.co.il tel:+972.52.486.3386 http://tkos.co.il skype:benavrhm ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf 2015-12-22 17:48 ` Jonathan Ben Avraham @ 2015-12-22 18:51 ` Yann E. MORIN 2015-12-23 6:53 ` Jonathan Ben Avraham 0 siblings, 1 reply; 6+ messages in thread From: Yann E. MORIN @ 2015-12-22 18:51 UTC (permalink / raw) To: buildroot Jonathan, All, On 2015-12-22 19:48 +0200, Jonathan Ben Avraham spake thusly: > On Tue, 22 Dec 2015, Yann E. MORIN wrote: > >On 2015-12-22 19:12 +0200, Jonathan Ben-Avraham spake thusly: > >>From: Jonathan Ben Avraham <yba@tkos.co.il> [--SNIP--] > >>+ $(INSTALL) -D -m 644 \ > >>+ $(@D)/utils/mount/nfsmount.conf $(TARGET_DIR)/etc/nfsmount.conf > > > >This file is only comments, save for a single un-commented line; > > [ NFSMount_Global_Options ] > > > >Would it not be better that we just generate a file with just that line, > >and a comment to see the sources for the format, like so: > > > > # See utils/mount/nfsmount.conf in nfs-utils sources > > # for the format of this file. > > [ NFSMount_Global_Options ] > > When debugging an unexpected NFS mount problem on a target for which you do > not have the build system, at a site that might not have Internet access, > with a root filesystem that might not have the "strings" utility, it is much > easier to have all of the mount options in front of you rather than to have > to leave the premises and start searching for the correct nfs-utils sources > in order to find the full nfsmount.conf. It's just a much more friendly way > of working. The lack of any nfsmount.conf cost me (my client really) several > hours of work while I installed "strings" and figured out that there is such > a thing as nfsmount.conf. It would be unfortunate for others to have to > repeat this. Most people assigned to debug these types of systems are even > less experienced than I. Well, one would argue that the system should have been tested before being deployed. That's DevOps in action! Hehe! ;-) (Kidding!) But as I said: I'm OK with the full file as well. If a user is so concerned with the size, he can provide a post-build script to tweak that file, or a rootfs-overlay to provide his own version. So, my Reviewed-by stands. ;-) Thanks! Regards, Yann E. MORIN. -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf 2015-12-22 18:51 ` Yann E. MORIN @ 2015-12-23 6:53 ` Jonathan Ben Avraham 0 siblings, 0 replies; 6+ messages in thread From: Jonathan Ben Avraham @ 2015-12-23 6:53 UTC (permalink / raw) To: buildroot On Tue, 22 Dec 2015, Yann E. MORIN wrote: > Date: Tue, 22 Dec 2015 19:51:31 +0100 > From: Yann E. MORIN <yann.morin.1998@free.fr> > To: Jonathan Ben Avraham <yba@tkos.co.il> > Cc: buildroot at busybox.net > Subject: Re: [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf > > Jonathan, All, > > On 2015-12-22 19:48 +0200, Jonathan Ben Avraham spake thusly: >> On Tue, 22 Dec 2015, Yann E. MORIN wrote: >>> On 2015-12-22 19:12 +0200, Jonathan Ben-Avraham spake thusly: >>>> From: Jonathan Ben Avraham <yba@tkos.co.il> > [--SNIP--] >>>> + $(INSTALL) -D -m 644 \ >>>> + $(@D)/utils/mount/nfsmount.conf $(TARGET_DIR)/etc/nfsmount.conf >>> >>> This file is only comments, save for a single un-commented line; >>> [ NFSMount_Global_Options ] >>> >>> Would it not be better that we just generate a file with just that line, >>> and a comment to see the sources for the format, like so: >>> >>> # See utils/mount/nfsmount.conf in nfs-utils sources >>> # for the format of this file. >>> [ NFSMount_Global_Options ] >> >> When debugging an unexpected NFS mount problem on a target for which you do >> not have the build system, at a site that might not have Internet access, >> with a root filesystem that might not have the "strings" utility, it is much >> easier to have all of the mount options in front of you rather than to have >> to leave the premises and start searching for the correct nfs-utils sources >> in order to find the full nfsmount.conf. It's just a much more friendly way >> of working. The lack of any nfsmount.conf cost me (my client really) several >> hours of work while I installed "strings" and figured out that there is such >> a thing as nfsmount.conf. It would be unfortunate for others to have to >> repeat this. Most people assigned to debug these types of systems are even >> less experienced than I. > > Well, one would argue that the system should have been tested before > being deployed. That's DevOps in action! Hehe! ;-) (Kidding!) Hi Yann, Yes, they usually are tested before deployment, but often the end-user environment changes in ways that the testers did not expect and the default values of unusual parameters need to be changed. I think that NFS mount is particularly problematic in this regard because of the many different ways that sys-admins set it up and the wide range of servers used. > But as I said: I'm OK with the full file as well. If a user is so > concerned with the size, he can provide a post-build script to tweak > that file, or a rootfs-overlay to provide his own version. > > So, my Reviewed-by stands. ;-) tnx, - yba > > Thanks! > > Regards, > Yann E. MORIN. > > -- 9590 8E58 D30D 1660 C349 673D B205 4FC4 B8F5 B7F9 ~. .~ Tk Open Systems =}-------- Jonathan Ben-Avraham ("yba") ----------ooO--U--Ooo------------{= mailto:yba at tkos.co.il tel:+972.52.486.3386 http://tkos.co.il skype:benavrhm ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf 2015-12-22 17:12 [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf Jonathan Ben-Avraham 2015-12-22 17:28 ` Yann E. MORIN @ 2015-12-22 19:53 ` Thomas Petazzoni 1 sibling, 0 replies; 6+ messages in thread From: Thomas Petazzoni @ 2015-12-22 19:53 UTC (permalink / raw) To: buildroot Dear Jonathan Ben-Avraham, On Tue, 22 Dec 2015 19:12:17 +0200, Jonathan Ben-Avraham wrote: > From: Jonathan Ben Avraham <yba@tkos.co.il> > > Installs /etc/nfsmount.conf on the target, mainly to be able to configure the > the NFS version that mount.nfs uses by default. Without this ability you are > likely to encounter mount errors when automounting directories from NFS > version 3 servers. > > Signed-off-by: Jonathan Ben Avraham <yba@tkos.co.il> > --- > package/nfs-utils/nfs-utils.mk | 2 ++ > 1 file changed, 2 insertions(+) Applied, thanks. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-12-23 6:53 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-12-22 17:12 [Buildroot] [PATCH 1/1 v1] nfs-utils: install /etc/nfsmount.conf Jonathan Ben-Avraham 2015-12-22 17:28 ` Yann E. MORIN 2015-12-22 17:48 ` Jonathan Ben Avraham 2015-12-22 18:51 ` Yann E. MORIN 2015-12-23 6:53 ` Jonathan Ben Avraham 2015-12-22 19:53 ` Thomas Petazzoni
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.