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

英語DLLがMinGWビルドできなくなっていたのを修正 #1667

Conversation

berryzplus
Copy link
Contributor

@berryzplus berryzplus commented May 10, 2021

PR の目的

一応タイトル通りなんですが、
本当に英語DLLのビルドができなくなっていたのか?
という点にはやや疑いがあります。
(理由:最近のMinGWビルドが失敗しているわけではないから。)

カテゴリ

  • ビルド関連
    • ローカルビルド

PR の背景

リソースのout-of-sourcesビルドの修復(#1662)を行った際に、英語DLLのリソースビルドがリソーススクリプトのUTF16化(#1456)に対応していないことに気付きました。

この件、現状で問題にはなっていませんが、キモいので対応しておきたいと思います。

PR のメリット

  • 英語以外の言語リソースを作るときに、英語版をコピって作れるようになります。

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

  • 現状、英語版DLLのビルドでエラーは検出されていないので、見方によっては「無駄な対応」です。
  • 現状、英語版DLLのビルドで異常を検出できないビルドバッチの不具合については、対処方法が分からないので放置します。

仕様・動作説明

サクラエディタの仕様・機能を変更するPRではありません。

リソーススクリプトのUTF16化(#1456)によって、
すべてのリソーススクリプト(*.rc, *.rc2)はUTF16LEでチェックアウトされます。

sakura/.gitattributes

Lines 7 to 8 in 6cc86ab

*.rc text working-tree-encoding=utf-16le-bom eol=crlf
*.rc2 text working-tree-encoding=utf-16le-bom eol=crlf

MinGWのリソースコンパイラwindresはUTF16LEに対応していないので、
sakura_core/Makefileでやっているようにビルド前にUTF-8に変換してやる必要があります。
なので、sakura_core/Makefileでやっている処理をそのままコピーすることによって対策します。

PR の影響範囲

  • MinGW版の英語DLLのビルドに影響する変更です。

テスト内容

ローカルで言語DLLのMinGWビルドができるようになっていることを確認します。

1, MinGW64コンソールを開く(C:\msys64\mingw64.exeを実行する)
2. sakura-editorのgitフォルダに移動する。
3. in-sourceビルドする。

cd /c/work/sakura-editor/sakura/sakura_lang_en_US
mingw32-make

関連 issue, PR

#1662
#1660
#1456

参考資料

@sonarqubecloud
Copy link

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 35b176f into sakura-editor:master May 11, 2021
@berryzplus berryzplus deleted the feature/fix_mingw_lang_resource_build branch May 11, 2021 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants