Skip to content

Add MacOS build testing#14707

Draft
andrewc12 wants to merge 3 commits intoopenzfs:masterfrom
andrewc12:andrew-appveyormac-01
Draft

Add MacOS build testing#14707
andrewc12 wants to merge 3 commits intoopenzfs:masterfrom
andrewc12:andrew-appveyormac-01

Conversation

@andrewc12
Copy link
Contributor

@andrewc12 andrewc12 commented Apr 2, 2023

The current appveyor config for building and testing on macos.
Signed-off-by: Andrew Innes andrew.c12@gmail.com

Motivation and Context

We're unable to run zfs tests on the macos github images, but we still need to be able to automatically run tests.

Description

By adding the config files required to build and test zfs on macos on appveyor, any forked repo with appveyor integration enabled will automatically run the zfs tests on macos.

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

The current appveyor config for building and testing on macos.
Signed-off-by: Andrew Innes <andrew.c12@gmail.com>
@andrewc12
Copy link
Contributor Author

⚠️ This PR needs PR #12110 to be merged first ⚠️

@andrewc12
Copy link
Contributor Author

andrewc12 commented Apr 2, 2023

It's possible to move appveyor.yaml to a cleaner location but that requires setting up appveyor in a specific way.
"YAML file alternative naming"
https://www.appveyor.com/docs/build-configuration/#:~:text=Pull%20request%E2%80%9D%20events-,YAML%20file%20alternative%20naming,-AppVeyor%20supports%20dot

@andrewc12 andrewc12 mentioned this pull request Apr 2, 2023
13 tasks
Signed-off-by: Andrew Innes <andrew.c12@gmail.com>
Signed-off-by: Andrew Innes <andrew.c12@gmail.com>
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