emacs-devel
[Top][All Lists]
Advanced

[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.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]