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

gh-100829: Show the Python Launcher version #100832

Closed
wants to merge 9 commits into from

Conversation

oleksis
Copy link

@oleksis oleksis commented Jan 7, 2023

@oleksis oleksis force-pushed the py_launcher2_version branch from d221f64 to 7308cac Compare January 7, 2023 23:43
PC/launcher2.c Outdated Show resolved Hide resolved
@oleksis
Copy link
Author

oleksis commented Feb 14, 2023

@zooba i update the changes for py launcher show the version of the application binary . Testing the changes with this versionLauncher.patch

In case apply the patch add the additional dependency version.lib; in the project PCbuild/pylauncher.vcxproj

 <Link>
      <AdditionalDependencies>version.lib;shell32.lib;pathcch.lib;%(AdditionalDependencies)</AdditionalDependencies>
      <SubSystem>Console</SubSystem>
 </Link>

Any advice, welcome!

@zooba
Copy link
Member

zooba commented Feb 14, 2023

There's still discussion (on the issue) to resolve about what this is for. I don't believe it's worth adding an extra option solely for users to run themselves, which is what the current PR does.

@terryjreedy
Copy link
Member

The issue has been closed, so I think this PR should be also.

@terryjreedy terryjreedy closed this Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants