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

Source code missing for test/data/helloworld binary #297

Open
jas4711 opened this issue Jan 19, 2024 · 4 comments
Open

Source code missing for test/data/helloworld binary #297

jas4711 opened this issue Jan 19, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@jas4711
Copy link

jas4711 commented Jan 19, 2024

Hi. I am packaging in-toto-golang for Debian.

We noticed that the test/data/helloworld is a binary executable that is stored in the git repository, and is used during self-tests. Where is the source code to generate that binary? Debian does not want to ship binaries that cannot be rebuilt from source code.

I tried replacing it with another binary that I built myself, but I see hard-coded SHA checksums in in_toto/runlib_test of the helloworld binary, and I'm not sure if there are other properties this binary must have in order for the self-tests to work. Can you help me? I will disable the self-tests that depend in this binary in order to proceed with inclusion into Debian.

Thanks,
Simon

@jas4711 jas4711 added the bug Something isn't working label Jan 19, 2024
@jas4711
Copy link
Author

jas4711 commented Jan 19, 2024

@adityasaky
Copy link
Member

IIRC it's only the hash but I can take another look. FWIW, we're evaluating the future of in-toto-golang and how it may be merged with github.com/in-toto/witness / go-witness, so perhaps it's best to hit pause on packaging this one? Is this for a specific Debian-based use case?

@jas4711
Copy link
Author

jas4711 commented Jan 19, 2024

Thanks for quick response!

IIRC it's only the hash but I can take another look.

The binary test/data/helloworld is in the git repository. We are pruning that file, and applying the previous patch to avoid test failures.

FWIW, we're evaluating the future of in-toto-golang and how it may be merged with github.com/in-toto/witness / go-witness, so perhaps it's best to hit pause on packaging this one? Is this for a specific Debian-based use case?

I need (parts of) in-toto for sigstore's rekor. Are your plans likely to be implemented in the next few weeks and rekor updated? If not I think it is okay to continue package in-toto-golang since it is necessary for rekor.

/Simon

@adityasaky
Copy link
Member

No, the consolidation effort will likely take longer than that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants