Re: package.el strings

[Eli Zaretskii <eliz@gnu.org>]

> From: Jean-Christophe Helary <jean.christophe.helary@gmail.com>
> Date: Sun, 17 Jun 2018 23:02:08 +0900
> Cc: Emacs developers <emacs-devel@gnu.org>
> 
> There is indeed another change I want to make but that can wait. It involves removing some of the text decorations to be able to straighten some strings.
> 
> If there is an agreement to proceed I can do that later. In the meanwhile it would be nice if you could first commit the biggest part of the modifications.

Thanks, but there are a couple of hunks I don't understand:

>  (defcustom package-archives `(("gnu" .
> -                               ,(format "http%s://elpa.gnu.org/packages/"
> -                                        (if (gnutls-available-p) "s" ""))))
> +                               ,(let ((https "https://elpa.gnu.org/packages/")
> +                                      (http   "http://elpa.gnu.org/packages/"))
> +                                  (if (gnutls-available-p) https http))))
> +

Why did you need this change?

> -           (dir (expand-file-name (format "archives/%s" name) package-user-dir))
> +           (dir (expand-file-name (concat "archives/" name) package-user-dir))

Same question here.
> -        (message "No packages to upgrade.")
> +        (message "No packages to upgrade")

I think our style is not to include a period in messages.