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

Buildbot should not only test "make check" but also "make install" #19698

Closed
nodakai opened this issue Dec 10, 2014 · 11 comments
Closed

Buildbot should not only test "make check" but also "make install" #19698

nodakai opened this issue Dec 10, 2014 · 11 comments
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. P-medium Medium priority T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@nodakai
Copy link
Contributor

nodakai commented Dec 10, 2014

... or make dist.

It would be nice to avoid troubles like #19678 .

@chris-morgan
Copy link
Member

In the last six months:

#15545
#19678
#19970
#20383

As I recall it there was one other one a few months ago that got fixed before it was filed.

Can we please do something about this?

@brson brson mentioned this issue Jan 2, 2015
65 tasks
@brson
Copy link
Contributor

brson commented Jan 2, 2015

I've added to the list.

@retep998
Copy link
Member

retep998 commented Nov 7, 2015

There have been a huge amount of issues with the -windows-gnu distribution being broken lately. Having some sort of integration testing for whether the built packages actually work would be really useful.

@edunham
Copy link
Member

edunham commented Nov 9, 2015

Currently testing make dist on auto builds in dev. I'll deploy to prod as soon as I get it working.

@nodakai
Copy link
Contributor Author

nodakai commented Mar 12, 2016

@edunham Was it deployed?

@Mark-Simulacrum
Copy link
Member

I think this may be somewhat done now, as we build artifacts for each PR. However, I'm not sure, so cc @brson @alexcrichton

@alexcrichton
Copy link
Member

Yes it'd still be great to test this!

@Mark-Simulacrum Mark-Simulacrum added T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. and removed A-infrastructure labels Jun 25, 2017
@Mark-Simulacrum
Copy link
Member

To update the current status: make dist's equivalent is run across all dist builders, but we do not check make install. I'm going to mark this as P-medium since I imagine distros would appreciate this.

@Mark-Simulacrum Mark-Simulacrum added C-enhancement Category: An issue proposing an enhancement or a PR with one. P-medium Medium priority and removed P-low Low priority labels Jul 22, 2017
@steveklabnik
Copy link
Member

Triage: I thought maybe something changed here, but am not totally sure.

@Mark-Simulacrum
Copy link
Member

No nothing changed here, we're still only running x.py dist, not x.py install on our builders. To me at least it's also not super clear how we'd test the installation...

@aidanhs
Copy link
Member

aidanhs commented Apr 16, 2019

We had a chat about this in the infra meeting today and I'm going to close to reflect our current feelings on the issue - in short, though it seems like a reasonable thing to do, we haven't seen an issue for quite a while and make install isn't a key thing we're looking to protect right now.

@aidanhs aidanhs closed this as completed Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. P-medium Medium priority T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

9 participants