Open
Conversation
fix: update windows10 vm
Installing a system dependency is out of scope for this script, therefore krb5 will be vendored
038c6e4 to
b5d24eb
Compare
PACordonnier
approved these changes
Jan 25, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is PoC showing what a rust migration could look like.
There should be no change to the interface.
What's new in this PR:
Requirements change
The dependencies change, node-gyp is no longer required for the build, such as python (but python is still needed for
src/server.pyHow to build
In any case, the build command is:
npm install, but os and installation method may vary:export KRB5_HOME=/path/to/your/prefixorexport PKG_CONFIG_PATH=/path/to/your/krb5/pkg-configexport KRB5_HOME=/path/to/your/homebrew/cellar/krb5(this is a custom prefix) orexport PKG_CONFIG_PATH=/path/to/your/homebrew/cellar/krb5/lib/pkg-config$env:KRB5_HOME=X:\path\to\you\mit\kerberos\installationNew
install_krb5.shnow installs MIT Kerberos into a foldervendorat the root of the project. If you want to build the project with this version, simply set the variableexport KRB5_HOME=${project_path}/vendor.npm run local_test, it will instantiate a KDC, a Kadmind and a server.py on your host, without docker. You must have the krb5 sbin binaries in your path. If you don't overwrite your PATH variables and you don't have system-wide binaries, you can set the variableexport KRB5_HOME=/path/to/your/krb5/prefixubuntu-vendored, it's the same as ubuntu but instead of getting krb5 from apt, it's using theinstall_krb5.shscriptBreaking Change
This PR has been tested on Ubuntu 20.04, MacOs 11 Big Sur, Windows 10 20H2, and with the containers.
All tests have been performed on a x86_64 processor, would be interested with feedback on ARM (macos).