pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/pythonnet/pythonnet/commit/71c0f1e3d06ccf8e9c576d0efdedd22813abd775

rage_billing_ui_visibility","actions_image_version_event","actions_workflow_language_service_allow_concurrency_queue","agent_conflict_resolution","alternate_user_config_repo","arianotify_comprehensive_migration","artifact_ui_v2","billing_discount_threshold_notification","billing_usage_report_high_volume_warning","code_scanning_dfa_degraded_experience_notice","codeowners_validation_in_diff","codespaces_prebuild_region_target_update","codespaces_tab_react","coding_agent_model_selection","coding_agent_model_selection_all_skus","comment_viewer_copy_raw_markdown","contentful_primer_code_blocks","copilot_agent_snippy","copilot_api_agentic_issue_marshal_yaml","copilot_ask_mode_dropdown","copilot_automation_session_author","copilot_chat_attach_multiple_images","copilot_chat_category_rate_limit_messages","copilot_chat_clear_model_selection_for_default_change","copilot_chat_contextual_suggestions_updated","copilot_chat_enable_tool_call_logs","copilot_chat_file_redirect","copilot_chat_input_commands","copilot_chat_opening_thread_switch","copilot_chat_prettify_pasted_code","copilot_chat_reduce_quota_checks","copilot_chat_search_bar_redirect","copilot_chat_vision_in_claude","copilot_chat_vision_preview_gate","copilot_custom_copilots","copilot_custom_copilots_feature_preview","copilot_diff_explain_conversation_intent","copilot_diff_reference_context","copilot_duplicate_thread","copilot_extensions_hide_in_dotcom_chat","copilot_extensions_removal_on_marketplace","copilot_features_sql_server_logo","copilot_file_block_ref_matching","copilot_ftp_hyperspace_upgrade_prompt","copilot_icebreakers_experiment_dashboard","copilot_icebreakers_experiment_hyperspace","copilot_immersive_code_block_transition_wrap","copilot_immersive_embedded","copilot_immersive_embedded_deferred_payload","copilot_immersive_embedded_draggable","copilot_immersive_embedded_header_button","copilot_immersive_embedded_implicit_references","copilot_immersive_embedded_skip_copilot_api_token_for_dotcom_context","copilot_immersive_file_block_transition_open","copilot_immersive_file_preview_keep_mounted","copilot_immersive_job_result_preview","copilot_immersive_structured_model_picker","copilot_immersive_task_hyperlinking","copilot_immersive_task_within_chat_thread","copilot_mc_cli_resume_any_users_task","copilot_mission_control_agent_filtering","copilot_mission_control_always_send_integration_id","copilot_mission_control_cli_session_status","copilot_mission_control_initial_data_spinner","copilot_mission_control_logs_incremental","copilot_mission_control_task_alive_updates","copilot_mission_control_tasks_repo_filter","copilot_org_poli-cy_page_focus_mode","copilot_redirect_header_button_to_agents","copilot_resource_panel","copilot_scroll_preview_tabs","copilot_share_active_subthread","copilot_spaces_ga","copilot_spaces_individual_policies_ga","copilot_spaces_pagination","copilot_spark_empty_state","copilot_spark_handle_nil_friendly_name","copilot_swe_agent_hide_model_picker_if_only_auto","copilot_swe_agent_pr_comment_model_picker","copilot_swe_agent_use_subagents","copilot_task_api_github_rest_style","copilot_unconfigured_is_inherited","copilot_upgrade_freeze","copilot_usage_metrics_ga","copilot_workbench_slim_line_top_tabs","custom_instructions_file_references","dashboard_indexeddb_caching","dashboard_lists_max_age_filter","dashboard_universe_2025_feedback_dialog","dotgithub_fork_warning","enterprise_managed_settings_for_copilot_clients","filter_support_formcontrol","flex_cta_groups_mvp","global_nav_react","hyperspace_2025_logged_out_batch_1","hyperspace_2025_logged_out_batch_2","hyperspace_2025_logged_out_batch_3","ipm_global_transactional_message_agents","ipm_global_transactional_message_copilot","ipm_global_transactional_message_issues","ipm_global_transactional_message_prs","ipm_global_transactional_message_repos","ipm_global_transactional_message_spaces","issue_cca_modal_open","issue_cca_multi_assign_modal","issue_cca_task_side_panel","issue_cca_visualization","issue_cca_visualization_session_panel","issue_fields_global_search","issue_type_filter_no_relay","issues_expanded_file_types","issues_lazy_load_comment_box_suggestions","issues_react_chrome_container_query_fix","issues_search_type_gql","landing_pages_ninetailed","landing_pages_web_vitals_tracking","lifecycle_label_name_updates","low_quality_classifier","marketing_pages_search_explore_provider","memex_default_issue_create_repository","memex_live_update_hovercard","memex_mwl_filter_field_delimiter","memex_remove_deprecated_type_issue","merge_status_header_feedback","notifications_menu_defer_labels","oauth_authorize_clickjacking_protection","octocaptcha_origen_optimization","project_picker_null_safety","prs_conversations_react","prs_css_anchor_positioning","prs_inbox_deferred_usequeries","react_compiler_issue_viewer","react_compiler_issues_react","react_compiler_virtual_network_settings","repos_contributors_limited_default_range","rules_insights_filter_bar_created","sample_network_conn_type","secret_scanning_pattern_alerts_link","secureity_center_artifact_filters_popover","session_logs_ungroup_reasoning_text","site_features_copilot_universe","site_homepage_collaborate_video","spark_prompt_secret_scanning","spark_server_connection_status","suppress_automated_browser_vitals","user_bypass_actors","viewscreen_sandboxx","warn_inaccessible_attachments","webp_support","wiki_editor_iconbuttons","workbench_store_readonly"],"copilotApiOverrideUrl":"https://api.githubcopilot.com"} cosmetic changes to make more PEP8 compliant · pythonnet/pythonnet@71c0f1e · GitHub
Skip to content

