Skip to content

Conversation

@karim
Copy link
Member

@karim karim commented May 12, 2018

No description provided.

@karim karim requested a review from justinclift May 12, 2018 12:09
@karim
Copy link
Member Author

karim commented May 12, 2018

versioninfo

@justinclift
Copy link
Member

Good idea. I have no clue personally as to whether this will negatively affect anything else (eg non-windows), but we can give it a go. 😄

With the Travis failure, it looks like one of the two compile tests worked fine (good), but the other failed due to Travis infrastructure problems. eg nothing to do with this PR.

I'll merge this now, and we can look over the resulting Windows builds tomorrow to make sure they're good + fine tune things a bit if needed. 😄

@justinclift justinclift merged commit 1865272 into sqlitebrowser:master May 12, 2018
@karim
Copy link
Member Author

karim commented May 12, 2018

Yes, it is just a Windows thing. The .rc is used by Windows Resource Compiler, so I doubt that it will affect anything else.

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