[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/dockerfile-mode 26239c3b95 043/104: Resolve spotify/#24
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/dockerfile-mode 26239c3b95 043/104: Resolve spotify/#24 |
Date: |
Sat, 29 Jan 2022 07:58:31 -0500 (EST) |
branch: elpa/dockerfile-mode
commit 26239c3b950ee738db2d1a9935f1198428ab0fc7
Author: Xi Shen <david.shen@CNSHHQ-0887.ef.com>
Commit: Xi Shen <david.shen@CNSHHQ-0887.ef.com>
Resolve spotify/#24
Use `cygwin-convert-file-name-to-windows' to handle filename in Cygwin
environment, and `convert-standard-filename' in other POSIX environment.
---
dockerfile-mode.el | 24 +++++++++++++++++++++---
1 file changed, 21 insertions(+), 3 deletions(-)
diff --git a/dockerfile-mode.el b/dockerfile-mode.el
index b14eae99b8..19a14f7807 100644
--- a/dockerfile-mode.el
+++ b/dockerfile-mode.el
@@ -19,6 +19,8 @@
(require 'sh-script)
(require 'rx)
+(declare-function cygwin-convert-file-name-to-windows "cygw32.c" (file
&optional absolute-p))
+
(defvar docker-image-name nil)
(defgroup dockerfile nil
@@ -80,6 +82,14 @@
(unless dockerfile-mode-abbrev-table
(define-abbrev-table 'dockerfile-mode-abbrev-table ()))
+(defun standard-filename (file)
+ "Convert the file name to OS standard.
+If in Cygwin environment, uses Cygwin specific function to convert the
+file name. Otherwise, uses Emacs' standard conversion function."
+ (format "%s" (if (fboundp 'cygwin-convert-file-name-to-windows)
+ (s-replace "\\" "\\\\" (cygwin-convert-file-name-to-windows
file))
+ (convert-standard-filename file))))
+
;;;###autoload
(defun dockerfile-build-buffer (image-name)
"Build an image based upon the buffer"
@@ -90,7 +100,11 @@
(save-buffer)
(if (stringp image-name)
(async-shell-command
- (format "%sdocker build -t %s -f \"%s\" \"%s\"" (if dockerfile-use-sudo
"sudo " "") image-name (buffer-file-name) (file-name-directory
(buffer-file-name)))
+ (format "%sdocker build -t %s -f \"%s\" \"%s\""
+ (if dockerfile-use-sudo "sudo " "")
+ image-name
+ (standard-filename (buffer-file-name))
+ (standard-filename (file-name-directory (buffer-file-name))))
"*docker-build-output*")
(print "docker-image-name must be a string, consider surrounding it with
double quotes")))
@@ -104,9 +118,13 @@
(save-buffer)
(if (stringp image-name)
(async-shell-command
- (format "%s docker build --no-cache -t %s -f \"%s\" \"%s\"" (if
dockerfile-use-sudo "sudo" "") image-name (buffer-file-name)
(file-name-directory (buffer-file-name)))
+ (format "%s docker build --no-cache -t %s -f \"%s\" \"%s\""
+ (if dockerfile-use-sudo "sudo" "")
+ image-name
+ (standard-filename (buffer-file-name))
+ (standard-filename (file-name-directory (buffer-file-name)))
"*docker-build-output*")
- (print "docker-image-name must be a string, consider surrounding it with
double quotes")))
+ (print "docker-image-name must be a string, consider surrounding it with
double quotes"))))
;; Handle emacs < 24, which does not have prog-mode
(defalias 'dockerfile-parent-mode
- [nongnu] elpa/dockerfile-mode 8771eb1d9a 022/104: Use async-shell-command instead of shell-command, (continued)
- [nongnu] elpa/dockerfile-mode 8771eb1d9a 022/104: Use async-shell-command instead of shell-command, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 6ed974d462 014/104: Autoload docker mode for Dockerfiles., ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode d1e9754285 037/104: Fix build failure when path of Dockerfile contains spaces., ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode e20c7e587a 025/104: Correctly set require-final-newline, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 35d10860d9 039/104: Fix inability to build images on Windows., ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 98434943c3 005/104: Adjust auto-mode regexp in README., ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 1015f3818d 016/104: Added copy and onbuild commands, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode a532e30371 023/104: Use syntax highlighting in README, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 6a64806b78 029/104: now treats single quotes like double quotes, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 34a0a5d082 030/104: Merge branch 'master' of github.com:spotify/dockerfile-mode, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 26239c3b95 043/104: Resolve spotify/#24,
ELPA Syncer <=
- [nongnu] elpa/dockerfile-mode fdebd1eda5 044/104: Pass --build-args to docker build, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode c462ba8650 007/104: Merge pull request #1 from thomasf/master, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 730b2dbd84 010/104: Update Dockerfile keywords to match docker 0.7.6, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 1fd094a409 009/104: Merge pull request #2 from purcell/patch-1, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 8f74b941d2 035/104: Add support for ARG command, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 93d6013437 046/104: Resolve spotify/#24, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode fd1cf02f06 040/104: Merge pull request #22 from jaccarmac/fix-windows-build, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 8273401efa 045/104: Merge pull request #26 from rcj/master, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode a464ee6e91 047/104: Merge branch 'master' of github.com:davidshen84/dockerfile-mode, ELPA Syncer, 2022/01/29
- [nongnu] elpa/dockerfile-mode 5f1e17e4e5 050/104: Merge pull request #27 from ju2wheels/feature/add_missing_keywords, ELPA Syncer, 2022/01/29