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

Add support for Linux enterprise deployable default settings #1814

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mjcheetham
Copy link
Collaborator

Mirroring #1811, add support to set default settings for Linux, via MDM tooling. For this platform we look for files in the /etc/git-credential-manager/config.d directory.

Files are a simple format of key=value\n, where the key is a configuration value name from docs/config.md. This format prevents values from containing a \n line feed character, but this is unlikely to be an issue in practice.

Copy link
Collaborator

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Looks good to me!

I am stumped by the compile errors on Windows, hopefully you can figure them out!

@mjcheetham
Copy link
Collaborator Author

I am stumped by the compile errors on Windows, hopefully you can figure them out!

It's because we target .NET Framework on Windows, and some of the overloads are not available, and I wrote these patches on Mac and Linux. Will check my syntax for .NET FX compat.

@mjcheetham mjcheetham force-pushed the linux-defaults branch 5 times, most recently from bb94df1 to d53f82a Compare January 28, 2025 08:59
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