Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(939)

Issue 3996047: code review 3996047: runtime: remove tiny (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by rsc
Modified:
13 years, 3 months ago
Reviewers:
CC:
r, r2, luriel, golang-dev
Visibility:
Public.

Description

runtime: remove tiny It is unmaintained and untested, and I think it's broken too. It was a toy to show that Go can run on real hardware, and it served its purpose. The source code will of course remain in the repository history, so it could be brought back if needed later.

Patch Set 1 #

Patch Set 2 : code review 3996047: runtime: remove tiny #

Patch Set 3 : code review 3996047: runtime: remove tiny #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -499 lines) Patch
M src/pkg/runtime/Makefile View 1 1 chunk +0 lines, -3 lines 0 comments Download
R src/pkg/runtime/tiny/386/defs.h View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/386/rt0.s View 1 chunk +0 lines, -28 lines 0 comments Download
R src/pkg/runtime/tiny/386/signal.c View 1 chunk +0 lines, -19 lines 0 comments Download
R src/pkg/runtime/tiny/386/sys.s View 1 chunk +0 lines, -92 lines 0 comments Download
R src/pkg/runtime/tiny/README View 1 chunk +0 lines, -123 lines 0 comments Download
R src/pkg/runtime/tiny/arm/defs.h View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/arm/rt0.s View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/arm/signal.c View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/arm/sys.s View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/bootblock View Binary file 0 comments Download
R src/pkg/runtime/tiny/dot-bochsrc View 1 chunk +0 lines, -18 lines 0 comments Download
R src/pkg/runtime/tiny/io.go View 1 chunk +0 lines, -53 lines 0 comments Download
R src/pkg/runtime/tiny/mem.c View 1 chunk +0 lines, -50 lines 0 comments Download
R src/pkg/runtime/tiny/os.h View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/runtime_defs.go View 1 chunk +0 lines, -14 lines 0 comments Download
R src/pkg/runtime/tiny/signals.h View 1 chunk +0 lines, -1 line 0 comments Download
R src/pkg/runtime/tiny/thread.c View 1 chunk +0 lines, -92 lines 0 comments Download

Messages

Total messages: 4
rsc
Hello r (cc: golang-dev@googlegroups.com), I'd like you to review this change.
13 years, 3 months ago (2011-01-25 21:44:47 UTC) #1
r2
LGTM
13 years, 3 months ago (2011-01-25 23:38:04 UTC) #2
luriel
Sad to see this go, but for people interested in Go running on raw hardware ...
13 years, 3 months ago (2011-01-26 01:08:00 UTC) #3
rsc
13 years, 3 months ago (2011-01-26 13:41:33 UTC) #4
*** Submitted as http://code.google.com/p/go/source/detail?r=820f713f76f6 ***

runtime: remove tiny

It is unmaintained and untested, and I think it's broken too.
It was a toy to show that Go can run on real hardware,
and it served its purpose.

The source code will of course remain in the repository
history, so it could be brought back if needed later.

R=r, r2, uriel
CC=golang-dev
http://codereview.appspot.com/3996047
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b