Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

not enough arguments in call to eyes.NewClient #1

Closed
davehaughton1 opened this issue Jul 11, 2016 · 2 comments
Closed

not enough arguments in call to eyes.NewClient #1

davehaughton1 opened this issue Jul 11, 2016 · 2 comments

Comments

@davehaughton1
Copy link

Hi, when I try compile basecoin, I get these errors:

cmd/basecoin/main.go:23: not enough arguments in call to eyes.NewClient
cmd/basecoin/main.go:41: not enough arguments in call to server.NewServer

It can be fixed by adding in the transport parameter (socket | grpc) for those calls in main.go

@ethanfrey
Copy link
Contributor

Seems to have been fixed in the meantime by someone else...

Time to close this issue?

@davehaughton1
Copy link
Author

Yep, thanks

ethanfrey added a commit that referenced this issue Oct 25, 2017
Refactor out BaseApp, StoreApp (per spec #1 and #2)
cwgoes added a commit that referenced this issue Aug 22, 2018
cwgoes pushed a commit that referenced this issue Aug 22, 2018
The URL to abci-spec.md fixed
cwgoes pushed a commit that referenced this issue Feb 8, 2019
* add a bunch of tests, add DONTCOVER text tag

- Also fix flaky test (closes: #3559). Don't test values
  returned by queries since there's no way to query a
  specific height via REST.

* GetTempDir -> NewTestCaseDir
@golangcibot golangcibot mentioned this issue Jun 10, 2019
5 tasks
alexanderbez added a commit that referenced this issue Jul 2, 2019
dogemos referenced this issue in dogemos/cosmos-sdk Apr 9, 2020
chengwenxi added a commit that referenced this issue Apr 23, 2020
R4R: Add SM2 support for the account system
tac0turtle added a commit that referenced this issue May 15, 2020
mergify bot pushed a commit that referenced this issue Oct 8, 2020
* require old chain halts before upgrade

* Update proto/ibc/core/client/v1/client.proto

Co-authored-by: Federico Kunze <31522760+fedekunze@users.noreply.github.com>

* start address reviews

* Apply suggestions from code review

Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>

* address reviews

* rework upgrade to ensure there is never more than one upgrade client in store

* fix tests

* fix conditional

* make proto-gen

* remove if statement skipping tests in upgrade keeper test

* address reviews

* correctly escape and unescape merkle keys

* add small conditional check

Co-authored-by: Federico Kunze <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
Co-authored-by: Colin Axner <colinaxner@berkeley.edu>
faddat referenced this issue in notional-labs/cosmos-sdk Sep 23, 2021
Bring current with SDK
patiee pushed a commit to patiee/cosmos-sdk that referenced this issue Mar 3, 2023
patiee pushed a commit to patiee/cosmos-sdk that referenced this issue Mar 4, 2023
patiee referenced this issue in graphprotocol/cosmos-sdk Mar 5, 2023
ulbqb pushed a commit to ulbqb/cosmos-sdk that referenced this issue Jun 8, 2023
cosmos#1)

* Replace tendermint with celestiaorg tendermint version

* Add new ABCI method skeleton: build works now

* Add fraudproof data structure

* Add helper methods to baseapp test for fraudproofs

* Add TODOs in baseapp test method

* Add SetupBaseParams with SMT, WIP to IAVL

* Add tracekv readOperations and getAllKeysUsedInTrace

* Cleanup errors

* Add loadLastVersion to IAVL rootmulti store

* Remove enableFraudProofGenerationMode

* Add substore tracing to rootmulti store

* Add ability to reset trace writer buffers after each commit

* Able to get the trace made by fraudulent state transition now

* Start writing getFraudProof method in BaseApp

* Substitute SMT with IAVL trees inside getFraudProof

* Implement getStoreProof inside rootmulti store

* Completed generateFraudProof with IAVL store

* Remove routerOpts from baseapp test when generating fraud proofs

* Switch out proof decoder

* Add Root in IAVL to use working hash instead of lastCommitID hash

* Fix getAppHash to use working hashes of IAVL stores

* Implement ABCI method to verify fraud proofs

* Start setup BaseApp from IAVL store WIP

* Add option to SetInitialHeight

* Initialize baseapp from IAVL trees

* Use deepIAVLTree in loadVersion

* Add SetTracerFor to mock store

* Add gdocs to new ABCI methods

* Refactor from fmt.errorf into errors.New

* Update error packages used

* Update docs to remove SMT references

* Add unit test for checkFraudulentStateTransition

* modify executeNonFraudulentStateTransactions to accomodate case when EndBlock is fraudulent

* Refactor code
hoanguyenkh pushed a commit to hoanguyenkh/cosmos-sdk that referenced this issue Apr 24, 2024
add IterateGranterGrants and update basic keeper test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants