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

(bug) mount: nfs mounts with () in name: wrong parsing of options #640

Open
muescha opened this issue Feb 9, 2025 · 1 comment
Open
Labels
bug Something isn't working

Comments

@muescha
Copy link
Contributor

muescha commented Feb 9, 2025

NFS Name with ()

I use MountainDuck, which by default mounts with the FTP domain in the NFS name like this:

$ mount -t nfs
MN - mydomain.com (ftp.mydomain.com):/ on /Users/muescha/Library/Group Containers/GS1111XU.duck/Library/Application Support/duck/Volumes.noindex/MN - mydomain.com.localized (nfs, asynchronous, nodev, nosuid, noowners, noatime, mounted by muescha)

The first ( is interpreted by the options parser instead of the last one before nfs:

jc --pretty mount
[
  {
    "filesystem": "/dev/disk3s1s1",
    "mount_point": "/",
    "options": [
      "apfs",
      "sealed",
      "local",
      "read-only",
      "journaled"
    ]
  },
  { ... },
  {
    "filesystem": "MN - mydomain.com (ftp.mydomain.com):/",
    "mount_point": "/Users/muescha/Library/Group Containers/GS1111XU.duck/Library/Application Support/duck/Volumes.noindex/MN - mydomain.com.localized",
    "options": [
      "ftp.mydomain.com):/ on /Users/muescha/Library/Group Containers/GS1111XU.duck/Library/Application Support/duck/Volumes.noindex/MN - mydomain.com.localized (nfs",
      "asynchronous",
      "nodev",
      "nosuid",
      "noowners",
      "noatime",
      "mounted by muescha"
    ]
  }
]

The -t nfs Parameter Doesn’t Work

I found that I cannot use the -t nfs parameter, while the -t apfs parameter works as expected.

$ jc --pretty mount -t nfs

jc:  Error - mount parser could not parse the input data.
             If this is the correct parser, try setting the locale to C (LC_ALL=C).
             For details use the -d or -dd option. Use "jc -h --mount" for help.

and

$ mount -t nfs | jc --mount

jc:  Error - mount parser could not parse the input data.
             If this is the correct parser, try setting the locale to C (LC_ALL=C).
             For details use the -d or -dd option. Use "jc -h --mount" for help.
@kellyjonbrazil
Copy link
Owner

Thanks for reporting this - we should be able to get a fix in the next release.

@kellyjonbrazil kellyjonbrazil added the bug Something isn't working label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants