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

MinGW Makefileにもbregonig.dllとctags.exeのzip展開を組み込む #1663

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

タイトル通りです。
MSVCで行っているpostBuildと同等のコピー処理を入れます。
※postBuildはバッチで、MinGWにはMakefileを使うのでコピー機能は若干高級になります。

カテゴリ

  • その他の問題

PR の背景

Grepテストの導入(#1660)の作業中に、MinGWビルド完了時に正規表現DLLのコピーが行われていないことに気付きました。
※PR #329 の対応漏れです。

ビルド完了時に正規表現DLLのコピーが行われないと、
正規表現DLLに依存する機能のテストを実行できません。

PR のメリット

  • MinGWビルドで正規表現DLLに依存する機能のテストを実行できるようになります。
  • ビルド前提条件に 7zip を含めることにより、過去にあった「めんどくさい議論」を考えなくて良いようにします。

PR のデメリット (トレードオフとかあれば)

  • MinGWビルドで 7zip のインストールが「必須」になります。
  • MSVCビルドとMinGWビルドで、zip展開を別々に記述することになります。
    👉 zipの更新時には「あちこち修正する」という作業が必要になります。
  • レビューできない懸念があるので、ドキュメント更新は含めていません。

仕様・動作説明

サクラエディタの仕様・機能に影響を与える変更ではありません。

MinGWビルドの完了時に、正規表現DLLとCTagsの実行ファイルを出力ディレクトリにコピーする処理を追加し、ビルド後すぐに正規表現やタグジャンプのテストができるようにします。

PR の影響範囲

  • MinGWビルドで 7zip のインストールが「必須」になります。
  • MinGWビルドの成果物が増えます。

テスト内容

MinGWビルド実行後に bregonig.dll と ctags.exe がコピーされていることを確認します。

  1. Msys2コンソールを開く(C:\msys64\msys2.exeを実行する)
  2. Msyu2コンソールで pacman -S p7zip を実行して 7zip をインストールする。
    1, MinGW64コンソールを開く(C:\msys64\mingw64.exeを実行する)
  3. sakura-editorのgitフォルダに移動する。
  4. in-sourceビルドする。
cd /c/work/sakura-editor/sakura/sakura_core
mingw32-make

関連 issue, PR

#329
#788

参考資料

@berryzplus berryzplus force-pushed the feature/add_postbuild_for_MinGW branch from 6b9d77b to 9655fa3 Compare May 9, 2021 08:05
@AppVeyorBot
Copy link

@AppVeyorBot
Copy link

windows向けの7zではなくmsys2版のパスを記載するよう修正
@sonarcloud
Copy link

sonarcloud bot commented May 10, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。

@berryzplus berryzplus merged commit ff36bcc into sakura-editor:master May 11, 2021
@berryzplus berryzplus deleted the feature/add_postbuild_for_MinGW branch May 11, 2021 06:10
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.

None yet

3 participants