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

SourceForge の htmlhelp2 を GitHub Pages に移行 #22

Merged
merged 6 commits into from
Jun 17, 2018

Conversation

kobake
Copy link
Member

@kobake kobake commented Jun 16, 2018

概要

SourceForge 上にホストされていた htmlhelp2 (http://sakura-editor.sourceforge.net/htmlhelp2/) を
GitHub Pages 側 (https://sakura-editor.github.io/help/) に移行する。

移行理由

SourceForge の Web サーバがしばしば重いor不安定であるという声が多いため、SourceForge 上の Web リソースを順次 GitHub Pages に移行していく。今回は htmlhelp2 を移行。

見た目のレビュー

ここで見れるようにしてあります。
https://kobake.github.io/sakura-editor.github.io/help/

SourceForge Web 領域のバックアップ置き場

量が多いので全てを配置しているわけではないですが、SourceForge の Web 領域の一部のバックアップ(今のところは htmlhelp, htmlhelp2 のみ)を以下に配置しました。

注記

  • sakura リポジトリ側の .chm ヘルプのリソースに同内容のファイル群がある (https://github.com/sakura-editor/sakura/tree/master/help/sakura/_RESOURCE) ため、そこへの参照方式 (submodule 等) にする選択もありましたが、まだ sakura リポジトリ側の help のファイル構成が変わる可能性を考え、今回は参照方式をとっていません。
  • ファイルのコピー元は上記ファイル群ではなく http://sakura-editor.sourceforge.net/htmlhelp2/ に配置されていたファイル群です。
  • https://github.com/sakura-editor/sakura/tree/master/help/sakura/_RESOURCEhttp://sakura-editor.sourceforge.net/htmlhelp2/ のファイル群の内容を WinMerge で比較してみましたが、差異は一箇所 (※) 以外無かったです。
    • ※その一箇所とは 6f12504 における HLP000075.html の1行です。実装内容から sakura リポジトリ側の記載が正しいと判断し、そのように修正しました。詳細は 6f12504 のコミットメッセージを参照。
  • GitHub Pages では .htaccess が使えないため、削除してあります。
  • SourceForge での htmlhelp2 では .htaccess による DirectoryIndex で HLP000001.html が指定されていましたが、GitHub Pages では .htaccess が使えないため、代わりに HLP000001.html から index.html へシンボリックリンクを張る形で DirectoryIndex と同等の挙動を果たすようにしました。
  • GitHub Pages ではファイルのエンコーディングを UTF-8 にしておかないと文字化けするため、エンコーディングを Shift_JIS から UTF-8 に変更しました(Jekyll を使う場合はエンコーディング指定もできるようだが、今の時点では Jekyll は使っていないため、エンコーディング変更の対処とした)。

関連 Issue

事後の予定

  • サクラエディタから HTML ヘルプを開くときの URL を GitHub Pages 側に変更する。
  • SourceForge 側の htmlhelp2 へのアクセスは GitHub Pages のほうへリダイレクトするように設定する。

検討保留事項

  • htmlhelp2 は移行するが、htmlhtlp (ANSI版ヘルプ) をどうするかについては今のところは検討を保留する。

@kobake
Copy link
Member Author

kobake commented Jun 16, 2018

@sakura-editor/sakura-developers
レビュー&問題なければマージお願いします。

@kobake kobake mentioned this pull request Jun 16, 2018
3 tasks
@KageShiron
Copy link
Member

  • シンボリックリンクはおそらくindex.htmlとHLP000001.htmlが独立して存在してしまい、SEOやブックマークで混乱しそう。meta refreshにした方が良さそうです。
  • chmと2重メンテになるのは避けたいところですね・・・。イマイチchmのコンパイル方法がわかってないんですが、ヘルプましんとかHTML Help Workshopでコンパイルするんでしょうか・・・?
  • マクロやプラグインヘルプも別々に配らず、統合してしまっても良い気がします
  • sourceforge.netとかになってるリンクは後で修正してpull request投げてみたいと思います。

@kobake
Copy link
Member Author

kobake commented Jun 16, 2018

シンボリックリンクはおそらくindex.htmlとHLP000001.htmlが独立して存在してしまい、SEOやブックマークで混乱しそう。meta refreshにした方が良さそうです。

元々のヘルプページでも DirectoryIndex の機能によって
http://sakura-editor.sourceforge.net/htmlhelp2/
http://sakura-editor.sourceforge.net/htmlhelp2/HLP000001.html
の両者が同じものを指していたので問題ないんじゃないかな、と思っていました。

内部的に DirectoryIndex を用いているかシンボリックリンクを用いているかは閲覧側(エンドユーザおよび検索エンジン)から見ると区別できない=別名の URL があること自体は構わないと思っているのですがどうでしょう。

meta refresh するとしたら
index.html アクセス時に HLP000001.html に飛ばすか
HLP000001.html アクセス時に index.html (というか / かな) に飛ばすか
だと思いますが、やるとしたら後者、、ですかね。
/ をブックマークしたい人のほうが多いと思うので。

chmと2重メンテになるのは避けたいところですね・・・。

そですね。今回の対応はあくまでも不安定な SourceForge から暫定で避難することを目的としていて、今後のメンテナンスの仕方については避難後にじっくり考えたいです。

イマイチchmのコンパイル方法がわかってないんですが、ヘルプましんとかHTML Help Workshopでコンパイルするんでしょうか・・・?

これは・・・僕もわかってません!
おそらくですけど現状のアクティブなメンバーの中でわかってる人いないのでは・・・と思っています。なので誰かががんばって調べる or SourceForge時代のメンテナンス担当の人の降臨を待つ、という感じで考えてます。

マクロやプラグインヘルプも別々に配らず、統合してしまっても良い気がします

僕もそう思っています。

ただ、現状ヘルプファイルのコンパイルの仕方さえ分からないところなので、そのあたりの情報整理できてからマクロとプラグインのファイル群の設置方式(ディレクトリ構成とか)を考えたいところです。Web 上への公開についてはそれらの整理が済んでからで良いかな、と考えています。

sourceforge.netとかになってるリンクは後で修正してpull request投げてみたいと思います。

ありがとうございます~

@kobake
Copy link
Member Author

kobake commented Jun 16, 2018

いろいろ改善の検討事項はあるのですが、現状 SourceForge がすごく不安定(今もメンテに入ってしまって Web が見れない)なので、まずはこの PR で避難だけ済ませてから別件として細かい調整を検討したいです。

Copy link
Member

@KageShiron KageShiron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いくつかのページを見た限りでは文字化けなどの問題はなさそうです。

@KageShiron KageShiron merged commit 57a6e74 into sakura-editor:master Jun 17, 2018
@kobake kobake deleted the help branch June 17, 2018 23:54
@kobake
Copy link
Member Author

kobake commented Jun 18, 2018

レビュー&マージありがとうございます!

ちょっとバタバタしてして確認が今になってしまいましたが、マージにより
https://sakura-editor.github.io/help/
が見れるようになったことが確認できました。

m-tmatma added a commit to m-tmatma/sakura-editor.github.io that referenced this pull request Dec 7, 2019
e5172cb526f26ad18ec5754282886b6270459588 時点での sakura 本体のヘルプを正にして
文字コード変換による問題のみを取り込んで修正する。
m-tmatma added a commit that referenced this pull request Dec 7, 2019
PR #22 の際の文字コード変換時の誤りを修正
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants