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.

Sign up for GitHub

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

Jump to bottom

Consider packaging the tests in the sourceonly tarball #9207

Closed
sorenstoutner opened this issue Sep 18, 2024 · 1 comment
Closed

Consider packaging the tests in the sourceonly tarball #9207

sorenstoutner opened this issue Sep 18, 2024 · 1 comment

Comments

@sorenstoutner
Copy link
Contributor

Would it be possible to package the tests directory in the sourceonly tarball? Debian has provisions for automatically running the tests every time the package is built or rebuilt, which helps catch a multitude of errors in the packaging.
@ecdsa
Copy link
Member

ecdsa commented Sep 19, 2024

that sounds reasonable

SomberNight added a commit that referenced this issue Oct 14, 2024
@SomberNight
build: include tests/ in tarballs
3ec16a0 
closes #9207
Yarny0 added a commit to Yarny0/nixpkgs that referenced this issue Oct 19, 2024
@Yarny0
electrum: 4.5.5 -> 4.5.6
d326fa8 
Release notes:
https://github.com/spesmilo/electrum/blob/4.5.6/RELEASE-NOTES

Beginning with this update,
electrum includes its tests in its release tarballs, cf.

spesmilo/electrum#9207
spesmilo/electrum@3ec16a0
https://github.com/spesmilo/electrum/blob/4.5.6/RELEASE-NOTES#L31

Therefore we drop the separate download of tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ecdsa @sorenstoutner