[インデックス 1764] ファイルの概要
このコミットは、Goコンパイラ(gc
)におけるsysimport.c
(後にbuiltin.c
に改名)の生成方法を根本的に変更するものです。以前は自動生成されたファイルをバージョン管理システム(Perforce)にもコミットしていましたが、この運用がユーザーにとって不便であったため、生成プロセスを改善し、ユーザーのビルド体験を向上させることを目的としています。
コミット
commit af678a593da3e2bc942aa3bf41e66bd50e0b3bab
Author: Russ Cox <rsc@golang.org>
Date: Thu Mar 5 18:26:12 2009 -0800
new approach for generating sysimport.c
(renamed to avoid any conflict with old p4 copies).
this approach doesn't require auto-generating
files also kept in p4, so it should be easier on
go users who don't sync very often.
this approach will be more work for go developers:
builtin.c needs to be copied to builtin.c.boot in p4
as new functions are added. mkbuiltin does this
for certain $USERs to help us remember.
R=r
DELTA=343 (176 added, 162 deleted, 5 changed)
OCL=25803
CL=25805
GitHub上でのコミットページへのリンク
https://github.com/golang/go/commit/af678a593da3e2bc942aa3bf41e66bd50e0b3bab
元コミット内容
sysimport.c
を生成するための新しいアプローチ(古いP4コピーとの競合を避けるために名前を変更)。
このアプローチは、P4にも保持されている自動生成ファイルを不要にするため、頻繁に同期しないGoユーザーにとってはより簡単になるはずです。
このアプローチはGo開発者にとってはより多くの作業を伴います。新しい関数が追加されるたびに、builtin.c
をP4内のbuiltin.c.boot
にコピーする必要があります。mkbuiltin
は、特定の$USER
に対してこれを実行し、開発者が忘れないようにします。
変更の背景
Go言語の初期段階において、コンパイラが使用する組み込み関数や型に関する情報を含むC言語のソースファイル(sysimport.c
)は、sys.go
やunsafe.go
といったGoのソースファイルから自動生成されていました。しかし、この自動生成されたファイルが同時にバージョン管理システム(当時はPerforce、通称P4)にもコミットされていたため、以下のような問題が発生していました。
- 同期の問題: ユーザーがGoのソースコードをP4から取得する際、
sysimport.c
が古いバージョンである場合がありました。この古いファイルが原因でビルドエラーが発生したり、予期せぬ動作を引き起こしたりすることがありました。特に、Goのソースコードを頻繁に同期しないユーザーにとっては、この問題は顕著でした。 - 開発者の負担: 自動生成されたファイルが変更された場合、開発者は手動でP4上でファイルを「開いて編集可能にする」(
p4 open
)必要があり、その後、新しい内容で上書きしてコミットするという手間がありました。これは、自動生成されるべきファイルが手動のバージョン管理プロセスに組み込まれていることによる非効率性でした。 - ビルドの不安定性:
Makefile
のルールが複雑で、sysimport.c
の生成ロジックがP4の操作と密接に結びついていたため、ビルド環境やP4の状況によってはビルドが不安定になる可能性がありました。
このコミットは、これらの問題を解決し、Goのビルドプロセスをより堅牢でユーザーフレンドリーにすることを目的としています。
前提知識の解説
このコミットの変更内容を理解するためには、以下の前提知識が必要です。
- Goコンパイラ(
gc
): Go言語の公式コンパイラであり、Goのソースコードを機械語に変換します。src/cmd/gc
ディレクトリにそのソースコードがあります。 - 組み込み関数と型(Built-ins): プログラミング言語において、特別な宣言なしに利用できる基本的な関数や型のことです。Go言語にも
len
、cap
、make
、new
などの組み込み関数や、int
、string
などの組み込み型が存在します。コンパイラはこれらの組み込み要素に関する情報を内部的に持っている必要があります。 sys.go
とunsafe.go
: Goの標準ライブラリの一部であり、Goランタイムやコンパイラが内部的に利用する低レベルな機能や、型安全性をバイパスする機能を提供します。これらのファイルには、Goの組み込み関数や型に関する情報が含まれており、コンパイラがC言語のコードを生成する際の元データとなります。6g
コンパイラ: コミット当時のGoコンパイラの名称の一つで、64ビットシステム向けのGoコンパイラを指します。Goのソースコード(.go
ファイル)をコンパイルして、中間ファイル(.6
ファイル)を生成します。.6
ファイル:6g
コンパイラによってGoのソースコードから生成される中間ファイルです。Goの抽象構文木(AST)や型情報などが含まれており、リンカや他のツールが利用します。- Perforce (P4): コミット当時、Goプロジェクトが利用していた集中型バージョン管理システムです。P4は、ファイルを編集する前に明示的に「開く」(
p4 open
)必要があるという特徴があり、自動生成されるファイルをバージョン管理下に置く場合に、開発者に余分な手間を強いることがありました。 - Makefile: ビルドプロセスを自動化するためのツールです。ソースファイルのコンパイル順序や依存関係を定義し、コマンドを実行します。
技術的詳細
このコミットの核心は、sysimport.c
(新名称builtin.c
)の生成とバージョン管理の分離です。
旧アプローチの問題点と解決策:
旧アプローチでは、src/cmd/gc/sysimport.c
はmksys.c
(現在のmkbuiltin1.c
)によってsys.go
とunsafe.go
から生成され、その上でPerforceにコミットされていました。Makefile
には、sysimport.c
がsys.go
、unsafe.go
、mksys.c
よりも古い場合に再生成を試みるルールがありましたが、このルールはp4 open sysimport.c
を含んでおり、Perforceのワークフローと衝突していました。ユーザーが古いP4同期を持っている場合、sysimport.c
が古いままでビルドが失敗する可能性がありました。
新アプローチでは、この問題を解決するために以下の変更が導入されました。
- ファイル名の変更:
sysimport.c
はbuiltin.c
に、mksys.c
はmkbuiltin1.c
にそれぞれ改名されました。これは、古いPerforceのコピーとの競合を避け、新しいアプローチを明確にするためです。 builtin.c.boot
の導入:sysimport.c
がbuiltin.c.boot
としてPerforceにコミットされるようになりました。このファイルは「ブートストラップコピー」として機能し、builtin.c
の生成が何らかの理由で失敗した場合のフォールバックとして使用されます。これにより、ユーザーは常にビルド可能な状態を保つことができます。mkbuiltin
スクリプトの導入:builtin.c
の生成は、新しく導入されたシェルスクリプトsrc/cmd/gc/mkbuiltin
によって行われるようになりました。このスクリプトは以下の手順を実行します。mkbuiltin1.c
をgcc
でコンパイルして実行可能ファイルmkbuiltin1
を生成します。sys.go
とunsafe.go
を6g
コンパイラでコンパイルし、中間ファイル(.6
ファイル)を生成します。mkbuiltin1
を実行し、生成された.6
ファイルから組み込み関数の情報を抽出し、C言語のコードとして_builtin.c
に出力します。- 開発者向けの自動更新:
mkbuiltin
スクリプトには、特定のユーザー(ken
,r
,rsc
など、Goの主要開発者)が実行した場合にのみ有効になる特別なロジックが含まれています。このロジックは、新しく生成された_builtin.c
がbuiltin.c.boot
と異なる場合、自動的にbuiltin.c.boot
をPerforce上で「開いて編集可能にし」(p4 open
)、_builtin.c
の内容で上書きします。これにより、builtin.c.boot
は常に最新のbuiltin.c
の内容を反映するようになり、開発者の手動での更新作業を軽減します。 - 最終的に、
_builtin.c
の内容を標準出力に出力し、_builtin.c
を削除します。
Makefile
の簡素化:Makefile
のbuiltin.c
生成ルールは大幅に簡素化されました。builtin.c:
の依存関係がsys.go unsafe.go mkbuiltin1.c mkbuiltin
に変更されました。- 生成コマンドは
mkbuiltin >builtin.c || (echo 'mkbuiltin failed; using bootstrap copy of builtin.c'; cp builtin.c.boot builtin.c)
となりました。これは、mkbuiltin
スクリプトを実行してbuiltin.c
を生成し、もしmkbuiltin
が失敗した場合は、builtin.c.boot
をフォールバックとして使用するという、より堅牢なロジックです。
この新しいアプローチにより、一般のGoユーザーはbuiltin.c
が常にビルド時に最新の状態で生成されるため、Perforceの同期状態に起因する問題を回避できます。一方、Goのコア開発者は、builtin.c.boot
の更新がmkbuiltin
スクリプトによって半自動化されるため、手動でのバージョン管理の負担が軽減されます。
コアとなるコードの変更箇所
このコミットにおける主要なコード変更は以下のファイルに集中しています。
src/cmd/gc/Makefile
:OFILES
リストからsysimport.$O
が削除され、builtin.$O
が追加されました。sysimport.c
を生成する複雑なルールが削除されました。- 新しく
builtin.c
を生成するための簡潔なルールが追加されました。このルールはmkbuiltin
スクリプトの実行と、失敗時のbuiltin.c.boot
へのフォールバックを含みます。 clean
ターゲットが更新され、mkbuiltin1
や_builtin.c
などの新しい生成物も削除されるようになりました。
src/cmd/gc/{sysimport.c => builtin.c.boot}
:sysimport.c
がbuiltin.c.boot
にファイル名が変更されました。これは、旧来のsysimport.c
が新しいブートストラップファイルとして機能することを示します。
src/cmd/gc/mkbuiltin
:- 新しく追加されたシェルスクリプトです。
builtin.c
を生成する主要なロジックを含みます。 mkbuiltin1.c
のコンパイル、sys.go
とunsafe.go
のコンパイル、mkbuiltin1
の実行、そして特定のユーザーに対するbuiltin.c.boot
の自動更新ロジックが含まれています。
- 新しく追加されたシェルスクリプトです。
src/cmd/gc/{mksys.c => mkbuiltin1.c}
:mksys.c
がmkbuiltin1.c
にファイル名が変更されました。- コメントと使用法メッセージが更新され、その役割がより明確になりました。
コアとなるコードの解説
src/cmd/gc/Makefile
の変更
--- a/src/cmd/gc/Makefile
+++ b/src/cmd/gc/Makefile
@@ -26,7 +26,7 @@ OFILES=\
mparith1.$O\
mparith2.$O\
mparith3.$O\
- sysimport.$O\
+ builtin.$O\
compat.$O\
$(LIB): $(OFILES)
@@ -40,25 +40,11 @@ y.tab.h: $(YFILES)\
y.tab.c: y.tab.h
test -f y.tab.c && touch y.tab.c
-# the test here checks whether we have 6g at all.
-# if so, use it. if not, just use the sysimport.c we have.
-# this happens on fresh perforce checkouts where
-# sysimport.c ends up with an older time stamp
-# than sys.go (or unsafe.go or mksys.c).
-sysimport.c: sys.go unsafe.go mksys.c
- if test -x $(BIN)/6g; then \
- p4 open sysimport.c; \
- gcc -o mksys mksys.c; \
- 6g sys.go; \
- 6g unsafe.go; \
- ./mksys sys >_sysimport.c && \
- ./mksys unsafe >>_sysimport.c && \
- mv _sysimport.c sysimport.c; \
- elif test -f sysimport.c; then \
- touch sysimport.c; \
- fi
+builtin.c: sys.go unsafe.go mkbuiltin1.c mkbuiltin
+ mkbuiltin >builtin.c || \
+ (echo 'mkbuiltin failed; using bootstrap copy of builtin.c'; cp builtin.c.boot builtin.c)
clean:
-\trm -f $(OFILES) *.6 enam.c 6.out a.out y.tab.h y.tab.c $(LIB) _sysimport.c
+\trm -f $(OFILES) *.6 enam.c 6.out a.out y.tab.h y.tab.c $(LIB) mkbuiltin1 builtin.c _builtin.c
install: $(LIB)
OFILES
の変更は、コンパイル対象のオブジェクトファイルがsysimport.o
からbuiltin.o
に変わったことを示します。- 最も重要な変更は
sysimport.c
の生成ルールが削除され、builtin.c
の新しいルールが追加された点です。- 旧ルールは、
6g
コンパイラの存在チェック、p4 open
、mksys.c
のコンパイルと実行、sys.go
とunsafe.go
のコンパイルなど、多くのステップを含んでいました。これは、sysimport.c
がバージョン管理下にありながら自動生成されるという矛盾を解決しようとする試みでした。 - 新ルール
builtin.c: sys.go unsafe.go mkbuiltin1.c mkbuiltin
は、builtin.c
がsys.go
、unsafe.go
、mkbuiltin1.c
、そして新しく追加されたmkbuiltin
スクリプトに依存することを示します。 - コマンド
mkbuiltin >builtin.c || (echo 'mkbuiltin failed; using bootstrap copy of builtin.c'; cp builtin.c.boot builtin.c)
は、mkbuiltin
スクリプトの出力をbuiltin.c
にリダイレクトします。もしmkbuiltin
の実行が失敗した場合(||
)、エラーメッセージを出力し、builtin.c.boot
(ブートストラップコピー)をbuiltin.c
にコピーしてフォールバックします。これにより、ビルドの堅牢性が向上します。
- 旧ルールは、
src/cmd/gc/mkbuiltin
の追加
--- /dev/null
+++ b/src/cmd/gc/mkbuiltin
@@ -0,0 +1,31 @@
+#!/bin/sh
+# Copyright 2009 The Go Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style
+# license that can be found in the LICENSE file.
+
+set -e
+gcc -o mkbuiltin1 mkbuiltin1.c
+6g sys.go
+6g unsafe.go
+rm -f _builtin.c
+./mkbuiltin1 sys >_builtin.c
+./mkbuiltin1 unsafe >>_builtin.c
+
+# If _builtin.c has changed vs builtin.c.boot,
+# check in the new change if being run by
+# one of the people who tends to work on
+# the compiler. This makes sure that changes
+# don't get forgotten, without causing problems
+# in end user Go repositories.
+case "$USER" in
+ken | r | rsc)
+ if ! cmp _builtin.c builtin.c.boot
+ then
+ p4 open builtin.c.boot
+ cp _builtin.c builtin.c.boot
+ fi
+esac
+
+cat _builtin.c
+rm -f _builtin.c
+```
* このシェルスクリプトは、`builtin.c`を生成するための中心的なロジックをカプセル化しています。
* `gcc -o mkbuiltin1 mkbuiltin1.c`: `mkbuiltin1.c`(旧`mksys.c`)をコンパイルして実行可能ファイル`mkbuiltin1`を生成します。
* `6g sys.go`と`6g unsafe.go`: `sys.go`と`unsafe.go`をGoコンパイラでコンパイルし、それぞれ`sys.6`と`unsafe.6`という中間ファイルを生成します。
* `./mkbuiltin1 sys >_builtin.c`と`./mkbuiltin1 unsafe >>_builtin.c`: `mkbuiltin1`を実行し、`sys.6`と`unsafe.6`から組み込み関数の情報を抽出し、C言語のコードとして`_builtin.c`に書き込みます。
* `case "$USER" in ... esac`ブロック: これが開発者向けの自動更新ロジックです。
* `ken | r | rsc)`: 環境変数`USER`が`ken`、`r`、`rsc`のいずれかである場合にのみ、このブロック内の処理が実行されます。これらはGoの主要開発者のユーザー名です。
* `if ! cmp _builtin.c builtin.c.boot`: 新しく生成された`_builtin.c`と、バージョン管理されているブートストラップファイル`builtin.c.boot`を比較します。両者が異なる場合、つまり`builtin.c`の内容が更新された場合にのみ、以下の処理が実行されます。
* `p4 open builtin.c.boot`: Perforce上で`builtin.c.boot`を編集可能状態にします。
* `cp _builtin.c builtin.c.boot`: 新しい`_builtin.c`の内容で`builtin.c.boot`を上書きします。これにより、`builtin.c.boot`が最新の状態に保たれます。
* `cat _builtin.c`: 生成された`_builtin.c`の内容を標準出力に出力します。`Makefile`のルールによって、この出力が`builtin.c`ファイルにリダイレクトされます。
* `rm -f _builtin.c`: 一時ファイルである`_builtin.c`を削除します。
### `src/cmd/gc/{mksys.c => mkbuiltin1.c}`の変更
```diff
--- a/src/cmd/gc/mksys.c
+++ b/src/cmd/gc/mkbuiltin1.c
@@ -1,8 +1,8 @@
-// Copyright 2009 The Go Authors. All rights reserved.\
+// Copyright 2009 The Go Authors. All rights reserved.\
// Use of this source code is governed by a BSD-style\
// license that can be found in the LICENSE file.\
\
-// Extract import data from sys.6 and generate C string version.\
+// Compile .go file, import data from .6 file, and generate C string version.\
\
#include <stdio.h>\
#include <stdlib.h>\
@@ -18,7 +18,7 @@ main(int argc, char **argv)\
char buf[1024], initfunc[1024], *p, *q;\
\
if(argc != 2) {\
-\t\tfprintf(stderr, "usage: sys sys\\n");\
+\t\tfprintf(stderr, "usage: mkbuiltin1 sys\\n");\
\t\tfprintf(stderr, "in file $1.6 s/PACKAGE/$1/\\n");\
\t\texit(1);\
\t}\
- ファイル名が
mksys.c
からmkbuiltin1.c
に変更されました。 - コメントが
Extract import data from sys.6 and generate C string version.
からCompile .go file, import data from .6 file, and generate C string version.
に変更され、このCプログラムの役割がより正確に記述されました。これは、Goの.go
ファイルをコンパイルして生成された.6
ファイルからインポートデータを抽出し、C言語の文字列形式で出力する役割を担っていることを示しています。 - エラーメッセージの
usage: sys sys
がusage: mkbuiltin1 sys
に更新されました。
これらの変更により、builtin.c
の生成プロセスは、バージョン管理システムから独立し、ビルド時に常に最新の状態で生成されるようになりました。これにより、Goユーザーのビルド体験が向上し、開発者にとってもbuiltin.c.boot
の更新が半自動化されるというメリットがもたらされました。
関連リンク
- Go言語の公式リポジトリ: https://github.com/golang/go
- Perforce (P4) の概要: https://www.perforce.com/products/helix-core (当時のGoプロジェクトで利用されていたバージョン管理システム)
参考にした情報源リンク
- Go言語のソースコード(特に
src/cmd/gc
ディレクトリ) - コミットメッセージと差分情報
- Go言語の初期のビルドシステムに関する一般的な知識
- Perforceバージョン管理システムの基本的な概念