[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] New package: ssh-deploy
From: |
Michael Albinus |
Subject: |
Re: [ELPA] New package: ssh-deploy |
Date: |
Thu, 25 Oct 2018 11:36:13 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
Christian Johansson <address@hidden> writes:
> Hi!
Hi Christian,
> The ssh-deploy plug-in for Emacs makes it possible to effortlessly
> deploy local files and directories to remote hosts via TRAMP
> (including but not limited to SSH, SFTP, FTP). It tries to provide
> functions that can be easily used by custom scripts.
Thanks for this, it looks interesting.
I've checked README.md in order to get a first impression.
First of all: could you limit yourself to 80 chars per line in all your files?
This makes
reading them more comfortable.
## Features:
* Define syncing configuration per directory or per file (using
`DirectoryVariables` or `File Variables`)
* Control whether uploads of files should be automatic on save
This sounds like shadowfile.el functionality. What is the difference in
your implementation? (Note: shadowfile.el was broken for a long
time. I've fixed this recently only, in Emacs 27.0.50)
* Automatic and manual detection of remote changes of files using local
revisions
How do you do this? Polling? File notifications?
* Launch difference sessions for directories using a custom implementation of
recursive directory differences over TRAMP based on `ediff`
IIRC, ediff offers already something like this. What do you do else?
* All operations support asynchronous mode if `(make-thread`) or `async.el` is
installed. (You need to setup an automatic authorization for this, i.e.
`~/.netrc`, `~/.authinfo` or `~/.authinfo.gpg` and/or key-based password-less
authorization)
I would rather point to auth-source.el, the de-facto standard in
Emacs. And if the problems with user interactions are solved in the
feature/tramp-thread-safe branch, this restriction might not be needed
anymore.
((nil . (
(ssh-deploy-root-local . "/Users/username/Web/MySite/")
(ssh-deploy-root-remote . "/ssh:address@hidden:/var/www/MySite/")
(ssh-deploy-on-explicit-save . t)
(ssh-deploy-remote-sql-database . "myuser")
(ssh-deploy-remote-sql-password . "mypassword")
(ssh-deploy-remote-sql-user . "myuser")
)))
This looks like directoy-local variables. Have you thought to use
connection-local variables instead?
> Regards
> Christian
Best regards, Michael.