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 for case-sensitive file names #42

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lasermind
Copy link
Contributor

Hi QnA!

Today I wanted to help out a bit, but then I noticed, I need to help out a bit. ;)

I became aware of some horrible file names, that may technically do work on webservers, but as soon as they hit an NTFS file system (of a collaborator, i.e. me, in this case), it gets ugly. We don't want that to happen for no-one. Also GitHub Desktop is confused, believing a file has changed completely, but instead it is "just" a duplicate file due to the case-sensitive naming.

githubdesktopconfused

Long story short: I provided a fixed for that – luckily, it was just one occurrence.

Bonus: While at it, I also renamed related files to keep things semantically together, to not lose track anymore.

I renamed:
STOWAWAY.png → stowaway_diagram.png
stowaway.png → singletx_diagram.png
Stonewall.png → stonewall_diagram.png
STONEWALL X2(3).png → stonewallx2_diagram.png

The problematic files are currently only used in graphics.md and csimplications.md. I edited them also, to match the new file names.

Hope you're okay with that, and can pull this little tech fix quickly.
Also a good reminder to not use horrible file names. ;) There are some more (in /assets/img/), but they do not cause harm.

L

Fixed case-sensitive image file names, producing duplicate files (on some file systems). While at it, renamed related files to keep things semantically together.

Renamed:
STOWAWAY → stowaway_diagram
stowaway → singletx_diagram
Stonewall → stonewall_diagram
STONEWALL X2(3) → stonewallx2_diagram

The problematic files are currently only used in 'graphics.md' and 'csimplications.md'. I edited them to match the new file names.
@lasermind
Copy link
Contributor Author

Is this project dead, @bitlevi, @BitcoinQnA? Or just no time to maintain because of other things?

Would love to hear a short ping from you.

✌️ L

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