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

new alias: GitSetLibGit2NativeLibraryPathToIncluded & GitEnableLogging #188

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

nils-a
Copy link
Contributor

@nils-a nils-a commented Feb 15, 2024

  • GitEnableLogging enables the git logging in LibGit2Sharp and routes the log messages into Cake. NOt sure, if we need this, but it felt like a nice addition while I was looking through the configuration-options of LibGit2Sharp.

  • GitSetLibGit2NativeLibraryPathToIncluded calculates the path to the included native libGit2 libraries and configures that path as NativeLibraryPath of LibGit2Sharp.

    If this works for all systems, we could also drop the use of the extra call and configure NativeLibraryPath automatically. (This would actually be my preferred solution, I think. Calling GitSetLibGit2NativeLibraryPathToIncluded() as a first statement on every build that uses Cake.Git feels a bit cumbersome.)

@@ -14,7 +14,7 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-20.04, ubuntu-latest, macos-latest, windows-2019, windows-2022]
os: [ubuntu-20.04, ubuntu-latest, macos-12, macos-13, macOS-13-arm64, windows-2019, windows-2022]
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Smaller agents which are free don't seem to be available yet: actions/runner-images#8439 (comment)

Copy link
Member

Choose a reason for hiding this comment

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

@nils-a One possible solution could be to use FlyCI which provides 500 min / month for free for open source projects.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pascalberger I reverted the changes I did to the CI machines. I have not tried FlyCI, though.

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.

None yet

2 participants