Marcus,
Sorry for the delay in response (I'll claim it is due to vacations and
all). Thanks for suggesting improvements to FUSEFileSystem. I have
some bug fixes and other small changes that I'm going to try and get
reviewed by alcor, so I'll try to help out with this.
Given the number of potential arguments that macfuse can take, I'd
rather add a method like:
- (NSArray *)fuseArguments; // Additional arguments to pass to fuse.
and have FUSEFileSystem call [self fuseArguments] and add those to the
set of arguments it passes to fuse. This way we can support a larger
set of fuse options and not worry about updating the code if Amit adds
any new options in the future. Given the rate at which options are
added to MacFUSE and the time between commits to FUSEFileSystem, I
suspect this would be a safer plan.
Since the "-s" argument is currently present by default, I will be
making that conditional on an isThreadSafe method returning YES, but
in general I'd like to avoid adding more like this.
Will that work for you?
ted
On Nov 21, 2007 10:28 AM, znek <[EMAIL PROTECTED]> wrote:
>
> Hi FUSEObjC users,
>
> in my project I'd like to see FUSEFileSystem.m implementing methods
> for mount time options, similar to the -shouldMountInFinder option,
> i.e.:
>
> --- snip ---
> Index: FUSEFileSystem.m
> ===================================================================
> --- FUSEFileSystem.m (revision 648)
> +++ FUSEFileSystem.m (working copy)
> @@ -789,7 +789,18 @@
>
>
>
> +#pragma mark Options
>
> +
> +- (BOOL)isLocal {
> + return NO;
> +}
> +
> +- (BOOL)shouldAllowOther {
> + return NO;
> +}
> +
> +
> #pragma mark -
>
> +- (BOOL)shouldAllowOther {
> + return NO;
> +}
> +
> +
> #pragma mark -
>
>
> @@ -1161,6 +1172,13 @@
> if ([self shouldMountInFinder]) {
> [arguments addObject:@"-oping_diskarb"]; // Ping diskarb to look
> for our ne
> w mount point.
> }
> + if ([self isLocal]) {
> + [arguments addObject:@"-olocal"];
> + }
> + if ([self shouldAllowOther]) {
> + [arguments addObject:@"-allow_other"];
> + }
> +
> [arguments addObject:mountPath];
>
> // Start Fuse Main
> --- snap ---
>
> I don't think it would hurt anybody, naming is subject to
> discussion...
>
> Cheers,
>
> Marcus
>
>
> >
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"macfuse-devel" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/macfuse-devel?hl=en
-~----------~----~----~----~------~----~------~--~---
