Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
|
|
|
|
Install gordon to $GOROOT where godoc expects to find him.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
godoc does not transparently decompress compressed files it finds.
The result is 404 errors.
2010/07/04 20:32:03 lstat /usr/lib/golang/doc/godocs.js: no such file or directory
2010/07/04 20:32:03 lstat /usr/lib/golang/favicon.ico: no such file or directory
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
The shell is interpolating the ${GOARM=foo} expression before
it reaches env.sh, making it into a no-op. Escape the first
dollar sign to prevent this.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
debian/rules, src/pkg/syscall/mksyscall.sh, and test/errchk all
require perl.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
gopprof is a perl script.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
Using mawk to build seems to work fine so far.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
If $DESTDIR/usr/bin/8c is already present and up to date, do not
forget to remove the temporary $DESTDIR/usr/bin/8c+.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
There was a missing backslash. While fixing it, speed up the
operation by avoiding an unnecessary checkout.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
|
|
Bang on the debian/rules file until it works. Changes in behavior:
- the test suite was not being run.
- the DEB_BUILD_OPTIONS settings were not being respected.
Now nocheck is respected and at least we complain about noopt
(the optimization level is not configurable in the upstream
makefile).
- the ‘debian/rules clean’ rule was flaky.
- the GOROOT dir was hard-coded to /usr/lib/google-go. Now it
is /usr/lib/golang to match the upstream domain name, but more
importantly it is configurable.
- there were a bunch of hardcoded wrappers in the debian/bin
dir, which made for unpleasant editing. Now they are automatically
built as needed.
- the wrappers did not let the user override the GO* parameters
through her own environment.
- files needed to run "godoc -http=:6060" were not installed.
- the rules for determining GOARCH, GOARM, etc were not cross-
compilation friendly.
Since I had to rewrite debian/rules anyway, I went ahead and put the
result in the public domain so it can be reused by upstream and other
packages.
Now it works for me as far as "godoc -http=:6060" on my 32-bit x86; no
guarantees beyond that, though. Please break it so we can get
something robust ready for release.
|
|
Glance over license headers and document findings in a format
vaguely similar to that documented at
<http://dep.debian.net/deps/dep5/>.
Still to do:
* match up files in src/pkg/debug/*/testdata/* to corresponding
source
* check MPEG license: does it permit modification?
* check pngsuite licensing: does it permit modification?
* figure out how to deal with third-party content in PDFs:
- color profiles are labelled as copyright Apple
- xkcd comic does not permit commercial use
* add license notices where missing. check the result with
‘licensecheck’.
|
|
Currently I see two show stoppers:
- the ugly wrapper for those env. variables. I don't really want to
have it but $GOROOT is also what the build process depends and makes
changes to. I will be surely more than happy if we can have a better
solution.
- missing man pages
|
|
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1741047
|
|
panic if base is invalid.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1702050
|
|
Use a count of -1 for infinity. Ditto for Replace.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1704044
Committer: Rob Pike <r@golang.org>
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/1693047
|
|
libmach: disassemble CMPPD as 6a expects
R=ken2
CC=Charlie Dorian, golang-dev
http://codereview.appspot.com/1704046
|
|
(Here, quoted strings are the official AMD names.)
The amd64 "movsxd" instruction, when invoked
with a 64-bit REX prefix, moves and sign extends
a 32-bit value from register or memory into a
64-bit register. 6.out.h spells this MOVLQSX.
6.out.h also includes MOVLQZX, the zero extending
version, which it implements as "movsxd" without
the REX prefix. Without the REX prefix it's only sign
extending 32 bits to 32 bits (i.e., not doing anything
to the bits) and then storing in a 32-bit register.
Any write to a 32-bit register zeros the top half of the
corresponding 64-bit register, giving the advertised effect.
This particular implementation of the functionality is
non-standard, because an ordinary 32-bit "mov" would
do the same thing.
Because it is non-standard, it is often mishandled or
not handled by binary translation tools like valgrind.
Switching to the standard "mov" makes the binaries
work better with those tools.
It's probably useful in 6c and 6g to have an explicit
instruction, though, so that the intent of the size
change is clear. Thus we leave the concept of MOVLQZX
and just implement it by the standard "mov" instead of
the non-standard 32-bit "movsxd".
Fixes issue 896.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1733046
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1705041
Committer: Russ Cox <rsc@golang.org>
|
|
R=iant
CC=golang-dev
http://codereview.appspot.com/1666048
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1650041
|
|
With these changes, goinstall is now able to use branches
maintained with Bazaar located in Launchpad.
Project aliases such as /project and /project/series are
supported in addition to specific user or team branches
such as /~user/project/branch. Temporary branches under
the +junk special project are also supported.
As a curious side effect, since Launchpad is able to import
code from other locations, they can be indirectly
accessible too if desired.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1699050
Committer: Russ Cox <rsc@golang.org>
|
|
R=adg
CC=golang-dev
http://codereview.appspot.com/1693046
|
|
The Makefile and cgo now rewrite / to _ when creating the path.
The .so for gosqlite.googlecode.com/hg/sqlite is named
cgo_gosqlite.googlecode.com_hg_sqlite.so, and then 6l and 8l
both include a default rpath of $GOROOT/pkg/$GOOS_$GOARCH.
This should make it easier to move binaries from one system
to another.
Fixes issue 857.
R=iant, r
CC=golang-dev
http://codereview.appspot.com/1700048
|
|
Fixes issue 868.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1695049
|
|
Fixes issue 879.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1678048
|
|
R=golang-dev, rsc
CC=golang-dev
http://codereview.appspot.com/1665047
Committer: Russ Cox <rsc@golang.org>
|
|
This is the Replace I suggested in the review of CL 1114041.
It's true that we already have
regexp.MustCompile(regexp.QuoteMeta(old)).ReplaceAll(s, new)
but because this Replace is doing a simpler job it is
simpler to call and inherently more efficient.
I will add the bytes implementation and tests to the
CL after the strings one has been reviewed.
R=r, cw
CC=golang-dev
http://codereview.appspot.com/1731048
|
|
(addressing comments from 1741045)
R=r
CC=golang-dev
http://codereview.appspot.com/1678047
|
|
For generating non-self-signed certs we need to be able to specify a
public key (for the signee) which is different from the private key (of
the signer).
R=rsc
CC=golang-dev
http://codereview.appspot.com/1741045
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1319042
Committer: Russ Cox <rsc@golang.org>
|
|
Benchmark of exp to 28 ns/op from 64 ns/op,
on 2.53GHz Intel Core 2 Duo.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1594041
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 834.
R=rsc, brainman
CC=golang-dev
http://codereview.appspot.com/1686047
Committer: Russ Cox <rsc@golang.org>
|
|
One goroutine started up and was waiting in rw. Then another
goroutine decided to close the pipe. The closing goroutine
stalled calling p.io.Lock() in pipeHalf.close. (This happened
in gccgo). If the closing goroutine had been able to set the
ioclosed flag, it would have gone on to tell the runner that
the pipe was closed, which would then send an EINVAL to the
goroutine sleeping in rw. Unlocking p.io before sleeping in
rw avoids the race.
R=rsc, rsc1
CC=golang-dev
http://codereview.appspot.com/1682048
|
|
Fixes issue 893.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1687045
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1729046
|
|
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/1727043
|
|
Fixes issue 761.
R=PeterGo, adg, rsc
CC=golang-dev
http://codereview.appspot.com/1121042
Committer: Russ Cox <rsc@golang.org>
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/1679045
|
|
R=rsc, Mateusz Czaplinski
CC=golang-dev
http://codereview.appspot.com/1600041
Committer: Russ Cox <rsc@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1678046
|
|
Fixes issue 758.
R=rsc, adg
CC=golang-dev
http://codereview.appspot.com/1078041
Committer: Andrew Gerrand <adg@golang.org>
|
|
Fixes issue 886.
R=r
CC=golang-dev
http://codereview.appspot.com/1667051
|
|
Fixes issue 822.
Credit to http://codereview.appspot.com/1442041 by danielfleischman
R=rsc
CC=golang-dev
http://codereview.appspot.com/1738047
|
|
Fixes issue 845.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1673049
|