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

Attempt to avoid "Preparing modules for first use" #140

Closed
wants to merge 1 commit into from
Closed

Attempt to avoid "Preparing modules for first use" #140

wants to merge 1 commit into from

Conversation

IlyaFinkelshteyn
Copy link

No description provided.

@chef-supermarket
Copy link

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.

GitHub Users Who Are Not Authorized To Contribute

The following GitHub users do not appear to have signed a CLA:

Please sign the CLA here.

@mwrock
Copy link
Contributor

mwrock commented Aug 20, 2016

I agree -noprofile is good but might not solve this problem.

You want to make sure scripts are pretended with $profilepreference = "silentlycontinue"

@IlyaFinkelshteyn
Copy link
Author

It is attempt to solve this issue. As you see $profilepreference = "silentlycontinue" is already set and stopped helping after latest Appveyor VHD updates. Therefore Christoph was needed to go back to previous Appveyor VHD. Adding -noprofile actually helps, and all builds are passed OK on the current image (https://ci.appveyor.com/project/IlyaFinkelshteyn/train/build/master-11). With previous image ruby_version=21 fails.
I am still trying to figure out what exact update broke previous behavior (which did not require -noprofile), but I have feeling that it could be Windows update we should not undo.

@IlyaFinkelshteyn
Copy link
Author

Anyhow I believe that this change together with latest "Visual Studio 2015" image for Appveyor will work OK.

@mwrock
Copy link
Contributor

mwrock commented Aug 20, 2016

Oh interesting. I do know appveyor added wmf5 to their image about a month ago which broke stuff in this area. I wonder if this is the latest round of "anniversary" updates that's new with P's 5.1.

@IlyaFinkelshteyn
Copy link
Author

Oh, hopefully I will understand what really happened because now I am suggesting it little blindly...

@chris-rock
Copy link
Contributor

@IlyaFinkelshteyn Wouldn't that require us to change reactivate the new image in our appveyor config?

@IlyaFinkelshteyn
Copy link
Author

IlyaFinkelshteyn commented Aug 23, 2016

Yes, with previous image and -noprofile one job fails (both in this PR and for my fork). Honestly still do not know why. New image and -noprofile works good.

@chris-rock
Copy link
Contributor

This fixes #153

@chris-rock
Copy link
Contributor

Thanks @IlyaFinkelshteyn. I ported the noprofile options to the latest master. Thanks a lot for highlighting the solution! This PR is superseeded by #152

@chris-rock chris-rock closed this Sep 20, 2016
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.

4 participants