-
Notifications
You must be signed in to change notification settings - Fork 7
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
Configurator #48
base: main
Are you sure you want to change the base?
Configurator #48
Conversation
Sync repo
tuned makefile -> new target make reinstall -> reinstalles everything without the secrets (used for update procedure) umbenannt: install_systemd.sh -> old/install_systemd.sh move install_systemd.sh to old - install is done by the make neue Datei: scripts/battery.sh giving battery its own script geändert: scripts/ip.sh bug in ip on raspbian (empty json object - see comments in the code) geändert: scripts/system.sh removed battery status misc: added systemctl return codes
…ency / fixed return code in systemctl
I've taken a very brief look through, though not enough to merge yet, just didn't want to leave you hanging. My immediate thought is that - assuming this doesn't alter existing functionality - I'll be willing to merge. That being said, here are my thoughts for future work on I would want to move towards a Go based solution that will do all the same work internally (so, just some simple toggles in the config). It shouldn't be too hard to write a custom bit of code that can hook into the same functions that call the scripts and provide the same functionality. As an intermediate step, the existing functionality could be made into a separate portion of the tool that is run only when an argument is called, for example, The reason I would prefer an internal solution (in Golang, not scripts) is for interoperability concerns - it is much easier to build a Windows/Mac OS version of the tool when built in devices do not depend on scripting and tools. To be clear, I use neither Windows nor Mac OS at home, but it would all the same be desirable for the sake of others (most users are on Windows, after all) As a personal preference, I also prefer not to use Makefiles and the like for Golang projects. PS: I really do appreciate the time and effort going into this, I'm sorry I can't be more responsive. |
Hi, thanks for your message. First off all, thanks for the insides. I see the problem with the platform with the platform in/dependency. Firstly I have to mention that I am not a go developer (and not even a developer :-)). So the only way to contribute from my side is with some scripts and a little bit of python. I am not an english native speaker - so due to you message - I have to ask some questions back: a) As far as I understand you - you like to "transfer" or "add" some functionality, which is actually covered by one or more scripts to ha-mqtt-iot natively? If you can answer both questions with yes - I have the following proposal:
So it should be easier to contribute scripts, etc for the different platforms, until you included the functionallity internally in ha-mqtt-iot. This proposal has the following advantages: What do you think - is this a good solution to keep your project goals in focus and open a way for me to contribute to your project with shell scripts and the config generator? |
Eventually, yes, but that will take time and effort - scripts are still the way forward for now.
Absolutely - I will always support running custom scripts, that's the main goal of the project.
I could, but let's wait until I can review this more - I don't think we need to throw anything out just yet
Again, could do, but I want to see what you already have first, it could be fine as is.
That actually sounds like a good plan. I don't know the details of what scripts we'd need on Windows, but I'm pretty sure we could make that work. Assuming some scripts are portable across Linux/Mac OS, we can symlink them (if somebody can test)
Agreed.
I think this is a great plan, other than closing this. Perhaps we can adapt this PR, or I might still merge and we can work from there. |
Just a note that I still want to look at this at some stage, just haven't made the time. |
Hello,
short summery about the changes:
is take care of in the Makefile
tested on:
Would be great if you can merge this upstream. Yes I know its a huge merge - but I don't want to bore you with less untested or unfinished code.
Kind regards
dev-foo-bar