search

tubemogul / puppet-aptly

Puppet module to deploy and manage aptly, a Debian repository management tool http://www.aptly.info/
https://forge.puppet.com/TubeMogul/aptly
Other
5 stars 28 forks source link

Fix syntax #30

Closed dhoppe closed 7 years ago

dhoppe commented 7 years ago

Please fix syntax, because the additional chars break the init script. This will fix #21.

aerostitch commented 7 years ago

What was wrong with the double quotes? You had an error?

rhoml commented 7 years ago

I can confirm that this extra characters break aptly start once they are removed aptly start normally

dhoppe commented 7 years ago

There is no error message, but the aptly.pid contains a process id which does not exist and the init script for aptly-api do not use double quotes and works as expected.

aerostitch commented 7 years ago

Sounds good then. I never noticed it before. Thanks for pointing it out!