setpermission.sh missing some directories

[slaboure]

For an apt-get installation to work, setpermission.sh must be modified to also "chown" /var/log/openhab (for logs) and /usr/share/openhab (for Dropbox timestamp files for example). This is not being done on 1.7.1

[teichsta]

could you have a look please @theoweiss?

[theoweiss]

Ok, I will check that.

[teichsta]

any chance to have a quick at this one today @theoweiss ?

[theoweiss]

Sorry no chance: a serious cold is visiting my family. I have to get these things managed.

[theoweiss]

@slaboure can you describe the issue a bit more in detail.
Normally set permissions is only needed when the openhab user in /etc/default/openhab was reconfigured or the openhab service has been started in between manually as root. It's meant for handling these special cases.

[teichsta]

Sorry no chance: a serious cold is visiting my family. I have to get these things managed.

no worries … get well soon!

[teichsta]

any update on this @slaboure @theoweiss ?

[theoweiss]

I will provide a fix till the weekend.

[theoweiss]

setpermissions.sh already changes permissions of /var/log/openhab . /usr/share/openhab has to be readonly for the openhab user, for security reasons. Files should only be changed in the $userdata directory and the log dir. The dropbox binding is already fixed with #3451. Open issues of setpermissions.sh should be fixed with #4166 .

[teichsta]

ftr: #4166 has been merged meanwhile …

[slaboure]

perfect, thanks!

On Sun, Mar 13, 2016 at 2:17 PM, Thomas Eichstädt-Engelen <
notifications@github.com> wrote:

ftr: #4166 #4166 has been merged
meanwhile …

—
Reply to this email directly or view it on GitHub
#3601 (comment).