search

cloudsidedev / appside

Multitenant environment automation.
http://cloudside.ch
GNU Affero General Public License v3.0
38 stars 7 forks source link

`appflow vhosts` adds too many newlines #115

Closed ocean90 closed 6 years ago

ocean90 commented 6 years ago

Tested on macOS with latest version.

Before:

127.0.0.1       localhost

First run:

127.0.0.1       localhost

192.168.80.2 atlantis

(Two before, one after)

Second run:

127.0.0.1       localhost

192.168.80.2 atlantis

(Two before, three after)

Third run:

127.0.0.1       localhost

192.168.80.2 atlantis

(Two before, five after)

ivomarino commented 6 years ago

hi @ocean90, thanks for reporting this, the issue probably is here: https://github.com/ttssdev/appflow/blob/master/lib/appflow_tools.py#L136-L137 -- we take a look for fixing this asap.

89luca89 commented 6 years ago

Eventually we can switch to this library https://github.com/jonhadfield/python-hosts this will ease the management of the hosts

89luca89 commented 6 years ago

Should be fixed now.