Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

niseabout.lをマルチフレーム版に含めたい #2

Closed
xyzzy-17-638 opened this issue May 25, 2012 · 7 comments
Closed

niseabout.lをマルチフレーム版に含めたい #2

xyzzy-17-638 opened this issue May 25, 2012 · 7 comments

Comments

@xyzzy-17-638
Copy link
Owner

@snmsts さん

こんにちは!

佐野さんが xyzzy Wiki で公開されている拡張lisp/niseaboutマルチフレーム版の標準の about.l としたいと思っています。

同梱の可否、配布条件等について教えていただけないでしょうか?

@snmsts
Copy link

snmsts commented May 25, 2012

xyzzyにライセンスが付く前のブツですよね。
良きようにお願いいたします。
多分、本体互換のライセンスということでいくとMITでお願いしますという話にするのが良いかなと思いますが、
その他で言って欲しい事があれば公言しますので。

@xyzzy-17-638
Copy link
Owner Author

ありがとうございます!

33958cafdd05eaca76bd370f0b0f6409bb36a666 ←冒頭にこんな感じで簡単なテキストを入れておこうと思うのですが、どうでしょうか?

  • コピーライト関連はマルチフレーム版の about.lに合わせて修正しました
  • パッケージを含めた変数の存在確認/取得方法が分からなかったので、find-packageして、find-symbolして、evalて、としつこい感じになっています(^_^;;

@snmsts
Copy link

snmsts commented May 26, 2012

  • 冒頭の文章
    おきづかいありがとうございます。個人の功名心的にはとても嬉しいのですが
    今後のソースの取り扱いや今後のcontributorのコードの扱い的に微妙になりかねないので
    少なくともソースにそういう文言は入れない方が良いかと思います。
  • コピーライト関連
    特に言うことない感じですね。諒解です。
  • しつこい感じ
    特に簡便な方法は思いつかないので良いかと思いますが、netinst::_install-data_に嫌な値が入っていたときのために
    evalよりはsymbol-valueあたりにした方が安全かと思います。

ご検討下さい。
…list とか lisとかは dll-listとか ni-listとかに変えたりassocのstring=を使ったのが3個もあるのはflet使ったりした方が良いかなーとか細かく思ったことはあったけど置いておこうという気持ちが勝ってしまった感じが…

@snmsts
Copy link

snmsts commented May 26, 2012

すみませんこの場合はevalしても良かったですね。

@xyzzy-17-638
Copy link
Owner Author

アドバイスありがとうございます! 指摘に沿って、 b1e4aec ←こんな感じで修正してみました

  • コード冒頭のテキストを xyzzy/lisp 下の標準的なものに合わせました
    • コードの起源とライセンスについては、リリース時のテキストで言及してもらおうと思います
  • eval ではなく symbol-value を用いるようにしました
  • netinst::*install-data* が読み込まれていない場合は読み込みを試行するようにしました
    • ぎこちないですが…

@snmsts
Copy link

snmsts commented May 26, 2012

  • 冒頭テキスト

    完璧です。ありがとうございます。

    • リリース時

      前のが要求しているみたいに読めますね、すみません、なくて大丈夫です。

  • symbol-value
    ありがとうございます。

  • 読み込み関数
    netinstaller読み込まれている前提だったですからね。ありがとうございます。

@xyzzy-17-638
Copy link
Owner Author

Pull Request を提出して、マージされました

マルチフレーム版の次回リリース (0.2.3.9) に含まれると思います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants