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

docs: Be explicit up front about getting the actual executable. #98

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

Conversation

gnprice
Copy link

@gnprice gnprice commented Jul 16, 2019

Looking at the getting-started page of the docs, because everything
is in terms of pyoxidizer run it's not obvious how to actually
lay hands on the single executable file that PyOxidizer produces.
Here's one example of a puzzled would-be user:
https://groups.google.com/forum/#!topic/pyoxidizer-users/rUyM3iTpZq0

Fix that by breaking up the first example into pyoxidizer build
and then an explicit invocation of the built executable.

(Also a smaller preparatory commit.)

gnprice added 2 commits July 15, 2019 18:57
This is what it looks like for me with the v0.2.0 release (modulo
adjusting the project's root directory to follow the example.)

Various other bits of output in examples have also changed.  But
it'd be a lot of work to update them all comprehensively, and I
think better to update piecemeal than not at all.
Looking at the getting-started page of the docs, because everything
is in terms of `pyoxidizer run` it's not obvious how to actually
lay hands on the single executable file that PyOxidizer produces.
Here's one example of a puzzled would-be user:
  https://groups.google.com/forum/#!topic/pyoxidizer-users/rUyM3iTpZq0

Fix that by breaking up the first example into `pyoxidizer build`
and then an explicit invocation of the built executable.
@indygreg indygreg force-pushed the main branch 25 times, most recently from 296b083 to 13e08f6 Compare November 23, 2020 01:46
@indygreg indygreg force-pushed the main branch 3 times, most recently from 8f998b7 to 5c5ac33 Compare December 1, 2021 03:16
@indygreg indygreg force-pushed the main branch 2 times, most recently from 5e0524b to 789c62e Compare May 31, 2022 04:07
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.

1 participant