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

Some minor enhancements / update of MingW Windows build #50

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cemeyer
Copy link

@cemeyer cemeyer commented Feb 13, 2019

Make 'winpkg' target 32- or 64-bit agnostic and document 64-bit build on more recent Fedora.

Document how to build stoken-gui.exe as a Windows mode application without a console.

With this, stoken-gui.exe works quite well for me on Windows 10. It doesn't seem to remember the saved pin, but can read the saved token. I'm not sure why and can't really debug. But one can create a shortcut to stoken-gui.exe with --pin ABCDEFGH as a workaround.

Document working build on more recent version of Fedora.  Use 64-bit build;
Windows has been predominantly 64-bit for a decade.

Document hacking in '-Wl,--subsystem,windows' to stoken-gui.exe build via
GTK_LIBS in README, in order to run stoken-gui as a "Windows" mode
application instead of a console application.  I'd love a better approach
here, but am not familiar enough with autotools to propose one.
@sorenisanerd
Copy link
Contributor

Apologies for the long wait. If you're still interested in this, would you consider adding a workflow (e.g. in .github/workflows/c-cpp.yml) that runs this build? That way we can be sure that it works. We can even publish the resulting binaries.

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