search

openhab / org.openhab.binding.zwave

openHAB binding for Z-Wave
Eclipse Public License 2.0
170 stars 202 forks source link

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

Open lucasec opened 7 months ago

lucasec commented 7 months ago

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:

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 commented 7 months ago

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 commented 7 months ago

Hopefully this should now be resolved.

apella12 commented 7 months ago

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 commented 5 months ago

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 commented 5 months ago

Thanks for fixing !