文字コードをUTF-8にしたい

[meltingrabbit]

概要

文字コードをUTF-8にしたい

詳細

現在SJISの文字コードをUTF-8にしたい．
ただし，現在のISSLの環境がSJISを要求するので，できるのはそれが解消されてからか．

close条件

なったら

備考

• 関連: 改行コードについて整理する #162

[meltingrabbit]

というか，だめ文字問題あるし，早くしないとじゃん < icebox

[sksat]

実はSJISが要求される環境が特殊環境(かつ，今後は撤退していく環境)なので，もうUTF-8にしてしまって，SJIS必要な時だけビルド前にUTF-8->SJISに変換すればいい説ないですか

[meltingrabbit]

SJISが必要なの，SILSではなくて，現ISSL OBCのボードのビルド環境（＝つまり実機，つまり本ソースコードのメイン目的，つまり一番大事なところ）なんだよね．

というのがあって，変えることに踏み切れてない．

[sksat]

まあいざ現状一番大事なやつを変えるのが怖いのはありますねえ．リモートリポジトリから今の渋い環境への依存を抜こうと思うと#pragma sectionとか消したくなってくるしなあ．

[meltingrabbit]

わかる．ここらへんのコンテキストスイッチをどうすればいいのか，結構悩み．（本当はcloneしたあと，ターゲットにあわせて前処理すべき場所なきがしているし．）

[sksat]

#121 は(GitHubがこの問題を修正するか)SJISである限り発生しうる問題なのでUTF-8にしたいポイント++

[meltingrabbit]

いろいろ問題が顕著になってきたので，やるぞやるぞやるぞ！

とはいえ．．．

[meltingrabbit]

現状の問題

• ISSLの重要なOBCの開発環境がSJIS only
• remote repositoryをutf-8にして，ビルドflowでutf-8にするという方針もある
  • が，一度でもビルドすると，commitしようにも全ファイルがdiffに．．．
• local repositoryからファイルをコピーしたworking dirにてビルド作業をする？
  • デバックポイント打ちながら，修正しながらデバッグしたい．．．

[meltingrabbit]

実は，

が，一度でもビルドすると，commitしようにも全ファイルがdiffに．．．

とあるが，coreがそうでも，user側には影響ないのでは？

つまり，coreはutf-8にしてしまって，userのrepositoryはsjisにしてしまい，post checkout hook or build flowでcoreをsjisにしてしまう？

問題は， sjis→utf-8の変換が冪等ではないので，２重実行をどう防ぐか．

[meltingrabbit]

コメントが少ないファイルもあるので，そうなるとsjisの自動文字コード判定結構見するんよなぁ．．．

どこかでflag持つのか？？

[sksat]

#170 とかもありますしそこらへんに謎の文字コードフラグを押し込んでおいてもいいかもしれない(渋いがマシではある)

[meltingrabbit]

フラグ

と思ったんだけど，例えばfetchとかcheckoutして部分的なファイルだけoverwriteされたときに詰むなとおもったので，やっぱ，

対象ファイルの文字コード判定 → utf-8 or non sjis ならutf-8と仮定してconvet

するしかないのかなと思い始めた．

[sksat]

うーむそういう用途あるのかー，まああるよなー．何故かPythonみたいに# -*- coding: utf-8 -*-みたいなことやるとかか...？

[meltingrabbit]

# -*- coding: utf-8 -*-

かなり渋い．．．

[sksat]

でも文字コードよしなに判別するのはもっと渋いという...

[meltingrabbit]

core更新時は，一旦discardしてもらって，更新して，また回す，かなぁ．．．

[meltingrabbit]

あー，でもbuild workflowにいれるには冪等性が．．．

いやそれは全体フラグがあればいい．

[meltingrabbit]

build flowで，

cd ${path_to_core}
git reset --hard HEAD
utf-8への変換

にすれば勝てる気がしてきた．

@sksat どう？

[meltingrabbit]

あ〜、でもcoreちょっといじってマイコンに書き込もうとすると、変更消えちゃうのか〜。

commitしてもらえれば。。。？

[sksat]

git reset --hard HEADよりは「commitしろ！」って言ってビルド落とすか，警告だけ出してstashしておく，とかかなあ．

[meltingrabbit]

branch変更とかでutf-8のものとsjisのものが混じってるときには， git reset --hard HEAD してからsjis変換，でいいんだよね．

意図的にcoreのコードを変えてる時がつらい．

[sksat]

あっそうか意図的な変更と意図的でない変更があるのか...

[meltingrabbit]

そうそう．なので，ノミナルは git reset --hard HEAD でいい気がしていて，でも乱暴なので，戻せるように stash 積んどくのがいいかもね．

[sksat]

ですかねー．stashは積んだらdiffなくなるのでstashだけでいいのでは？あとコーナーケース踏むとめちゃくちゃstash積まれてしまうことがあったりしそうですが，まあ消えるよりはマシだしいいか．

[sksat]

あとどうせbuild flow変わるなら #82 もやっちゃいたいかも？

[meltingrabbit]

stashは積んだらdiffなくなるのでstashだけでいいのでは？

そうだったんだ．stash全く使わないので知らなかった．

[meltingrabbit]

あとどうせbuild flow変わるなら #82 もやっちゃいたいかも？

あり．（お願いしたいかも）
ひとまず #184 がマージされた後かな．

#82 ってもしかして僕のレビュー待ちだった？（何かで止まってたと思ってた）

[sksat]

レビューと歩調合わせるの待ちだった気がします．特にコード的なブロッカーは無かったはず．とりあえずconflict出てるんでrebaseしときます．

[meltingrabbit]

大PRらがマージされたんで，closeします．問題起きたらまたreopenするということで．