c2a-core icon indicating copy to clipboard operation
c2a-core copied to clipboard

改行コードについて整理する

Open meltingrabbit opened this issue 3 years ago • 4 comments

概要

コーディング規約に

https://github.com/ut-issl/c2a-core/blob/94a4007362ae5192bfead701672391a0323e1a79/Docs/General/CodingRule.md#L135

とあるが,現状これはWindows環境でcloneすると auto crlfがかかってるだけなので,repository上ではLFになっている気がする.

機械的に統一/チェックをする,もしくはコーディング規約を変えるのか,といったことを整理したい.

詳細

  • 現状のコーディング規約ではCR+LFに統一することを要求している
  • この規約は機能/強制しておらず,現状のリモートリポジトリのソースコードのファイルの改行コードはLFになっている
  • CR+LFは特定の開発環境(Renesas)から来ている制約であり,本当に要求されるのはここでビルドする際にCR+LFになっていることのみ
  • Gitの設定でcore.autocrlf=trueになっていれば作業時にはCR+LFになる
  • Git for Windowsのデフォルト設定がcore.autocrlf=trueであるため,なんとかなってはいる
  • 改行コードを何にするにしても,これは各自の設定任せではなく.gitattributesで設定すべき

関連

  • https://github.com/ut-issl/c2a-core/issues/21

close条件

整理できたら

meltingrabbit avatar Jan 04 '22 05:01 meltingrabbit

現状もリモートリポジトリ全部LFっぽいですし,規約の方を変えて.gitattributes追加だけでよさそう?

sksat avatar Jan 04 '22 09:01 sksat

その説が濃厚だと思ってる.

meltingrabbit avatar Jan 04 '22 09:01 meltingrabbit

これちゃっちゃとやっちゃうか.next core update欄に移動した.

meltingrabbit avatar Jan 04 '22 12:01 meltingrabbit

適当に関係者と話して,この方向で行くことにした

meltingrabbit avatar Jan 06 '22 00:01 meltingrabbit