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

add --version to invoke.py arguments #2038

Merged
merged 7 commits into from
Dec 20, 2022
Merged

add --version to invoke.py arguments #2038

merged 7 commits into from
Dec 20, 2022

Conversation

lstein
Copy link
Collaborator

@lstein lstein commented Dec 16, 2022

This commit allows invoke.py to print out its name and version number when given the --version argument. I had to move some status messages around in order to make the output clean.

There is still an early message about initializing patchmatch that interferes with a clean print of the version, and in fact the --no-patchmatch argument is not doing anything. This will be the subject of a subsequent PR.

This commit allows invoke.py to print out its name and version
number when given the --version argument. I had to move some
status messages around in order to make the output clean.

There is still an early message about initializing patchmatch
that interferes with a clean print of the version, and in fact the
--no-patchmatch argument is not doing anything. This will be the
subject of a subsequent PR.
@lstein lstein requested review from Kyle0654 and ebr December 16, 2022 18:59
@lstein lstein closed this Dec 16, 2022
Needed to support the web backend.
@lstein lstein reopened this Dec 16, 2022
@lstein
Copy link
Collaborator Author

lstein commented Dec 17, 2022

This is ready for review.

@lstein lstein requested review from mauwii and removed request for Kyle0654 December 19, 2022 20:57
@lstein lstein enabled auto-merge (squash) December 19, 2022 20:58
@ebr
Copy link
Member

ebr commented Dec 19, 2022

I'll try to test this later tonight, but noticed that an empty create_installer.sh file was added - is this intentional?

Copy link
Member

@ebr ebr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I tried to take a stab at avoiding the patchmatch messages, but quickly folded :)

(question about the empty installer/create_installer.sh still remains though)

@mauwii
Copy link
Contributor

mauwii commented Dec 20, 2022

(question about the empty installer/create_installer.sh still remains though)

This is really strange, since it is only empty when viewed in this PR, but when I check out the PR it is not empty, also not empty when I view the File in the Commit itself.

So let's knock on wood that this is only a bug in GitHub's WebUI 🙈

@ebr
Copy link
Member

ebr commented Dec 20, 2022

ok, I see what happened - the permissions were adjusted from 0755 to 0644, i.e. create_installer.sh was made non-executable. @lstein , was that intentional?

@lstein lstein merged commit 6e98b55 into main Dec 20, 2022
@lstein lstein deleted the lstein-show-version branch December 20, 2022 15:14
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.

None yet

3 participants