feat: added support for multiple Linux distros (#4)

Author: soumyamahunt

README.md

@@ -1,15 +1,15 @@
 # Setup Swift
 
 [![GitHub Action](https://img.shields.io/github/v/tag/SwiftyLab/setup-swift?logo=github&label=GitHub)](https://badge.fury.io/gh/SwiftyLab%2Fsetup-swift)
-[![Supports macOS, Ubuntu & Windows](https://img.shields.io/badge/platform-macOS%20%7C%20Ubuntu%20%7C%20Windows-blue?label=platform)](https://help.github.com/en/actions/automating-your-workflow-with-github-actions/virtual-environments-for-github-hosted-runners#supported-runners-and-hardware-resources)
+[![Supports macOS, Linux & Windows](https://img.shields.io/badge/platform-macOS%20%7C%20Linux%20%7C%20Windows-blue?label=platform)](https://help.github.com/en/actions/automating-your-workflow-with-github-actions/virtual-environments-for-github-hosted-runners#supported-runners-and-hardware-resources)
 [![Swift 5.9](https://img.shields.io/badge/Swift-5.9-orange?logo=swift&logoColor=white)](https://swift.org)
 [![CI/CD](https://github.com/SwiftyLab/setup-swift/actions/workflows/main.yml/badge.svg)](https://github.com/SwiftyLab/setup-swift/actions/workflows/main.yml)
 [![CodeFactor](https://www.codefactor.io/repository/github/swiftylab/setup-swift/badge)](https://www.codefactor.io/repository/github/swiftylab/setup-swift)
 [![codecov](https://codecov.io/gh/SwiftyLab/setup-swift/graph/badge.svg?token=XWfSpWQ6gC)](https://codecov.io/gh/SwiftyLab/setup-swift)
 
 ![swift.org](https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fraw.githubusercontent.com%2FSwiftyLab%2Fsetup-swift%2Fmain%2Fpackage.json&query=%24.swiftorg&logo=swift&logoColor=white&label=swift.org)
 
-[GitHub Action](https://github.com/features/actions) that will setup [Swift](https://swift.org) environment with specified version. Works on both Ubuntu and macOS runners with limited support on Windows runners.
+[GitHub Action](https://github.com/features/actions) that will setup [Swift](https://swift.org) environment with specified version. Works on both Linux and macOS runners with limited support on Windows runners.
 
 ## Usage
 

__tests__/platform.test.ts

@@ -52,6 +52,17 @@ describe('platform detection', () => {
     expect(platform.file).toBe('ubuntu1804-aarch64')
   })
 
+  it('detects centos', async () => {
+    setos({os: 'linux', dist: 'CentOS', release: '7'})
+    jest.spyOn(os, 'arch').mockReturnValue('x64')
+    const platform = await Platform.currentPlatform()
+    expect(platform.name).toBe('centos')
+    expect(platform).toBeInstanceOf(LinuxPlatform)
+    expect((platform as LinuxPlatform).version).toBe(7)
+    expect(platform.arch).toBe('x86_64')
+    expect(platform.file).toBe('centos7')
+  })
+
   it('detects windows', async () => {
     setos({os: 'win32', dist: 'Windows', release: '10.0.17063'})
     jest.spyOn(os, 'arch').mockReturnValue('x64')

__tests__/snapshot.test.ts

@@ -145,7 +145,7 @@ describe('fetch tool data based on options', () => {
     }
   })
 
-  // Ubuntu tests
+  // Linux tests
 
   it('fetches ubuntu 18.04 latest swift 4 tool', async () => {
     setos({os: 'linux', dist: 'Ubuntu', release: '18.04'})
@@ -297,6 +297,20 @@ describe('fetch tool data based on options', () => {
     expect(lTool.docker).toBe('5.2.5-focal')
   })
 
+  it('fetches centos 7 latest swift tool', async () => {
+    setos({os: 'linux', dist: 'CentOS', release: '7'})
+    jest.spyOn(os, 'arch').mockReturnValue('x64')
+    const tool = await Platform.toolchain(latest)
+    expect(tool).toBeTruthy()
+    const lTool = tool as LinuxToolchainSnapshot
+    expect(lTool.download).toBeTruthy()
+    expect(lTool.dir).toBeTruthy()
+    expect(lTool.platform).toBeTruthy()
+    expect(lTool.branch).toBeTruthy()
+    expect(lTool.download_signature).toBeTruthy()
+    expect(lTool.docker).toBeTruthy()
+  })
+
   // Windows tests
 
   it('fetches windows 10 latest swift 5 tool', async () => {

dist/index.js

@@ -1,10 +1,10 @@
 import * as path from 'path'
 import {promises as fs} from 'fs'
 import * as core from '@actions/core'
-import {exec} from '@actions/exec'
 import * as toolCache from '@actions/tool-cache'
 import {VerifyingToolchainInstaller} from './verify'
 import {LinuxToolchainSnapshot} from '../snapshot'
+import {PackageManager} from './package_manager'
 import {MODULE_DIR} from '../const'
 
 export class LinuxToolchainInstaller extends VerifyingToolchainInstaller<LinuxToolchainSnapshot> {
@@ -29,16 +29,15 @@ export class LinuxToolchainInstaller extends VerifyingToolchainInstaller<LinuxTo
         .trim()
         .replace(/^\\+|\\+$|^\$+|\$+$/g, '')
         .trim()
-      return command.length ? command : []
+      return command.length ? command.split(' ') : []
     })
     if (!commands.length) {
       core.debug(`Skipping dependencies install as no commands in "${content}"`)
       return
     }
     try {
-      // TODO: make update generalized accross linux platforms
-      await exec('sudo', ['apt-get', 'update'])
-      await exec('sudo', [...commands, '-y'])
+      const packageManager = new PackageManager(commands)
+      await packageManager.install()
     } catch (error) {
       core.debug(`Dependencies installation failed with "${error}"`)
     }

src/installer/linux.ts

@@ -0,0 +1,20 @@
+import {exec} from '@actions/exec'
+
+export class PackageManager {
+  readonly name: string
+  readonly installationCommands: string[]
+
+  constructor(installationCommands: string[]) {
+    this.name = installationCommands[0]
+    this.installationCommands = installationCommands
+  }
+
+  protected async update() {
+    await exec('sudo', [this.name, 'update'])
+  }
+
+  async install() {
+    await this.update()
+    await exec('sudo', [...this.installationCommands, '-y'])
+  }
+}