Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

consider nfs4 a remote filesystem #7

Merged
merged 1 commit into from
May 15, 2016
Merged

consider nfs4 a remote filesystem #7

merged 1 commit into from
May 15, 2016

Conversation

klaernie
Copy link
Contributor

No description provided.

@rolinh rolinh merged commit 62d4dd2 into rolinh:master May 15, 2016
@rolinh
Copy link
Owner

rolinh commented May 15, 2016

Hi,

Thanks for your contribution!

@klaernie klaernie deleted the nfs4 branch May 15, 2016 15:48
@klaernie
Copy link
Contributor Author

no big deal - was trying to figure out a possible solution for #8 but noticed this while at it. Comparing this to systemd/systemd#3140 I have to agree with @poettering : there has to be a better way than hardcoding this and maintaining it everything a new FS pops up.

@rolinh
Copy link
Owner

rolinh commented May 16, 2016

Yeah, I wish Linux had something like the BSDs which makes it for a much simpler and reliable implementation:

int
is_remote(const struct fsmntinfo *fs)
{
        if (fs->flags & MNT_LOCAL)
                return 0;

        return 1;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants