-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Add Travis configuration for MinGW64 builds #116
base: master
Are you sure you want to change the base?
Conversation
also, remove export of MAKE
I think that covers everything you've mentioned, except
I'm not sure of the necessity of that, TBH. It's the toolchain binary location, but I don't really know whether it's not exported already by that point. Locally, I didn't use the exact same approach, since there are special binary launchers for Windows that take care of exactly things like that I guess. I can try looking through docs to clarify. |
It seems like Travis isn't happy yet: https://travis-ci.org/github/zenorogue/hyperrogue/jobs/689351528
The good news is that this seems to be exactly the error message given at msys2/MSYS2-packages#1962 (reported 2 days ago), and so maybe if we just wait a few days everything will be back to normal and Travis will become happy? I notice the lines
in the middle there. I vaguely remember seeing the same help doc that you must have seen, re that first |
Added a commit fixing a quirk in Re As for the former errors, now that I look at it more closely, it seems odd that there is no EDIT: msys2/MSYS2-packages#1962 (comment) |
Let's see what it thinks about this one. Also, it's a bit unexpected to me that Travis automatically pulls updates to its config even from unapproved changes, but I guess it's for the best: it lets test things more quickly. |
sound of forehead slapping |
Yeah, I think this was in the wrong place. I can try just one more (admittedly hacky) thing, and that's to bypass Chocolatey-package initial configuration (/NoUpdate option), upgrade pacman, and then resume as usual. This might be necessary, because last update to the package was in October of the last year. |
Seems like I was overly optimistic with removing the "draft" status. For reference, this here seems to be the same kind of issue. |
In case I was too oblique with my reference to msys2/MSYS2-packages#1962 (comment) , I think the really important comment is this one:
In other words, it sounds like the Msys2 maintainers broke something within the past 72 hours, and (they didn't say so but I hope) if we just wait a while they may fix it. I know how tempting it is to keep trying to solve the problem Right Now, but it may be a better use of time to just let this PR sit for a week and then pick it up again and see if it's magically fixed itself. Your "this here" was also posted in the past 24 hours, indicating that this is a very new (and I hope temporary) upstream issue. |
I believe I'd found references to the same problem from earlier than that. But you may be right still. Sorry for commit-spam, by the way. It's sort of the only way to test, but if the latest one fails, I'll probably stop and wait for the proper fix. In my defense (quote from me earlier),
This was my reasoning, since there's also the matter of Chocolatey package being used. |
https://chocolatey.org/packages/msys2#comment-4919416069 OK, I admit that I should've paid more attention. Looks like the problem is indeed very recent, and will probably be fixed soon. |
Looks like (msys2/MSYS2-packages#1970 (comment)) the issue has been fixed. I should probably clean up the commits here, and try again. |
deccf66
to
3cf0a21
Compare
also shorten cmds a bit
3cf0a21
to
1067df8
Compare
remove taskkills per msys2/MSYS2-packages/issues/1962#issuecomment-632703947 also shorten pacman cmds
It builds alright now, but the test fails due to "missing" DLL. Must be something funny going on with the |
the whole toolchain is quite large, but totally unnecessary
following https://travis-ci.community/t/how-to-set-path-inside-mingw64-environment/8621 also remove chocolatey from caching, because something seems to change there every time
Could the current failure https://travis-ci.org/github/zenorogue/hyperrogue/jobs/690404626 be due simply to writing |
Actually, it does say a bit more, it's just that for inexplicable reasons that output is disconnected from where it logically belongs: this line specifically (it's what I referred to in my last comment).
EDIT: let's try something really dumb first. As for what you suggest, I think I did try spelling out the extension explicitly, but haven't yet tried the backslash. I can try that next. |
mingw-mesa contains opengl32.dll
At last, it works. Sure took some ugly hacks to pull off though. Now to think up a way to clean up this mess. |
also delete -y arg from all pacman invocations because updating database doesn't make sense when not doing full-upgrade, which choco should do for us
also add verbose flag to choco uninstall
also cat choco log for diagnostics
also try removing old mingw installation manually, so that chocolatey doesn't invalidate cache
.travis.yml
Outdated
if [[ "$TRAVIS_USE_AUTOTOOLS" == "1" ]]; then | ||
if [[ "$TRAVIS_BUILD_SYSTEM" == "autotools" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funny. Fixing this has led to MacOS-autotools build failing because autotools
are apparently pre-installed.
MacOS has them pre-installed per https://docs.travis-ci.com/user/reference/osx
The newly added MinGW-autotools build fails, but if I read this correctly, it's because it's unique in the fact that it also includes On an unrelated note, I don't really like how on build and test steps there have to be checks whether build-host is Windows. Would it be better if, at early preparation steps, And the output being out of order might not be an isolated occurence. I don't know how I feel about their workaround though. |
Based on: https://docs.travis-ci.com/user/reference/windows/#how-do-i-use-msys2
Just one -- Makefile.simple -- job for now, since that seemed to work fine locally for me. Automake-build could also be added, but not sure of its necessity.