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

Changing tar command #6

Merged
merged 1 commit into from
Jan 2, 2017
Merged

Conversation

sylvchev
Copy link
Contributor

@sylvchev sylvchev commented Jan 2, 2017

I'm currently installing Gentoo on a new machine and I use your very nice script.
I was using a live USB based on Debian to install Gentoo and I ran into a problem with the order of the options: the tar command silently failed and did not take into account the exclude dirs. By changing the order of the options on the command line, everything went fine.

@TheChymera TheChymera merged commit 9a96781 into TheChymera:master Jan 2, 2017
@TheChymera
Copy link
Owner

Hm , this is strange, because it never happened to me, and if what you are saying is right, I would expect it to happen all the time.

Still, your change seems to be consistent with the tar man page recommended usage, so I'll pull it. I guess having some CI tests would really make things easier. Do you have any idea if one can get ravis or circleCI (or anything else) to install a Gentoo system?

@sylvchev
Copy link
Contributor Author

sylvchev commented Jan 2, 2017

This is a very good idea ! TravisCI or CircleCI could be of great help. I will try to see if it is possible to make use of the new "cloud" images available on http://distfiles.gentoo.org/releases/amd64/autobuilds/current-stage4-amd64-cloud/.
It could really be nice a to have an automated system for verifying gentoo installation/images.

@TheChymera
Copy link
Owner

TheChymera commented Jan 2, 2017

I talked to the infra team and @robbat2 said that docker is not officially supported. Those images are openstack (and EC2, though I have not tested this) compatible, but travis and circleCI use docker. So we would have to find a different solution. Please let me know if it turns out I am wrong.

@sylvchev
Copy link
Contributor Author

sylvchev commented Jan 2, 2017

Thank you for the update, I will see if I could contribute in some way (with colleagues from my lab or students). It could be very nice to have process to generate a simple docker image.

@TheChymera
Copy link
Owner

I think there are already some, but they are not “official”: https://hub.docker.com/u/gentoo/ . Tell me if you get any CI service provider to work with them.

@robbat2
Copy link

robbat2 commented Jan 3, 2017

@sylvchev what do docker images need that aren't in the normal stage3 images?

@TheChymera
Copy link
Owner

TheChymera commented Jan 4, 2017

So, it seems what I talked about actually happened, and even this fairly minor syntax change broke the script. It's fixed now, but this should go to show that even for something like this we should have CI tests if we want people to be able to use the development version. @sylvchev have you found any time to look into this?

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