adafruit / Adafruit_CircuitPython_Bundle

A bundle of useful CircuitPython libraries ready to use from the filesystem.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Incorrect Link/Submodule

sommersoft opened this issue · comments

Rolling through the doc updates, I discovered that the drivers/ds18x20 module link points to Adafruit_CirctuitPython_OneWire. While a DS18x20 is a OneWire device, it does have it's own library...

MS Paint screenshot...
ds18x20_onewire_mixup

The hash listed on drivers/ds18x20 matches the last commit to OneWire.

3 days ago (from current writing), adabot updated OneWire twice using the same 3 PRs...not sure if that's the culprit, just the only thing I found out of the ordinary looking through the history.

Oh weird, yeah it must have been a copy paste mistake when adding those modules. I just updated the .gitmodules to point at the right path. If you already have the repository cloned you probably want to delete and re-clone with a full recursive clone to make sure you get the latest submodule state. Apparently a 'git submodule sync' would also update the state but I tried it and it didn't do what I expected (very likely some odd submodule command incantation is required to make git happy--easier to just reclone).