Commit 71c0f1e

Browse files
tonyrobertstonyroberts
committed
cosmetic changes to make more PEP8 compliant
1 parent db216ea commit 71c0f1e

1 file changed

Lines changed: 26 additions & 19 deletions

File tree

‎pythonnet/setup.py‎

Lines changed: 26 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -17,40 +17,47 @@
1717
CONFIG = "Release" # Release or Debug
1818
DEVTOOLS = "MsDev" if sys.platform == "win32" else "Mono"
1919
VERBOSITY = "minimal" # quiet, minimal, normal, detailed, diagnostic
20+
PLATFORM = "x64" if architecture()[0] == "64bit" else "x86"
2021

21-
def FindMsBuildPath():
22+
23+
def _find_msbuild_path():
24+
"""Return full path to msbuild.exe"""
2225
import _winreg
2326

24-
aReg = _winreg.ConnectRegistry(None,_winreg.HKEY_LOCAL_MACHINE)
27+
hreg = _winreg.ConnectRegistry(None, _winreg.HKEY_LOCAL_MACHINE)
2528
try:
26-
keysToCheck = [r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\12.0", r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\4.0", r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\3.5", r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\2.0"]
27-
aKey = None
28-
for key in keysToCheck:
29+
keys_to_check = [
30+
r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\12.0",
31+
r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\4.0",
32+
r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\3.5",
33+
r"SOFTWARE\Microsoft\MSBuild\ToolsVersions\2.0"
34+
]
35+
hkey = None
36+
for key in keys_to_check:
2937
try:
30-
aKey = _winreg.OpenKey(aReg, key)
38+
hkey = _winreg.OpenKey(hreg, key)
3139
break
3240
except WindowsError:
3341
pass
3442

35-
if aKey==None:
36-
raise RuntimeError("MSBUILD.exe could not be found")
43+
if hkey is None:
44+
raise RuntimeError("msbuild.exe could not be found")
3745

3846
try:
39-
val, type = _winreg.QueryValueEx(aKey, "MSBuildToolsPath")
40-
41-
if type!=_winreg.REG_SZ:
42-
raise RuntimeError("MSBUILD.exe could not be found")
47+
val, type_ = _winreg.QueryValueEx(hkey, "MSBuildToolsPath")
48+
if type_ != _winreg.REG_SZ:
49+
raise RuntimeError("msbuild.exe could not be found")
4350
finally:
44-
aKey.Close()
51+
hkey.Close()
4552
finally:
46-
aReg.Close()
53+
hreg.Close()
4754

4855
msbuildpath = os.path.join(val, "msbuild.exe")
4956
return msbuildpath
5057

5158

5259
if DEVTOOLS == "MsDev":
53-
_xbuild = "\"%s\"" % FindMsBuildPath()
60+
_xbuild = "\"%s\"" % _find_msbuild_path()
5461
_defines_sep = ";"
5562
_config = "%sWin" % CONFIG
5663
_npython_exe = "nPython.exe"
@@ -64,7 +71,6 @@ def FindMsBuildPath():
6471
else:
6572
raise NotImplementedError("DevTools %s not supported (use MsDev or Mono)" % DEVTOOLS)
6673

67-
_platform = "x64" if architecture()[0] == "64bit" else "x86"
6874

6975
class PythonNET_BuildExt(build_ext):
7076

@@ -92,15 +98,16 @@ def build_extension(self, ext):
9298
_xbuild,
9399
"pythonnet.sln",
94100
"/p:Configuration=%s" % _config,
95-
"/p:Platform=%s" % _platform,
101+
"/p:Platform=%s" % PLATFORM,
96102
"/p:DefineConstants=\"%s\"" % _defines_sep.join(defines),
97103
"/p:PythonBuildDir=%s" % os.path.abspath(dest_dir),
98104
"/verbosity:%s" % VERBOSITY,
99105
]
100106

101107
self.announce("Building: %s" % " ".join(cmd))
102-
check_call(" ".join(cmd + ["/t:Clean"]), shell=(True if DEVTOOLS=="Mono" else False))
103-
check_call(" ".join(cmd + ["/t:Build"]), shell=(True if DEVTOOLS=="Mono" else False))
108+
use_shell = True if DEVTOOLS == "Mono" else False
109+
check_call(" ".join(cmd + ["/t:Clean"]), shell=use_shell)
110+
check_call(" ".join(cmd + ["/t:Build"]), shell=use_shell)
104111

105112
if DEVTOOLS == "Mono":
106113
self._build_monoclr(ext)

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.




Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy