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

[WIP] mingw ビルドを appveyor に組み込むための残課題 #476

Conversation

berryzplus
Copy link
Contributor

昨日いろいろ試行錯誤した結果、見えてきた残課題を共有します。
この PR 自体は will never be merged です。
適用順を入れ替えたほうが良さそうな修正が含まれています。

共有しとかないといけない内容としては大きく2点。

  • ソースコードのUNICODE化はまだ終わっていない
  • appveyor ビルドが遅い原因(ロケールを日本語に変える)を取り除く方法がある

@KENCHjp
Copy link
Member

KENCHjp commented Sep 24, 2018

あれ、このまえ、文字コードの一覧作ったのですが、
#112 (comment)

@m-tmatma さんによると、
#112 (comment)
test-int2dec.cppだけってことだったかとおもって、 #112 は閉じたのですが、足りない??

@berryzplus
Copy link
Contributor Author

berryzplus commented Sep 24, 2018

対象はcppじゃないです。 >> funccode.hsrc

また、プロジェクト設定で shiftjis を宣言しているのを見落としていました。
HeaderMake.cppにベタ書きされたコマンドオプションの修正も必要です。

コミットは分けてあるので、「こんな感じの修正」はコミット別差分で見れると思います。

@KENCHjp
Copy link
Member

KENCHjp commented Sep 24, 2018

対象はcppじゃないです。 >> funccode.hrsc

了解です。私の作ったリストは、全ファイル対象にしてますので、もし参考になれば。

@beru beru added CI appveyor など CI 関連 【ChangeLog除外】 MinGW MinGW labels Sep 25, 2018
@berryzplus
Copy link
Contributor Author

この PR 自体は will never be merged です。

と言いつつ、構成物の大半がマージされました・・・ 😄
重要な変更のうち残っているのは次の2つ。
もうそろそろ閉じる感じです。

タイトル コミット 対応予定
MinGWビルドをappveyorに組み込む 790b81e 単独で入れたい
MsBuildのバグに対処する a4678ae あえて放置

単独で入れるPRはそのうち出す感じです。

MsBuildの~のほうは、放置します。

放置してたら vs2019 で対応されんかな~という期待をこめて。
sakura-editor/management-forum#24

いろんな見方あると思いますがパラメータ設計の不具合と見てます。
対象がcl.exeなんで日本で騒いでもどうにもならん気がします。
VC++ MVPの人とか日本MSの人から伝えてもらえば対応が叶うんじゃないかと思っとります。

@berryzplus
Copy link
Contributor Author

#494 の PR に役割を引き継いだので閉じます。

@berryzplus berryzplus closed this Sep 29, 2018
@berryzplus berryzplus deleted the feature/enable_mingw_build_in_appveyor branch September 29, 2018 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】 MinGW MinGW
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants