Skip to content

Fix build error#27

Open
marquisrobb wants to merge 1 commit intocloudflarearchive:masterfrom
marquisrobb:master
Open

Fix build error#27
marquisrobb wants to merge 1 commit intocloudflarearchive:masterfrom
marquisrobb:master

Conversation

@marquisrobb
Copy link
Copy Markdown

@marquisrobb marquisrobb commented Jul 18, 2019

Error:

`➜ sidh git:(master) make
GOPATH=/Users/user/workspace/bfx/sidh/marq/sidh/build go get github.com/henrydcase/nobs/hash/sha3
mkdir -p /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf internal /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf etc /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf p503 /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf p751 /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf sidh /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
cp -rf sike /Users/user/workspace/bfx/sidh/marq/sidh/build/src/github.com/cloudflare/sidh
GOPATH=/Users/user/workspace/bfx/sidh/marq/sidh/build go vet github.com/cloudflare/sidh/p503

github.com/cloudflare/sidh/p503

build/src/github.com/cloudflare/sidh/p503/arith_amd64.s:695:1: [amd64] fp503Mul: use of unnamed argument 0(FP); offset 0 is z+0(FP)
build/src/github.com/cloudflare/sidh/p503/arith_amd64.s:696:1: [amd64] fp503Mul: use of unnamed argument 8(FP); offset 8 is x+8(FP)
make: *** [test-p503] Error 2
➜ sidh git:(master)`

@kriskwiatkowski
Copy link
Copy Markdown
Contributor

Thanks for contribution. This repo is outdated and we will be removing it.
The code has been ported here:
https://github.com/cloudflare/circl/tree/master/dh/sidh

@initpwn
Copy link
Copy Markdown

initpwn commented Oct 5, 2021

Thanks for contribution. This repo is outdated and we will be removing it. The code has been ported here: https://github.com/cloudflare/circl/tree/master/dh/sidh

In that case, please add a reference of new repo in README.

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

Successfully merging this pull request may close these issues.

3 participants