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

Wrapper script to use ktx compare for git diff. #919

Merged
merged 4 commits into from
Jun 14, 2024
Merged

Conversation

MarkCallow
Copy link
Collaborator

@MarkCallow MarkCallow commented Jun 7, 2024

Those wishing to use this must run, or have run, install-gitconfig.sh so that .gitconfig, which now includes a "ktx-compare" diff command will be included by their repo clone's .git/config.

They need to have the ktx command installed and in $PATH.

They need to add the line

*.ktx2 binary diff=ktx-compare

to their clone's .git/info/attributes. This is not included in the repo's .gitattributes because not everyone will have the ktx command installed. This will not work in a user-global or system-global Git attributes file because those are lower priority than .gitattributes so are read first and .gitattributes already has an entry for *.ktx2, indicating binary, which overrides anything from the global files.

Will be happy to accept a PR to add a .ps1 equivalent script.

Those wishing to use this must run, or have run, install-gitconfig.{ps1,sh}
so that .gitconfig, which now includes a "ktx-compare" diff command will
be included by your repo clone's `.git/config`.

They need to have the `ktx` command installed and in the $PATH.

They need to add the line

  *.ktx2 binary diff=ktx-compare

to their clone's .git/info/attributes or to a global Git attributes file.
This is not included in the repo's .gitattributes because not everyone
will have the ktx command installed.
Explicity call bash in "ktx-compare" so it will work on Windows,
provided bash is installed.
@MarkCallow MarkCallow merged commit 8cd4fea into main Jun 14, 2024
18 checks passed
@MarkCallow MarkCallow deleted the diffscript branch June 14, 2024 00:10
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.

1 participant