branch: elpa/php-mode
commit 904313508d2884c8cd39a15d509cc89f42a2951d
Author: USAMI Kenta <[email protected]>
Commit: USAMI Kenta <[email protected]>

    Ship php-flymake.el and php-align.el in the package
    
    Neither file was listed in Eask's `files', so neither ended up in the
    built package.  php-mode.el requires php-flymake at load time, which
    makes the result unusable:
    
        $ eask package /tmp/dist
        $ tar xf /tmp/dist/php-mode-1.26.1.tar && cd php-mode-1.26.1
        $ emacs -Q --batch -L . --eval "(require 'php-mode)"
        Eager macro-expansion failure: (file-missing "Cannot open load file"
          "No such file or directory" "php-flymake")
    
    php-flymake.el has been missing from the list since it was added in
    dc67eae, and php-align.el since the move into lisp/ in cc63080.  CI
    never caught it because `eask test ert' runs against the source tree,
    where both files are on the load path either way.
    
    php-align.el is not required by anything -- users load it themselves --
    but it is just as much part of the package, and shipping it is the point
    of it being here.
    
    With both listed, the built package loads and php-flymake is defined.
    The two `rx' warnings php-flymake.el now contributes to `eask compile'
    are pre-existing.
---
 Eask | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Eask b/Eask
index b1a157d6f2..deef9331f4 100644
--- a/Eask
+++ b/Eask
@@ -16,11 +16,13 @@
  "lisp/php-indent.el"
  "lisp/php-keywords.el"
  "lisp/php-face.el"
+ "lisp/php-flymake.el"
  "lisp/php-format.el"
  "lisp/php-project.el"
  "lisp/php-local-manual.el"
  "lisp/php-ide-phpactor.el"
  "lisp/php-ide.el"
+ "lisp/php-align.el"
  "lisp/php-mode-debug.el")
 
 (script "test" "echo \"Error: no test specified\" && exit 1")

Reply via email to