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

Support macOS 14 #55

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Support macOS 14 #55

merged 1 commit into from
Jun 11, 2024

Conversation

jugglinmike
Copy link
Contributor

@jugglinmike jugglinmike commented Jun 4, 2024

macOS version 14 removed the /usr/local/var/ directory. /run/ and /var/run/ are defined by the Filesystem Hierarchy Standard for the purpose of UNIX-domain sockets 1. However, macOS 14 does not include /run/, and /var/run/ is not world-writable on that platform. In order to support macOS 14, use the /tmp/ directory, instead. Extend the configuration of the project's continuous integration environment to include macOS 14.

Footnotes

  1. https://refspecs.linuxfoundation.org/FHS_3.0/index.html

@jugglinmike jugglinmike requested a review from stalgiag June 4, 2024 22:26
@jugglinmike
Copy link
Contributor Author

Looks like this needs some work--the tests fail in CI but not my development machine. I'm on the case.

macOS version 14 removed the `/usr/local/var/` directory. `/run/` and
`/var/run/` are defined by the Filesystem Hierarchy Standard for the
purpose of UNIX-domain sockets [1]. However, macOS 14 does not include
`/run/`, and `/var/run/` is not world-writable on that platform. In
order to support macOS 14, use the `/tmp/` directory, instead. Extend
the configuration of the project's continuous integration environment to
include macOS 14.

[1] https://refspecs.linuxfoundation.org/FHS_3.0/index.html
@jugglinmike
Copy link
Contributor Author

This is ready for review, now!

Copy link
Contributor

@stalgiag stalgiag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't realize that macOS 14 gets rid usr/local/var. That seems like an odd and consequential change. Switching to tmp makes sense for this task. Tested.Thanks for handling it!

node-version: ['18.x']
test-command:
- npm run test-style
- npm run test-types
- npm run test-unit
exclude:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jugglinmike jugglinmike merged commit aaa05a2 into w3c:main Jun 11, 2024
4 checks passed
@jugglinmike jugglinmike deleted the macos-14 branch June 11, 2024 01:42
jugglinmike added a commit to bocoup/at-driver-servers that referenced this pull request Jun 19, 2024
macOS version 14 removed the `/usr/local/var/` directory. `/run/` and
`/var/run/` are defined by the Filesystem Hierarchy Standard for the
purpose of UNIX-domain sockets [1]. However, macOS 14 does not include
`/run/`, and `/var/run/` is not world-writable on that platform. In
order to support macOS 14, use the `/tmp/` directory, instead. Extend
the configuration of the project's continuous integration environment to
include macOS 14.

This commit has been cherry-picked from the "aria-at-automation-driver"
project, where it was previously reviewed and approved [2].

[1] https://refspecs.linuxfoundation.org/FHS_3.0/index.html
[2] w3c/aria-at-automation-driver#55
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