Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Fix a number of issues #15

Merged
merged 1 commit into from
Jan 3, 2024
Merged

Fix a number of issues #15

merged 1 commit into from
Jan 3, 2024

Conversation

snoyberg
Copy link
Contributor

These were uncovered when researching commercialhaskell/stack#105. What
I discovered:

  • There was an extra layer of .git directories in computed paths
  • The HEAD file should be added dependently regardless of whether it's a
    detached head
  • Add a missing check in relRef for newline
  • Switch newline handling to check both CR and LR for Windows support

These were uncovered when researching commercialhaskell/stack#105. What
I discovered:

* There was an extra layer of `.git` directories in computed paths
* The HEAD file should be added dependently regardless of whether it's a
  detached head
* Add a missing check in relRef for newline
* Switch newline handling to check both CR and LR for Windows support
@snoyberg
Copy link
Contributor Author

snoyberg commented Jul 4, 2017

Bumping

@bitemyapp
Copy link

Bump

@ulidtko
Copy link

ulidtko commented Jul 6, 2023

FWIW, tested the patch — it does fix gitrev failing to invalidate the TH splice on git changes.

Absolutely must-merge.

@yitz-zoomin
Copy link

Ping @acfoltzer pls pls merge this PR. Thanks.

@acfoltzer
Copy link
Owner

I'm fine with merging this but I do not develop Haskell any longer, so have no means of testing it myself. Please feel free to make a fork if you'd like to continue further development of this project.

@acfoltzer acfoltzer merged commit 2f80720 into acfoltzer:master Jan 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants