Merge pull request #384 from rosenbjerg/chore/fixes

Minor fixes

Former-commit-id: b8d1c26f4f
This commit is contained in:
Malte Rosenbjerg 2023-01-31 23:16:23 +01:00 committed by GitHub
commit 5ebe0bb71d
4 changed files with 6 additions and 9 deletions

View file

@ -15,8 +15,8 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [windows-latest, ubuntu-latest]
timeout-minutes: 6
os: [windows-latest, ubuntu-latest, macos-latest]
timeout-minutes: 7
steps:
- name: Checkout
@ -28,10 +28,9 @@ jobs:
dotnet-version: '7.0.x'
- name: Prepare FFMpeg
uses: Iamshankhadeep/setup-ffmpeg@v1.2
uses: FedericoCarboni/setup-ffmpeg@v2
with:
token: ${{ secrets.GITHUB_TOKEN }}
version: "4.4"
- name: Test with dotnet
run: dotnet test FFMpegCore.sln --logger GitHubActions

View file

@ -12,9 +12,7 @@ public static string GetPipePath(string pipeName)
{
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
return $@"\\.\pipe\{pipeName}";
if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
return $"unix:{Path.GetTempPath()}/CoreFxPipe_{pipeName}";
return $"unix:/tmp/CoreFxPipe_{pipeName}";
return $"unix:{Path.Combine(Path.GetTempPath(), $"CoreFxPipe_{pipeName}")}";
}
}
}

View file

@ -1,6 +1,6 @@
MIT License
Copyright (c) 2022 Vlad Jerca
Copyright (c) 2023 Vlad Jerca
Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal

View file

@ -231,6 +231,6 @@ Older versions of ffmpeg might not support all ffmpeg arguments available throug
### License
Copyright © 2021
Copyright © 2023
Released under [MIT license](https://github.com/rosenbjerg/FFMpegCore/blob/master/LICENSE)