search

rm-hull / luma.lcd

Python module to drive PCD8544, HT1621, ST7735, ST7567 and UC1701X-based LCDs
https://luma-lcd.readthedocs.io
MIT License
158 stars 57 forks source link

aux.py can't write on MS Windows #84

Closed pasquyonline closed 5 years ago

pasquyonline commented 5 years ago

Hi, guys, first, this library is wonderfull, great job! second, I need to clone the project on MS Windows machine, but on this operating system (any version!!!) a file can't be named "aux.xx" (xx stand for anything you want) so the file aux.py can't be created.

So, my request is: it is possible to rename this file to _aux.py for example?

Thanks in advice! Bye

thijstriemstra commented 5 years ago

we're talking about https://github.com/rm-hull/luma.lcd/blob/master/luma/lcd/aux.py right? Are you serious, this would blow my mind..

pasquyonline commented 5 years ago

Yes is that file. Believe me, I'm very serious! I didn't even believe it, then I did tests on Windows 10 and Windows 7: same error message.

This is a link that describe "the thing"!! https://www.reddit.com/r/linuxmasterrace/comments/9ukft0/windows_10_still_cant_create_a_file_called_auxtxt/?depth=2

pasquyonline commented 5 years ago

I tried also on a Windows Xp on a virtual machine and also in that operating system there is that limitation!

rm-hull commented 5 years ago

Related issue: #44

thijstriemstra commented 5 years ago

@rm-hull what are your thoughts on renaming it?

rm-hull commented 5 years ago

Wondered about creating a backlit_device class with this functionality, and then those devices that have a backlight would inherit from that instead of device, and then we could just remove the aux.py file completely.

thijstriemstra commented 5 years ago

sounds good. removing a module is backwards-incompatible though (so bump version to nxt level?)

rm-hull commented 5 years ago

so bump version to nxt level?

Agreed - it would have to be v2.0.0