search

geissdoerfer / shepherd

MIT License
13 stars 6 forks source link

updated ptp-slave role #6

Closed mchwalisz closed 5 years ago

mchwalisz commented 5 years ago

It should now match the debian package.

geissdoerfer commented 5 years ago

I was complaining about the duplicated code between the shepherd-ptp debian package and this ansible role offline. Wouldn't it make sense to at least have the systemd override files for ansible pulled in from the path of the debian package?

mchwalisz commented 5 years ago

I was thinking about the issue. We should, actually, replace most of the ansible role with tasks to install the debian package provided by us. This will unify the code and make it easier to keep track of both.

It of course depends on what we want to achieve with ansible playbooks.

geissdoerfer commented 5 years ago

Thanks. In fact, there is already an ansible playbook to install the debian packages, aimed at actual users. For developers it's inconvenient to build the debian packages first. I'd still like to offer a more direct way. On the other hand, once you actually develop, you'll probably anyway just use make install and so on to install just the components that you're currently working with. So maybe we just provide a playbook to setup all requirements like build tools and instructions how to setup a local development environment..