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

sakura-editor/sakura/installer/readme.txt を修正したいと思い立ったが・・・ #328

Closed
KENCHjp opened this issue Aug 11, 2018 · 14 comments
Labels
document ドキュメント
Milestone

Comments

@KENCHjp
Copy link
Member

KENCHjp commented Aug 11, 2018

https://github.com/sakura-editor/sakura/blob/master/installer/readme.txt

の中身を現状に添わせたい。

GitHubだと、
https://github.com/sakura-editor/sakura/tree/master/installer
ここにも表示される。

基本的には、
#67 (comment)
ここで行われているようなことをやればいいと思っている。

readme.txtを見ると、

set_time.js SVNからチェックアウト後、キーワードファイルの更新日時を復元する

こんな事が書いてある。でもそのファイルが見当たらない???

ふと、installer作ろうって人はきっとGitHubは見れる人だし、
あえてreadme.txtっていらんのではないだろうかと思い始める。

https://github.com/sakura-editor/sakura#ci-build-appveyor

ここあたりに追記して、readme.txtは削除してはいかがでしょう?

@berryzplus
Copy link
Contributor

https://github.com/sakura-editor/sakura/blob/master/installer/readme.txt

の中身を現状に添わせたい。

いいと思います。たぶん、拡張子を.mdに変えるだけで markdown が使えるようになるので、今までより見やすくできる気がしています。
https://github.com/sakura-editor/sakura/blob/master/unittest.md

ふと考える、installer作ろうって人はきっとGitHubは見れる人だし、
あえてreadme.txtっていらんのではないだろうかと思い始める。

https://github.com/sakura-editor/sakura#ci-build-appveyor
ここあたりに追記して、readme.txtは削除してはいかがでしょう?

詳細な話題は個別ファイルに書かれているのがベターだと思っています。
https://github.com/sakura-editor/sakura の実体は https://github.com/sakura-editor/sakura/blob/master/README.md なので、同じように sakura/installer/README.md を作ればいいような気がします。もちろん、topにリンクを加えるのには賛成です。

  1. リネームして.md化
  2. 内容を .md に合わせる
  3. 内容を最新化する (#67の内容はここで入る)
  4. sakura/README.md → sakura/installer/README.md のリンクを貼る

2.までは速攻でできそう。3はちょっと大変ってところですかねぇ。

@KENCHjp
Copy link
Member Author

KENCHjp commented Aug 11, 2018

同じように sakura/installer/README.md を作ればいいような気がします。もちろん、topにリンクを加えるのには賛成です。

なるほど、それはナイスアイデアですね。

@m-tmatma
Copy link
Member

基本的には、
#67 (comment)
ここで行われているようなことをやればいいと思っている。

インストーラを作成するには build-all.bat を呼ぶか
build-all.bat で呼ばれているバッチファイルで build-installer.bat の部分までを順番に呼べば作成できます。(ローカルでビルドするときは zipArtifacts.bat は呼ぶ必要はない。呼んでもいいけど)

appveyor 対応するために全自動化したのでインストーラの作成自体はすごく簡単になっています。

リネームして.md化
内容を .md に合わせる
内容を最新化する (#67の内容はここで入る)
sakura/README.md → sakura/installer/README.md のリンクを貼る

やりましょうか?

@KENCHjp
Copy link
Member Author

KENCHjp commented Aug 11, 2018

記載内容

  1. ローカルでインストーラ作成テストをする方法について記載する
    正式リリースは、appveyorで作成されるものにしてもいいのかなと。
  2. innosetupを用意する(innosetup-5.5.9-unicode.exe か innosetup-5.6.1-unicode.exe)
  3. バイナリは、https://ci.appveyor.com/project/sakuraeditor/sakura からzipをダウンロードする。
  4. 3.でダウンロードしたinstaller作成対象bit(32bit/64bit)用のバイナリを抽出
  5. installer\sakura フォルダを作ってinstaller\sinst_srcの中身と、4. で抽出したバイナリをほおりこむ。
  6. 対象のinstallerを作成

こんな感じかな。

@KENCHjp
Copy link
Member Author

KENCHjp commented Aug 11, 2018

やりましょうか?

あ、お願いしたいっす。

@m-tmatma
Copy link
Member

対応はたぶん、8/13 ぐらいの予定です。

@m-tmatma
Copy link
Member

書く内容はインストーラに限定した方がいいですか?

それともビルド全体に対して書いて、その中の章で
インストーラに関して書いた方がいいですか?

appyeyor の説明にも内容的にかぶる部分があって
どうしようかな?と思ってます。

@KENCHjp
Copy link
Member Author

KENCHjp commented Aug 11, 2018

@m-tmatma さん
ビルド全体を書くとappyeyor とドド被りな気がしております。
私が欲しかったのは、ローカルでインストーラーを作成する手順というよりは、
.issを修正するための環境を最小の手数で作る手順かと。
最終的に書きやすい方でお任せいたします。

@m-tmatma
Copy link
Member

#334 を作りました。

@m-tmatma
Copy link
Member

#337 でドキュメントを更新しました。

@m-tmatma m-tmatma added this to the next release milestone Aug 13, 2018
@m-tmatma m-tmatma added the document ドキュメント label Aug 13, 2018
@m-tmatma
Copy link
Member

#337 をマージ

@KENCHjp
Copy link
Member Author

KENCHjp commented Aug 13, 2018

すいません、Ok.って言ったのですが、
readme.md
73行目(ごめんなさいリンクで引用できなかった)

installer\sakura\keyword*.kwd

拡張子.kwd以外も入るのではなかろうかと。

@m-tmatma
Copy link
Member

拡張子.kwd以外も入るのではなかろうかと。

#340 で対応しました。

@m-tmatma m-tmatma reopened this Aug 13, 2018
@m-tmatma
Copy link
Member

#340 をマージした

@ds14050 ds14050 added the document ドキュメント label Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント
Projects
None yet
Development

No branches or pull requests

4 participants