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

Make CVS value expansion work in git. #14

Closed
wants to merge 5 commits into from

Conversation

mithro
Copy link
Contributor

@mithro mithro commented May 23, 2020

A lot of the files in repository have the old $Header: XXX $ and $Id: XXX $ style tags.

This pull request removes the expanded tags in the code and uses git filters + attributes to expand them on checkout.

For this to work, a person needs to run git config --local include.path ../.gitconfig to enable this to work so make setup-git is added to the Makefile.

  • $Id$ and $Revision$ gets expanded to the git describe --always --long output
  • $Date$ gets expanded to the date of the last value commit value.
  • $Author$ gets expanded to the email address of the person who last committed to the file.
  • $Header$ gets expanded to <filename> <git describe> <last modification commit date> <last modification author address>.

@RTimothyEdwards
Copy link
Owner

Pulled, but I have not followed the instructions above. When you say "a person needs to run", do you mean me? "../.gitconfig" implies that there is a ".gitconfig" outside of the repo? Why?

@mithro
Copy link
Contributor Author

mithro commented May 23, 2020

@RTimothyEdwards A person needs to run make setup-git after checking out for the first time.

The git config --local include.path ../.gitconfig means --local means relative to local git config file (which is .git/config). So <basepath>/.git/../.gitconfig which is the .gitconfig file at <basepath>/.gitconfig inside the repo.

@mithro mithro closed this May 23, 2020
@mithro mithro deleted the cvs-expansion-fixes branch May 23, 2020 22:40
@thud06 thud06 mentioned this pull request Feb 23, 2024
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