On Mon, Jun 27, 2011 at 01:11:18PM -0400, Kenneth R Westerback wrote:
> On Mon, Jun 27, 2011 at 12:06:29PM -0400, Ted Unangst wrote:
> > On Mon, 27 Jun 2011, Kenneth R Westerback wrote:
> > 
> > > On Mon, Jun 27, 2011 at 12:08:15AM -0400, Ted Unangst wrote:
> > > > Remove some words that are not typically found in OpenBSD mount man 
> > > > pages.  
> > > > Write support is simply not going to happen, so we can summarize that 
> > > > situation much more succinctly.  (and enforce readonly mounts in the 
> > > > program.)
> > > 
> > > Seems good to me, except for the mysterious admonition to start the
> > > NTFS partition at offset 63. I don't know what on the disklabel(8)
> > > page would provide support for, or info about this recommendation.
> > > And in the world of disks 64 is much more popular now, even in the
> > > Windows world.
> > > 
> > > I suggest removing that text too.
> > 
> > Deleted that whole section.  User errors are not bugs.
> > 
> > > On re-reading it I don't see any mention that -o ro is automatic and
> > > unchangeable. i.e. filesstems can only be mounted read only. I think
> > > that needs to be in there. Possibly in the first paragraph, and mentioned
> > > in the options sections (i.e. -o rw won't work, if that is true).
> > 
> > Added a mention.  New diff below.
> 
> Works for me. I'm sure jmc@ will have tweeks when he surfaces. :-)
> 
> .... Ken
> 

actually only one wee one, and otherwise ok:

> > 
> > Index: mount_ntfs.8
> > ===================================================================
> > RCS file: /home/tedu/cvs/src/sbin/mount_ntfs/mount_ntfs.8,v
> > retrieving revision 1.11
> > diff -u -r1.11 mount_ntfs.8
> > --- mount_ntfs.8    13 Oct 2010 15:40:31 -0000      1.11
> > +++ mount_ntfs.8    27 Jun 2011 16:05:26 -0000
> > @@ -55,12 +55,14 @@
> >  to the global filesystem namespace at the location
> >  indicated by
> >  .Pa node .
> > -This command is normally executed by
> > +This command is invoked by
> >  .Xr mount 8
> > -at boot time, but can be used by any user to mount an
> > -NTFS file system on any directory that they own (provided,
> > -of course, that they have appropriate access to the device that
> > -contains the file system).
> > +when using the syntax
> > +.Bd -ragged -offset 4h
> > +.Nm mount Op options
> > +-t ntfs
> > +.Ar special Ar node
> > +.Ed
> >  .Pp
> >  The
> >  .Ar special
> > @@ -80,6 +82,7 @@
> >  .Pp
> >  The supported NTFS versions include both NTFS4, as used by Microsoft
> >  Windows NT 4.0, and NTFS5, as used by Microsoft Windows 2000 and XP.
> > +Only read-only operation is permitted, this will be automatically enforced.

i'd prefer s/this will be/which is/
otherwise you need a semi-colon

jmc

> >  .Pp
> >  The options are as follows:
> >  .Bl -tag -width Ds
> > @@ -134,37 +137,12 @@
> >  .Bd -literal -offset indent
> >  # cat /mnt/foodir:\e$INDEX_ROOT:\e$I30
> >  .Ed
> > -.Ss Limited support for writing
> > -There is limited writing ability for files.
> > -Limitations:
> > -.Pp
> > -.Bl -bullet -compact
> > -.It
> > -file must be non-resident
> > -.It
> > -file must
> > -.Em not
> > -contain any holes (uninitialized areas)
> > -.It
> > -file can't be compressed
> > -.El
> > -.Pp
> > -Note that it's not currently possible to create or remove files
> > -on NTFS filesystems.
> > -.Pp
> > -.Sy Warning :
> > -do not mount NTFS filesystems read-write.
> > -The write support is not very useful and is not tested well.
> > -It's not safe to write to any file on NTFS; you might damage the 
> > filesystem.
> > -Unless you want to debug NTFS filesystem code, mount the NTFS filesystem
> > -read-only.
> >  .Sh SEE ALSO
> >  .Xr mount 2 ,
> >  .Xr unmount 2 ,
> >  .Xr disklabel 5 ,
> >  .Xr fstab 5 ,
> >  .Xr disklabel 8 ,
> > -.\" .Xr mbrlabel 8 ,
> >  .Xr mount 8
> >  .Sh HISTORY
> >  Support for NTFS first appeared in
> > @@ -185,23 +163,7 @@
> >  port was done by
> >  .An Julien Bordet Aq [email protected] .
> >  .Sh BUGS
> > -The write support should be enhanced to actually be able to change
> > -file size, and to create and remove files and directories.
> > -It's not very useful right now.
> > -.Pp
> > -If the attempt to mount NTFS gives you an error like this:
> > -.Bd -literal -offset indent
> > -# mount -t ntfs /dev/wd0k /mnt
> > -mount_ntfs: /dev/wd0k on /mnt: Invalid argument
> > -.Ed
> > -.Pp
> > -make sure that the appropriate partition has the correct entry in the
> > -disk label, particularly that the partition offset is correct.
> > -If the NTFS partition is the first partition on the disk, the
> > -offset should be '63' (see
> > -.Xr disklabel 8 ) .
> > -.\" .Xr mbrlabel 8
> > -.\" could help you to set up the disk label correctly.
> > +Only read support is enabled.
> >  .Pp
> >  If the NTFS partition is marked as
> >  .Ql dynamic
> > Index: mount_ntfs.c
> > ===================================================================
> > RCS file: /home/tedu/cvs/src/sbin/mount_ntfs/mount_ntfs.c,v
> > retrieving revision 1.12
> > diff -u -r1.12 mount_ntfs.c
> > --- mount_ntfs.c    14 Apr 2007 17:07:28 -0000      1.12
> > +++ mount_ntfs.c    27 Jun 2011 04:05:37 -0000
> > @@ -109,6 +109,8 @@
> >             }
> >     }
> >  
> > +   mntflags |= MNT_RDONLY;
> > +
> >     if (optind + 2 != argc)
> >             usage();

Reply via email to