From: Stefan Monnier <monnier@iro.umontreal.ca>
To: morgan@ice9.digital
Cc: emacs-devel@gnu.org
Subject: Re: [ELPA] New package: kixtart-mode
Date: Wed, 09 Nov 2022 17:04:17 -0500 [thread overview]
Message-ID: <jwvv8nnom0t.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <631d-636c1380-131-1f4ad1e0@218117756> (morgan@ice9.digital's message of "Wed, 09 Nov 2022 21:55:27 +0100")
Hi Morgan,
> I'd like to submit language support for KiXtart as a GNU ELPA package
> (patch is attached).
Thanks, looks good. Find below my sig a suggested patch to fix a few
minor issues I found along the way (the main one is arguably the way
`kixtart-tempo-tags` is passed to `tempo-define-template`).
> There is a slight complication in that there is an old kixtart-mode
> package on MELPA which just provided some font locking. I do have
> permission from the author of this previous package to take over the
> package name, effectively replacing the MELPA package with the new GNU
> ELPA package. See confirmation from the author of the previous package
> here: https://github.com/ryrun/kixtart-mode/issues/2
Looks fine from here.
> I have gone through the FSF copyright assignment process and received
> confirmation that it was successfully completed.
The only issue on my side is that your name doesn't yet appear in the
`copyright.list` file I can get from the FSF. Sometimes the FSF clerk
forgets to update the file on the server, so it's nothing to worry about
(and sometimes it's a mismatch in the names I use which make my searches
fail).
> Would there be any issues in adding the package?
There is one issue, yes: is KiXtart Free Software?
According to Wikipedia, the license is "Closed source Careware".
And indeed, I can't find any source code on http://www.kixtart.org/.
If that's indeed proprietary code, then I think we probably wouldn't
want to host `kixtart-mode` on (Non)GNU ELPA.
Stefan
diff --git a/kixtart-mode.el b/kixtart-mode.el
index 0cb8b9edc4..72b7dc6724 100644
--- a/kixtart-mode.el
+++ b/kixtart-mode.el
@@ -848,7 +848,7 @@ removed when template insertion is interactive."
(remove 'p template)
template))))
-(defconst kixtart-tempo-tags nil
+(defvar kixtart-tempo-tags nil
"Tempo tags for KiXtart Mode.")
(define-abbrev-table 'kixtart-mode-abbrev-table nil
@@ -863,18 +863,17 @@ TAG, DOCUMENTATION, and ELEMENTS are passed directly to
`tempo-define-template'. TAG is also used as the abbrev string
which will be expanded to the template."
(declare (indent 2))
- (let ((template (gensym)))
- `(let ((,template (tempo-define-template (concat "kixtart-" ,tag)
- (quote ,@elements)
- ,tag
- ,documentation
- kixtart-tempo-tags)))
- (define-abbrev kixtart-mode-abbrev-table ,tag "" (identity ,template)
+ (let ((funname (intern (concat "kixtart-template-" tag))))
+ `(progn
+ (defalias ',funname
+ (tempo-define-template (concat "kixtart-" ,tag)
+ (quote ,@elements)
+ ,tag
+ ,documentation
+ 'kixtart-tempo-tags))
+ (define-abbrev kixtart-mode-abbrev-table ,tag "" #',funname
:system t)
- (put (identity ,template) 'no-self-insert t)
- (defalias (intern (concat "kixtart-template-" ,tag))
- (identity ,template)
- ,documentation))))
+ (put ',funname 'no-self-insert t))))
(kixtart--define-template
"while"
@@ -962,16 +961,16 @@ which will be expanded to the template."
(define-key map (kbd "C-c C-t C-b") #'tempo-backward-mark)
(define-key map (kbd "C-c C-t C-f") #'tempo-forward-mark)
(define-key map (kbd "C-c C-t C-t") #'tempo-complete-tag)
- (define-key map (kbd "C-c C-t I") 'kixtart-template-ifelse)
- (define-key map (kbd "C-c C-t c") 'kixtart-template-case)
- (define-key map (kbd "C-c C-t d") 'kixtart-template-do)
- (define-key map (kbd "C-c C-t e") 'kixtart-template-foreach)
- (define-key map (kbd "C-c C-t f") 'kixtart-template-for)
- (define-key map (kbd "C-c C-t i") 'kixtart-template-if)
- (define-key map (kbd "C-c C-t l") 'kixtart-template-else)
- (define-key map (kbd "C-c C-t s") 'kixtart-template-select)
- (define-key map (kbd "C-c C-t u") 'kixtart-template-function)
- (define-key map (kbd "C-c C-t w") 'kixtart-template-while)
+ (define-key map (kbd "C-c C-t I") #'kixtart-template-ifelse)
+ (define-key map (kbd "C-c C-t c") #'kixtart-template-case)
+ (define-key map (kbd "C-c C-t d") #'kixtart-template-do)
+ (define-key map (kbd "C-c C-t e") #'kixtart-template-foreach)
+ (define-key map (kbd "C-c C-t f") #'kixtart-template-for)
+ (define-key map (kbd "C-c C-t i") #'kixtart-template-if)
+ (define-key map (kbd "C-c C-t l") #'kixtart-template-else)
+ (define-key map (kbd "C-c C-t s") #'kixtart-template-select)
+ (define-key map (kbd "C-c C-t u") #'kixtart-template-function)
+ (define-key map (kbd "C-c C-t w") #'kixtart-template-while)
(define-key map (kbd "C-c C-u") #'kixtart-up-script-block)
map))
next prev parent reply other threads:[~2022-11-09 22:04 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-09 20:55 [ELPA] New package: kixtart-mode morgan
2022-11-09 21:10 ` Philip Kaludercic
2022-11-09 21:24 ` Morgan Willcock
2022-11-09 22:04 ` Stefan Monnier [this message]
2022-11-09 22:48 ` Morgan Willcock
2022-11-09 23:58 ` Stefan Monnier
2022-11-10 0:20 ` Morgan Willcock
2022-11-10 0:40 ` Juanma Barranquero
2022-11-13 4:18 ` Richard Stallman
2022-11-13 7:09 ` Eli Zaretskii
2022-12-11 23:33 ` Richard Stallman
2022-12-12 12:14 ` Eli Zaretskii
2022-12-11 23:50 ` Óscar Fuentes
2022-12-12 6:17 ` North Year
2022-12-12 12:21 ` Eli Zaretskii
2022-12-12 14:00 ` Óscar Fuentes
2022-12-20 23:59 ` Morgan Willcock
2022-11-10 8:16 ` Akib Azmain Turja
2022-11-10 8:25 ` Eli Zaretskii
2022-11-10 9:00 ` Akib Azmain Turja
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jwvv8nnom0t.fsf-monnier+emacs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=emacs-devel@gnu.org \
--cc=morgan@ice9.digital \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.