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

Use attribute keys suggested by official documentation for macOS #25

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

CthulhuDen
Copy link
Contributor

Fixes #24.

  1. Remove user namespace in macOS. I'm not sure though if it was needed in previous versions of macOS or with other filesystems, I'm using macOS Sonoma (current) with APFS filesystem (default) and have no access to any other combination. Maybe we should also set the namespaced attributes for the compatibility with some previous configurations, or maybe this was just never tested to work in macOS.
  2. As well as com.dropbox.ignored, also set com.apple.fileprovider.ignore#P attribute, which seems to be used by the newer Dropbox versions (mentioned in dropbox help page for setting a file or folder to be ignored).

@mweirauch
Copy link
Owner

It looks like the 'user' namespace is not really needed on Linux either. I don't remember where I got this from. It might have been recommended to distinguish from system attributes.

Here is a(nother) list of supported attributes. Both seem fine.

The new macOS attribute apparently hasn't been around when I implemented this in 2020.
Here the help page from 2022-10.

Thanks again for the PR! Very much appreciated. Crossing fingers the release GH actions still work.

@mweirauch mweirauch merged commit b82313e into mweirauch:master Dec 13, 2023
6 checks passed
@mweirauch mweirauch added this to the 0.1.2 milestone Dec 13, 2023
@mweirauch mweirauch added the bug Something isn't working label Dec 13, 2023
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

Successfully merging this pull request may close these issues.

Not working in MacOS Sonoma
2 participants