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

Building for windows using Docker #22

Merged
merged 6 commits into from
Aug 8, 2019
Merged

Building for windows using Docker #22

merged 6 commits into from
Aug 8, 2019

Conversation

drowe67
Copy link
Owner

@drowe67 drowe67 commented Jul 25, 2019

  1. Using Danilo's fine work to allow us to build freedv-gui for windows using Docker. This is another step in automating the build process, and allows us to use Docker containers rather than a Fedora machine.

    Thanks Danilo for showing how this can be done 👍

  2. Now building working Windows installers, people have tested the EXEs with good results.

  3. Fixed a FIFO size bug that had broken 2020 Rx.

  4. Automation script to build freedv-gui on a remote machine and upload 32 and 64 bit Windows installers to a web site in a single step.

@ghost
Copy link

ghost commented Jul 25, 2019

That is on my wish list! Bravo

@hobbes1069
Copy link
Collaborator

Looks interesting! It doesn't really matter since we're just using the mingw capabilities but technically Fedora 28 is EOL.

@hogthrob
Copy link

Hi,
changing the Fedora release is a simple matter of changing FED_REL in docker-compose-win.yml to any desired number and verifying that the build instructions in the docker file fdv_win_fedora/Dockerfile are still working for this release.

@drowe67
Copy link
Owner Author

drowe67 commented Aug 8, 2019

Thanks @hobbes1069 @hogthrob I'll try Fedora 30 on the next pass on this code. Certainly makes my life easier - one script to run for the whole build and deploy process.

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.

3 participants