Skip to content

Conversation

oalders
Copy link
Member

@oalders oalders commented Apr 21, 2018

This can be merged now.

@oalders oalders force-pushed the oalders/permissions branch from 2b283c1 to b0f5aac Compare April 21, 2018 23:38
@oalders oalders requested a review from ranguard April 21, 2018 23:55
@oalders
Copy link
Member Author

oalders commented Apr 21, 2018

This is needed to be able to start up the web app both post puppet upgrade. In its current state this code should be fine to be merged now.

Copy link
Member

@haarg haarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the less css compiler really getting installed in /usr/bin/less? How is that happening? If that's really the case, that's the thing that should be fixed. The less css compiler should never have an executable named less.

@oalders
Copy link
Member Author

oalders commented May 18, 2018

This is me being in a hurry and not getting it done properly. I'll rework this PR. Will flag it as WIP for the time being.

@oalders oalders changed the title s/lessc/less/ [WIP] s/lessc/less/ May 18, 2018
@oalders
Copy link
Member Author

oalders commented May 25, 2018

This will be fixed in puppet.

@oalders oalders closed this May 25, 2018
@oalders oalders deleted the oalders/permissions branch May 25, 2018 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants