GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Build Linux + macOS bundles (Travis), build EXE (AppVeyor) by bittner · Pull Request #123 · PythonTurtle/PythonTurtle

Conversation

@bittner

@bittner bittner commented Oct 2, 2018

Copy link
Copy Markdown
Member
  • Use absolute import in __main__.py for PyInstaller to work
  • Add AppVeyor CI configuration

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch from b1ff538 to 99f5bf9 Compare October 2, 2018 09:45
@bittner

bittner commented Oct 2, 2018

Copy link
Copy Markdown
Member Author

Can you create an account at AppVeyor and enable automatic builds for this repository, please?

This way we should see what AppVeyor is doing on Windows, with the configuration in this PR.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch 2 times, most recently from a375ad3 to fac41dd Compare October 2, 2018 11:46
@cool-RR

cool-RR commented Oct 2, 2018 via email

Copy link
Copy Markdown
Member
I created an AppVeyor project for this repo but I'm not sure how to enable automatic builds. Also I tried to add you as a collaborator but I couldn't figure out how to do that. I went through the settings pages but no luck. Any clues?
…
On Tue, Oct 2, 2018 at 2:50 PM Peter Bittner ***@***.***> wrote: Can you create an account at AppVeyor <https://www.appveyor.com/> and enable automatic builds for this repository, please? This way we should see what AppVeyor is doing on Windows, with the configuration in this PR. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#123 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADdyr-LdwJZqObliteINh9jab_a67Rbks5ug1MHgaJpZM4XDozY> .
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 2, 2018 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

AppVeyor has a horr^H^H^H^H^H user interface that is not meant to be loved. I have suffered this morning, too. 😒

I think the users depend on what you have configured on GitHub (you grant the CI access to your GitHub organizations, and they allow access to accounts listed there; that's how CI services usually do it).

I'll try to push a change, then we maybe see something moving. Or can you trigger a build manually in your account?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch from fac41dd to ea51699 Compare October 2, 2018 14:51
@bittner

bittner commented Oct 2, 2018

Copy link
Copy Markdown
Member Author

It doesn't look like a build on AppVeyor was triggered. Can you check in the project settings of this repository on GitHub whether you have granted access to AppVeyor?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch from ea51699 to d1d67e6 Compare October 2, 2018 16:51
@cool-RR

cool-RR commented Oct 3, 2018

Copy link
Copy Markdown
Member

"Can you check in the project settings of this repository on GitHub whether you have granted access to AppVeyor?" I confirmed it's granted. (It was that way from before I made the check.)

"can you trigger a build manually in your account?" Did that now.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 3, 2018

Copy link
Copy Markdown
Member Author

Can you share the link to the AppVeyor project? Please, also try whether the URL is accessible when you're not logged in into your AppVeyor account (i.e. anonymous access is allowed).

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cool-RR

cool-RR commented Oct 3, 2018 via email

Copy link
Copy Markdown
Member
https://ci.appveyor.com/project/cool-RR/pythonturtle Seems to be accessible anonymously.
…
On Wed, Oct 3, 2018 at 11:12 AM Peter Bittner ***@***.***> wrote: Can you share the link to the AppVeyor project? Please, also try whether the URL is accessible when you're not logged in into your AppVeyor account (i.e. anonymous access is allowed). — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#123 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADdykjyGIYzXu2-KgvOhCgzqY4sh9ISks5uhHF8gaJpZM4XDozY> .
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 3, 2018

Copy link
Copy Markdown
Member Author

AppVeyor builds are not triggered. There should be another link next to Travis CI here in the UI. Can you check what's wrong? Maybe AppVeyor has support tickets, or infos that help pinning the problem in their knowledge base.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 3, 2018

Copy link
Copy Markdown
Member Author

Can you please also create an OAuth Token on GitHub via travis setup releases? (see Travis docs)

I let Travis build a Linux bundle and upload it to GitHub. That's where we need the encrypted OAuth Token. I have the code already prepared, only the token is now missing.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner bittner mentioned this pull request Oct 3, 2018
Build installer packages automatically #107
Open
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch from d429c88 to cd340b7 Compare October 3, 2018 14:27
@bittner

bittner commented Oct 3, 2018

Copy link
Copy Markdown
Member Author

If AppVeyor proves to be too cumbersome we can try Shippable. They provide Windows builds since recently. Do you want to try to create an account?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch 7 times, most recently from 53867d5 to e0c2db0 Compare October 4, 2018 04:52
@cool-RR

cool-RR commented Oct 4, 2018

Copy link
Copy Markdown
Member

I created an account, and enabled this project: https://app.shippable.com/github/cool-RR/PythonTurtle/dashboard

I'll do the oauth token later.

bittner reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner bittner mentioned this pull request Oct 4, 2018
ValueError: Unknown load command: 42 in Mach-O loader pyinstaller/pyinstaller#582
Closed
@cool-RR

cool-RR commented Oct 4, 2018

Copy link
Copy Markdown
Member

Could you tell me which commands to run for the oauth token?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 4, 2018

Copy link
Copy Markdown
Member Author

The travis command and a link to the related docs are in the comment above. 😃

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner bittner mentioned this pull request Oct 4, 2018
[OSX/Python] Unable to download 3.6 archive. The archive may not exist. Please consider a different version. travis-ci/travis-ci#9744
Closed
@bittner

bittner commented Oct 4, 2018

Copy link
Copy Markdown
Member Author

The builds on AppVeyor seem to work now! Awesome! 🎉

Can you try downloading and running the .EXE file that was built? The build spits out warnings all the way down, but who knows? Maybe it's working anyway.

I created an account, and enabled this project: https://app.shippable.com/github/cool-RR/PythonTurtle/dashboard

Cool! You need to make sure you have Windows builds enabled (as of the link in the comment above). I've been told by their support that, currently, we can only run Linux builds or Windows builds (for free) from the same account or GitHub organization.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner bittner changed the title WIP: Add AppVeyor CI configuration Add AppVeyor CI configuration, build Linux and macOS bundles Oct 4, 2018
@bittner bittner changed the title Add AppVeyor CI configuration, build Linux and macOS bundles Build Linux + macOS bundles (Travis), build EXE (AppVeyor CI) Oct 4, 2018
@bittner bittner changed the title Build Linux + macOS bundles (Travis), build EXE (AppVeyor CI) Build Linux + macOS bundles (Travis), build EXE (AppVeyor) Oct 4, 2018
@bittner

bittner commented Oct 4, 2018

Copy link
Copy Markdown
Member Author

I think, we could merge this PR now. I've packed more into it than originally planned.

This would also trigger an upload to PyPI (and we could see if it works or why it fails).

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

bittner added 5 commits October 4, 2018 11:14
@bittner
Use absolute import in __main__ for PyInstaller to work
5bc33a0
Add AppVeyor CI configuration (draft)
@bittner
Add clean and bundle commands to setup.py
f8ff509
@bittner
Build Linux bundle w/ Travis, Windows bundle w/ AppVeyor, add badge
b1bbe23
@bittner
Release Linux bundle on GitHub
bf00d93
@bittner
Build macOS bundle on Travis
9191e4f
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch 2 times, most recently from 7e908c4 to a392466 Compare October 4, 2018 10:48
@bittner
Use proper base class for setuptools commands
d61578c
Fix Pylint and flake8 issues introduced w/ commit ad19ac8
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch 2 times, most recently from b7d77f4 to 0e7637f Compare October 4, 2018 11:18
@bittner
Add Shippable CI config for Windows builds
11f98f4
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch 3 times, most recently from cd691ee to f162b76 Compare October 4, 2018 12:03
@bittner
Use CI config provided by Shippable support
56d450d
@bittner
bittner force-pushed the feature/add-appveyor-ci-config branch from f162b76 to 56d450d Compare October 4, 2018 12:04
@cool-RR
cool-RR merged commit 7112dec into PythonTurtle:master Oct 4, 2018
@cool-RR

cool-RR commented Oct 4, 2018

Copy link
Copy Markdown
Member

Great job! I downloaded the exe and it worked! It crashed after exiting the program, but that's probably because of a bug we have.

Do you still have interest in Shippable?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner
bittner deleted the feature/add-appveyor-ci-config branch October 4, 2018 15:00
@bittner

bittner commented Oct 4, 2018

Copy link
Copy Markdown
Member Author

Do you still have interest in Shippable?

Yes, I'd like to get it running. Just to verify that there's a working alternative to AppVeyor. This will also give us a feeling which of the two is easier to handle, has a nicer UI, etc.

What's still missing now with both Travis and AppVeyor (and Shippable of course) is the automatic upload of created application bundles (executables) to GitHub when a tag is pushed. We need to work on that. ☝️

P.S.: Have you bought your space suit, now that we're getting ready to fly to mars? 😄

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 4, 2018

Copy link
Copy Markdown
Member Author

Re Shippable: Can you verify you have done all steps from Section 1 in Shippable's instructions for Windows builds?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cool-RR

cool-RR commented Oct 4, 2018

Copy link
Copy Markdown
Member

I added you as an administrator on AppVeyor, let me know whether that worked.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cool-RR

cool-RR commented Oct 4, 2018

Copy link
Copy Markdown
Member

I've added you as a collaborator on GitHub, I believe that'll make you a collaborator on Shippable, let me know if it doesn't.

I haven't done anything from the Shippable instructions. Looking at it now it says I need to "Purchase the on-demand WindowsServer 2016 SKU". So, that costs money?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bittner

bittner commented Oct 4, 2018 via email

Copy link
Copy Markdown
Member Author
Hi Ram, No, it shouldn't cost money. Shippable support told me that we need to remove the Linux capabilities (repos that trigger Linux builds, probably) from the account. Because we can only have either Linux or Windows builds enabled, currently. In any case, if it shows something to pay, this should go away once you remove the other option (repo, or whatever it is). I'll forward you all the communication I have received by email. The chat history on the website may not be included, but I'm not sure. Kind regards, Peter Am Do., 4. Okt. 2018 um 23:06 Uhr schrieb Ram Rachum < [email protected]>:
…
I've added you as a collaborator on GitHub, I believe that'll make you a collaborator on Shippable, let me know if it doesn't. I haven't done anything from the Shippable instructions. Looking at it now it says I need to "Purchase the on-demand WindowsServer 2016 SKU". So, that costs money? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#123 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAol8AjpG9Klgd7shOuVYcbKGL_RNC5yks5uhnhjgaJpZM4XDozY> .
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cool-RR

cool-RR commented Oct 5, 2018

Copy link
Copy Markdown
Member

Weren't you made an administrator on the Shippable project? https://app.shippable.com/github/cool-RR/PythonTurtle/settings I hope I can make you an administrator so you could do it.

If not let me know and I'll contact their support.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@bittner @cool-RR
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.