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

Case sensitivity issue with doc/actec/zerodim2pol_0_0.md #1897

Open
lucasec opened this issue Nov 24, 2023 · 5 comments
Open

Case sensitivity issue with doc/actec/zerodim2pol_0_0.md #1897

lucasec opened this issue Nov 24, 2023 · 5 comments

Comments

@lucasec
Copy link

lucasec commented Nov 24, 2023

My local git instance keeps showing that I have an unstaged change to the file doc/actec/zerodim2pol_0_0.md.

Was a little puzzled by this until I realized there are actually two versions of this file in the repo:

  • doc/actec/zerodim2pol_0_0.md (lowercase z)
  • doc/actec/Zerodim2pol_0_0.md (uppercase Z)

These both seem to have been introduced via a methodical database update by @cdjackson, so not sure if just submitting a PR to delete one of these files would be sufficient. The files are not quite exact duplicates of each other but I assume one is unintentional.

Obviously I could work around by doing development on a case sensitive filesystem, but may be worth fixing to avoid creating confusion for future contributors.

@cdjackson
Copy link
Collaborator

Thanks for pointing this out. I think removing the offending file will likely resolve this, but it does highlight an issue with the exporter because this obviously shouldn't have happened in the first place!

Leave it with me and I'll see if I can resolve this with an update to the export script.

@cdjackson
Copy link
Collaborator

Hopefully this should now be resolved.

@apella12
Copy link
Contributor

apella12 commented Nov 24, 2023

Thanks for fixing this. I have struggled with this orphan file for months. My bad for not creating an issue.

Not that it matters, but I'm going to try to fix the 700 controller pull request as I had problems with this file appearing

@apella12
Copy link
Contributor

My latest rebase of main popped up with a similar problem. Looks like it happened with #1914 with two files showing "modified" ( Capital Z)

doc\thermofloor\Zsmoke_0_0.md
src\main\resources\OH-INF\thing\thermofloor\Zsmoke_0_0.xml

but the lower case files appear to be in the binding

doc\thermofloor\zsmoke_0_0.md
src\main\resources\OH-INF\thing\thermofloor\zsmoke_0_0.xml

@apella12
Copy link
Contributor

Thanks for fixing !

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

No branches or pull requests

3 participants