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

Fixes for the VC Bundle with Git on Windows #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NathanielJS1541
Copy link

@NathanielJS1541 NathanielJS1541 commented Sep 7, 2024

There were a couple of issues that meant that functionality of the vc bundle when using git on Windows just didn't work at all.

I have opened #2 and #3 to report these issues, and since the issues are fairly simple to fix I have opened this PR with my suggestions.

However, in light of this it may be worth checking the consistency of the other platforms and version control types. I would do this, but I only work with git.

EDIT:
I've browsed the source, and it seems that only the mercurial template uses Abr. Hash: rather than AbrHash: etc. I still think moving git-windows/vc.bat inline with git-unix/vc is the right call so there are no differences between the vc-git.awk files.

EDIT 2:
I've also checked the rest of the .bat files, and none of the rest of them need escaping either.

These extra escape characters are needed in the git pretty format so
they are not replaced by variables in the batch script.

Signed-off-by: Nathaniel Struselis <Nathaniel.Struselis@hotmail.co.uk>
Signed-off-by: Nathaniel Struselis <Nathaniel.Struselis@hotmail.co.uk>
@NathanielJS1541
Copy link
Author

Sorry, just rebased my commits to include the signed-off-by line.

NathanielJS1541 added a commit to NathanielJS1541/LaTeX_Template that referenced this pull request Sep 9, 2024
These files are taken from a pull request I recently opened
(rohieb/ctan-support-vc#4) which fixes a couple
of issues with the vc bundles specific to git on windows.
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