**** BEGIN LOGGING AT Mon Jun 04 03:00:00 2018 Jun 04 04:08:13 ravikp7: (y) Jun 04 06:13:02 ping me if you guys wish to do the PyPruss meeting Jun 04 06:45:13 zeekhuge: Wormo : abhishek_ : I have completed the cpp bindings and the daemon. The swig generated bindings are working fine as well. Please review it, if possible. Jun 04 06:47:55 zeekhuge: about the filepaths, i have reserved a location for each pru in /tmp/pruX where the firmware will be copied by the bindings and then to /lib/firmware by the daemon. I didn't understand why a config file is needed. Can you elaborate on it a bit more? Jun 04 07:06:36 muneeb17: Using a `config` file is a standard procedure used with daemons. The thing is, If you have a variable (value can be changed) and cannot ask user about that variable in an interactive way (daemons are supposed to be non-interactive) , and also have permissions issues (/etc/*) files require root permissions to be changed, then you can use a file in such cases. We call such a file a `config` file. Jun 04 07:07:28 Other option you have is add a command line parameter to the daemon start command, and do not start daemon without that parameter. Jun 04 07:07:50 The `/tmp/pruX` approach provides no security benefits, and is still vulnerable. Jun 04 07:12:03 Ok, got it.. I'm also facing an issue with the ExecStartPost command. I need to change the permissions of the socket file after it is created, so i put chmod in the StartPost but it throws an error. Maybe a config file can do that too... Jun 04 07:17:40 zeekhuge: also, in the daemon, os.chmod is not working for the socket file, so was thinking of other ways Jun 04 07:17:58 I dont get it. Jun 04 07:18:01 What are you using for server thing ? Jun 04 07:18:14 unix sockets Jun 04 07:18:46 oh ! okay. Jun 04 07:19:09 Isnt there any such library that would do all this socket thing itself ? Jun 04 07:19:30 yeah, i used python socketserver Jun 04 07:19:48 had to write c++ client though Jun 04 07:24:52 zeekhuge: Also, what about the plan for next week? Jun 04 07:25:10 When is the first review ? Jun 04 07:25:17 the official one .. Jun 04 07:25:30 starts on 11th Jun 04 07:25:46 11-15 Jun 04 07:26:37 `MyRequestHandler` not a good name. Jun 04 07:30:42 Also, you can similarly use the config file to specify the RPMsg channel name Jun 04 07:30:59 Can you please push your latest code ? or is it already up ? Jun 04 07:31:20 yeah, i have pushed it a few minutes back Jun 04 07:33:20 Ok, got it.. I'm also facing an issue with the ExecStartPost command. I need to change the permissions of the socket file after it is created, so i put chmod in the StartPost but it throws an error. Maybe a config file can do that too... Jun 04 07:33:21 What error does it throw ? Jun 04 07:37:27 zeekhuge: When i start the service, it runs fine, but after reload, just gives `unit entered failed state` Jun 04 07:37:59 with a failure exit code.. not very informative Jun 04 07:40:04 this is my unit file : https://gist.github.com/MuneebMohammed/d8c9f528d9a8bef9a8aa3e5b86870799 Jun 04 07:41:23 muneeb17: this *.service file is for `/etc/init.d/` na ? Jun 04 07:41:56 how about using this format - https://github.com/fhd/init-script-template/blob/master/template Jun 04 07:42:00 zeekhuge: I'm using systemd Jun 04 07:45:20 In that case, did you try using `ExecReload` ? Jun 04 07:46:47 To prevent the reload crash .. Jun 04 07:49:08 It's not exactly a reload thing. the service fails during startup too Jun 04 07:50:36 and I tried using that ExecReload too.. but in vain Jun 04 07:52:50 muneeb17: Please mention me in the convo, I would forget that we were talking otherwise :P Jun 04 07:52:59 So, try journalctl Jun 04 07:53:14 journalctl is logger for systemd services Jun 04 07:53:28 `man journalctl` Jun 04 07:54:37 zeekhuge: Ok, I will try it. Jun 04 07:54:39 from the man, looks like `journalctl -u ` should do .. Jun 04 07:56:31 zeekhuge: gives the same log as systemctl status. Jun 04 07:58:20 muneeb17: Does it work fine when you start the service manually ? Jun 04 07:59:11 zeekhuge: It only works, when i do daemon-reload and start. Doesn't work in any other case Jun 04 08:00:00 I mean manually ... like executing the `ExecStart` and all commands from the terminal Jun 04 08:00:39 yeah, it works fine like that Jun 04 08:00:53 zeekhuge: ^ Jun 04 08:02:41 zeekhuge: maybe the socket isn't created in the time between ExecStart and ExecStartPost ? and it throws a file not found error, maybe? Jun 04 08:05:27 muneeb17:`man systemd.exec` Jun 04 08:05:36 you see StandardOutput//Error there ? Jun 04 08:05:50 try setting them to journal ? Jun 04 08:06:42 Ideally, it should execute `ExecStartPost` only when `ExecStart` returns .. and in a similar way for others too . Jun 04 08:08:31 zeekhuge: I added sleep 1 to the ExectStartPost and its working. Jun 04 08:09:59 commit and push ? Jun 04 08:10:21 or maybe pastebintit ? Jun 04 08:10:35 *pastebinit Jun 04 08:10:43 muneeb17: ^^ Jun 04 08:12:22 zeekhuge: I pushed the commit Jun 04 08:13:09 https://github.com/MuneebMohammed/PRUSS-Bindings/commit/fae83d70b0a575ee94051f8a0354f0846d55c155 Jun 04 08:27:41 zeekhuge: With better commit message : https://github.com/MuneebMohammed/PRUSS-Bindings/commit/939bc6c2f9b65df096b43296700aaafd9084add4 Jun 04 08:27:58 so... what about the next week? Jun 04 08:28:07 That sleep is not elegant. Jun 04 08:28:21 I tried this - http://paste.ubuntu.com/p/jtThNXdmKn/ Jun 04 08:28:34 and it worked completely fine. Jun 04 08:28:35 muneeb17: ^ Jun 04 08:29:38 zeekhuge: didn't work for me, but.. i'll try again Jun 04 08:35:57 muneeb17: Any error ? or just not working ? Jun 04 08:36:40 Make sure you use chmod before server_forever Jun 04 08:40:27 zeekhuge: ughh! chmod before serve_forever it is. Sorry for wasting your time... Jun 04 08:41:49 Its okay. I should have tried this earlier. Jun 04 08:42:14 Will leave now. Work calling (been calling for an hour now :P) Jun 04 08:42:37 zeekhuge: yeah, sure. Thanks a lot! Jun 04 15:23:49 In this ever changing world, now Microsoft is acquiring Github! Jun 04 15:24:45 no one could even think of it, 4-5 years back Jun 04 15:41:57 Yep. Jun 04 16:55:02 * anirban1998[m] sent a long message: anirban1998[m]_2018-06-04_16:55:01.txt Jun 04 17:14:12 abhishek_: zeekhuge : Wormo : What is the plan for the next week? DMA? Jun 04 18:04:52 Will get back on this tomorrow. Jun 04 18:05:33 abhishek_: So, should I submit the report tomorrow? Jun 04 18:06:17 Submit what you have done, and update it tomorrow. **** BEGIN LOGGING AT Tue Jun 05 02:05:33 2018 **** ENDING LOGGING AT Tue Jun 05 03:00:14 2018