Skip to content
This repository has been archived by the owner on Oct 19, 2021. It is now read-only.

Convert to bazel managed dependencies #6

Closed
prestonvanloon opened this issue Dec 21, 2018 · 2 comments
Closed

Convert to bazel managed dependencies #6

prestonvanloon opened this issue Dec 21, 2018 · 2 comments
Assignees

Comments

@prestonvanloon
Copy link
Member

bazel-go-ethereum should attempt to load bazel dependencies, if they are not already present.

This would be a "maybe" type rule where the downstream client would load this library and call

load("@com_github_ethereum_go_ethereum//:deps.bzl", "go_ethereum_repositories")

go_ethereum_repositories()

The go_ethereum_repositories rule will load any repository necessary to build go-ethereum which is not already installed. What we are seeing is that there are mixed versions of the same library in our downstream client which leads to wasted/redundant work.

@prestonvanloon prestonvanloon self-assigned this Dec 21, 2018
@prestonvanloon
Copy link
Member Author

Mostly done in #7

rauljordan pushed a commit that referenced this issue Apr 5, 2019
* added tests for new abi encoding features (#4)

* added tests from bytes32[][] and string[]

* added offset to other types

* formatting

* Abi/dynamic types (#5)

* Revert "Abi/dynamic types (#5)" (#6)

This reverts commit dabca31d797623d43bd780721fc0ad461d24be53.

* Abi/dynamic types (#7)

* some cleanup

* Apply suggestions from code review

apply suggestions

Co-Authored-By: vedhavyas <vedhavyas.singareddi@gmail.com>

* added better formatting (#8)

* review chnages

* better comments
@nisdas
Copy link
Member

nisdas commented Jun 1, 2020

Closing this as its been done as of: df74fa9

@nisdas nisdas closed this as completed Jun 1, 2020
rauljordan pushed a commit that referenced this issue Apr 19, 2021
* internal/build: implement signify's signing func
* Add signify to the ci utility
* fix output file format
* Add unit test for signify
* holiman's + travis' feedback
* internal/build: verify signify's output
* crypto: move signify to common dir
* use go-minisign to verify binaries
* more holiman feedback
* crypto, ci: support minisign output
* only accept one-line trusted comments
* configurable untrusted comments
* code cleanup in tests
* revert to use ed25519 from the stdlib
* bug: fix for empty untrusted comments
* write timestamp as comment if trusted comment isn't present
* rename line checker to commentHasManyLines
* crypto: added signify fuzzer (#6)
* crypto: added signify fuzzer
* stuff
* crypto: updated signify fuzzer to fuzz comments
* crypto: repro signify crashes
* rebased fuzzer on build-signify branch
* hide fuzzer behind gofuzz build flag
* extract key data inside a single function
* don't treat \r as a newline
* travis: fix signing command line
* do not use an external binary in tests
* crypto: move signify to crypto/signify
* travis: fix formatting issue
* ci: fix linter build after package move

Co-authored-by: Marius van der Wijden <m.vanderwijden@live.de>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants