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

Universal ctags (2018-09-16 の daily build) を組み込む #454

Merged
merged 5 commits into from
Sep 18, 2018

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Sep 17, 2018

#450: Universal ctags を組み込む

#451 の PR 用のブランチを別名に名前変更して、rebase でコミットを
修正して Universal ctags 用のブランチを作りました。

インストーラでライセンスファイルをコピーするときに license\ctags というフォルダに
入れるようにしました。bregonig.dll のライセンスファイルにも水平展開可能だと思います。

beru
beru previously approved these changes Sep 17, 2018
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

動作確認してみました。問題無いと思います。

インストール後にすぐに tags ファイル作成してタグジャンプ出来るのは楽で良いですね。別途入れる必要が無いので。

少し気になったのはヘルプの記載です。
HLP000278.html
HLP000261.html

ただ別PRで対応するのでも良いと思います。

@m-tmatma
Copy link
Member Author

インストール後にすぐに tags ファイル作成してタグジャンプ出来るのは楽で良いですね。別途入れる必要が無いので。

少し気になったのはヘルプの記載です。
HLP000278.html
HLP000261.html

ただ別PRで対応するのでも良いと思います。

#458 で issue 作成しました。

他の人から突っ込み入るかもしれないので、1日ほどマージしないでおいておきます。

@m-tmatma
Copy link
Member Author

#455 のマージでコンフリクトしたので解消しました。

@m-tmatma m-tmatma changed the title Universal ctags を組み込む Universal ctags (2018-09-16 の daily build) を組み込む Sep 17, 2018
beru
beru previously approved these changes Sep 17, 2018
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

再確認して特に問題はありませんでした。

build-all.bat 動かして cppcheck.exe の実行で結構待たされるな~と思いましたがそれはまた別問題ですね。bgプロセスで並行的に出来ると良いのかもと思いつきました。

@m-tmatma m-tmatma added the document ドキュメント label Sep 17, 2018
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

ba2da72 でインストーラー再作成してタグファイルの作成やタグジャンプが出来る事を確認しました。ヘルプについても更新が確認出来ました。

@KENCHjp KENCHjp added installer installer 関連 license ライセンス関連 document ドキュメント specification change ■仕様変更 labels Sep 18, 2018
@m-tmatma m-tmatma merged commit 21fcf4e into sakura-editor:master Sep 18, 2018
@m-tmatma m-tmatma deleted the feature/Universal-ctags branch September 18, 2018 11:25
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…ctags

Universal ctags (2018-09-16 の daily build) を組み込む
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント installer installer 関連 license ライセンス関連 specification change ■仕様変更
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants