autopkg / recipes

Recipes for AutoPkg
231 stars 206 forks source link

Spotify architecture support #443

Closed kevinmcox closed 1 year ago

kevinmcox commented 1 year ago

Adds architecture variables since Spotify does not provide a Universal application.

kevinmcox commented 1 year ago

Output of a run:

Spotify.txt

homebysix commented 1 year ago

For future changes like this, I would recommend setting the default value to the Intel flavor (in this case an empty string) instead of to the Apple Silicon flavor. This way administrators who don't update their overrides immediately will still be distributing the same architecture as before, while the new architecture will be available to those who choose to use it.

kevinmcox commented 1 year ago

@homebysix that makes sense. I guess I was thinking at some point we need to start assuming ARM as the default, so why not now?