misohena / el-easydraw

Embedded drawing tool for Emacs
https://misohena.jp/blog/2021-09-21-emacs-easy-draw.html
GNU General Public License v3.0
769 stars 21 forks source link

Miscellaneous features and fixes #13

Closed xgqt closed 1 year ago

xgqt commented 1 year ago

HUGE THANKS FOR AWESOME PACKAGE!

This PR:

misohena commented 1 year ago

PRを書いて頂いたのに申し訳ありませんが、あまり必要性を感じないのでマージするのは止めておきます。 いかに自分がそれを必要としているかを書いて頂けるともっと伝わると思います。 Packageメタデータについては誤解を与えたようなので後で削除するかもしれません。私はその方法でバージョン番号を管理するつもりがありません。

xgqt commented 1 year ago

@misohena

いかに自分がそれを必要としているかを書いて頂けるともっと伝わると思います

I think my comment on the PR explains it. To be more clear the two main things that I wished were in el-easydraw are:

Packageメタデータについては誤解を与えたようなので後で削除するかもしれません。

This is the response to the comment I made on one of commits I believe. This PR has nothing to do with that.

私はその方法でバージョン番号を管理するつもりがありません。

It is good practice and helpful if somebody (like you) wants to distribute the package. It is also helpful for OS maintainers who package Emacs Lisp packages in their package repositories.

misohena commented 1 year ago
misohena commented 1 year ago

すみません、cl-libの件は私の間違いでした。今確かめたら確かに警告が出ました。コマンドラインからMakefileでやっているのと同じようにバイトコンパイルして確かめたと思ったのですが、何かオプションに漏れがあったのかもしれません。 e77d58d6b9663382bf04cbceec54609ff933aed4 で修正しました。