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

fix autogen.sh for WSL/Windows git hook file link attempt #352

Merged
merged 1 commit into from
May 13, 2024

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented May 10, 2024

Similar to wolfSSL/wolfssl#6798 this PR addresses #346 for wolfTPM.

The autogen.sh is updated to check for WSL/Windows, and if detected, does not attempt to link git hook files. When running in WSL for a Windows drive, (e.g. /mnt/c/) new warning messages will appear regarding not attempting to link the pre-commit files which otherwise don't actually get linked. Rather, a zero length file is created, causing an Visual Studio.

gojimmypi:/mnt/c/workspace/wolfTPM-gojimmypi-pr
$ ./autogen.sh
Linux ln does not work on shared Windows file system in WSL.
The pre-commit.sh file will not be copied to .git/hooks/pre-commit
The pre-push.sh file will not be copied to .git/hooks/pre-commit
Making missing build-aux directory.
Touching missing build-aux/config.rpath file.
autoreconf: export WARNINGS=
autoreconf: Entering directory '.'
autoreconf: configure.ac: not using Gettext
[... snip / etc ...]

This PR addresses #346

@dgarske dgarske merged commit 2a84508 into wolfSSL:master May 13, 2024
3 checks passed
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.

3 participants