Skip to content

remove printing config.log when build fail in appveyor build script#1440

Merged
daschuer merged 1 commit intomixxxdj:masterfrom
sblaisot:appveyor-build-script
Dec 23, 2017
Merged

remove printing config.log when build fail in appveyor build script#1440
daschuer merged 1 commit intomixxxdj:masterfrom
sblaisot:appveyor-build-script

Conversation

@sblaisot
Copy link
Copy Markdown
Member

When a build fail on appveyor, the build script prints the whole config.log. This one is about 1000 lines and contain lots of try/fail to determine the correct configuration, so a lot of errors.
Thus, it is hard to catch the real error that makes the build fail somewhere 1000 lines above the end of buils log.

Commenting-out this should ease finding the real build error cause. I leave it commented out in the build script to allow esay uncommenting in case ob buils system debugging.

@sblaisot sblaisot mentioned this pull request Dec 22, 2017
@daschuer
Copy link
Copy Markdown
Member

Thank you!

@daschuer daschuer merged commit c16aac5 into mixxxdj:master Dec 23, 2017
@sblaisot
Copy link
Copy Markdown
Member Author

it seems this has not landed into 2.1 branch
@dashuer can you cherry-pick this one to 2.1 please ?

@daschuer
Copy link
Copy Markdown
Member

Oh sorry, I must have tricked myself. Fixed.

@sblaisot sblaisot deleted the appveyor-build-script branch December 31, 2017 17:25
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.

2 participants