GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-87868: Sort and remove duplicates in getenvironment() by aisk · Pull Request #102731 · python/cpython

Conversation

@aisk

@aisk aisk commented Mar 15, 2023 •
edited by bedevere-bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

This should fix #87868 .

  • Issue: [_winapi] correctly sort and remove duplicates in getenvironment() #87868

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@aisk
pythongh-87868: correctly sort and remove duplicates in getenvironment()
aed39b7
@aisk
aisk requested a review from gpshead as a code owner March 15, 2023 15:59
@bedevere-bot bedevere-bot mentioned this pull request Mar 15, 2023
[_winapi] correctly sort and remove duplicates in getenvironment() #87868
Closed
@bedevere-bot bedevere-bot added the awaiting review label Mar 15, 2023
eendebakpt
eendebakpt reviewed Mar 15, 2023
View reviewed changes
Comment thread Misc/NEWS.d/next/Windows/2023-03-15-23-53-45.gh-issue-87868.4C36oQ.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

AlexWaygood added a commit to AlexWaygood/cpython that referenced this pull request Mar 15, 2023
@AlexWaygood
Exclude ceval.c from the C-analyzer tool
6da2198
The "check if generated files are up to date" CI check appears to be currently failing on all PRs (but not on pushes to main) 

See, for example:
- python#94468
- python#94468
- python#102731

This appears to be because the C-analyzer tool doesn't like the `#line` directives introduced in python@70185de. I'm advised by the message printed to the terminal in https://github.com/python/cpython/actions/runs/4428706945/jobs/7768216988#step:14:84 that this is the appropriate short-term fix!

Cc. @ericsnowcurrently
This was referenced Mar 15, 2023
Exclude ceval.c from the C-analyzer tool #102735
Merged
C-analyzer tool cannot parse #line directives #102737
Closed
AlexWaygood added a commit that referenced this pull request Mar 15, 2023
@AlexWaygood
Exclude ceval.c from the C-analyzer tool (#102735)
215007b
The "check if generated files are up to date" CI check appears to be currently failing on all PRs (but not on pushes to main) 

See, for example:
- #94468
- #94468
- #102731

This appears to be because the C-analyzer tool doesn't like the `#line` directives introduced in 70185de. I'm advised by the message printed to the terminal in https://github.com/python/cpython/actions/runs/4428706945/jobs/7768216988#step:14:84 that this is the appropriate short-term fix!
@AlexWaygood
Merge branch 'main' into getenvironment
4de761b
@aisk
aisk force-pushed the getenvironment branch from 5837143 to 0182d07 Compare March 17, 2023 15:39
@aisk
check null in normalize_environment
04f7d59
@aisk
aisk force-pushed the getenvironment branch from 0182d07 to 04f7d59 Compare March 17, 2023 15:57
Fidget-Spinner pushed a commit to Fidget-Spinner/cpython that referenced this pull request Mar 27, 2023
@AlexWaygood @Fidget-Spinner
Exclude ceval.c from the C-analyzer tool (python#102735)
7204c41
The "check if generated files are up to date" CI check appears to be currently failing on all PRs (but not on pushes to main) 

See, for example:
- python#94468
- python#94468
- python#102731

This appears to be because the C-analyzer tool doesn't like the `#line` directives introduced in python@70185de. I'm advised by the message printed to the terminal in https://github.com/python/cpython/actions/runs/4428706945/jobs/7768216988#step:14:84 that this is the appropriate short-term fix!
@aisk
aisk requested a review from eendebakpt April 3, 2023 12:10
warsaw pushed a commit to warsaw/cpython that referenced this pull request Apr 11, 2023
@AlexWaygood @warsaw
Exclude ceval.c from the C-analyzer tool (python#102735)
abb9cbd
The "check if generated files are up to date" CI check appears to be currently failing on all PRs (but not on pushes to main) 

See, for example:
- python#94468
- python#94468
- python#102731

This appears to be because the C-analyzer tool doesn't like the `#line` directives introduced in python@70185de. I'm advised by the message printed to the terminal in https://github.com/python/cpython/actions/runs/4428706945/jobs/7768216988#step:14:84 that this is the appropriate short-term fix!
eendebakpt
eendebakpt reviewed Aug 19, 2023
View reviewed changes
Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Lib/test/test_subprocess.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

aisk and others added 6 commits August 20, 2023 16:17
@aisk @eendebakpt
Update Modules/_winapi.c
c43abb2
Co-authored-by: Pieter Eendebak <[email protected]>
@aisk @eendebakpt
Update Modules/_winapi.c
3f6319a
Co-authored-by: Pieter Eendebak <[email protected]>
@aisk @eendebakpt
Update Lib/test/test_subprocess.py
8ef3b9c
Co-authored-by: Pieter Eendebak <[email protected]>
@aisk
Merge remote-tracking branch 'upstream/main' into getenvironment
46491c5
@aisk
Adapt latest changes from main branch
ca13205
@aisk
Fix test case
f727004
@aisk
aisk requested a review from eendebakpt August 20, 2023 12:41
eendebakpt
eendebakpt reviewed Aug 22, 2023
View reviewed changes
Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Lib/test/test_subprocess.py Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

aisk added 2 commits September 3, 2023 17:26
@aisk
update for review comments
6b9e873
@aisk
add win32 related env validation tests
dd53528
@aisk
aisk requested a review from eendebakpt September 3, 2023 11:32
eendebakpt
eendebakpt reviewed Sep 3, 2023
View reviewed changes
Comment thread Modules/_winapi.c Resolved

Uh oh!

There was an error while loading. Please reload this page.

@aisk
fix memory leak
37fa2f2
@zooba

zooba commented Jan 9, 2024

Copy link
Copy Markdown
Member

CI looks good (and Azure Pipelines can be ignored here), so congrats! It's in!

erlend-aasland reacted with rocket emoji
All reactions
  • 🚀 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@zooba zooba added OS-windows needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Jan 9, 2024
@miss-islington-app

This comment has been minimized.

Sign in to view
@miss-islington-app

This comment has been minimized.

Sign in to view
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 9, 2024
@aisk @AlexWaygood
@eendebakpt @erlend-aasland @miss-islington
pythongh-87868: Sort and remove duplicates in getenvironment() (pytho…
12db2cd
…nGH-102731)

(cherry picked from commit c31be58)

Co-authored-by: AN Long <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
@bedevere-app

bedevere-app Bot commented Jan 9, 2024

Copy link
Copy Markdown

GH-113865 is a backport of this pull request to the 3.12 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 9, 2024
@aisk @AlexWaygood
@eendebakpt @erlend-aasland @miss-islington
pythongh-87868: Sort and remove duplicates in getenvironment() (pytho…
255872a
…nGH-102731)

(cherry picked from commit c31be58)

Co-authored-by: AN Long <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Jan 9, 2024
@bedevere-app

bedevere-app Bot commented Jan 9, 2024

Copy link
Copy Markdown

GH-113866 is a backport of this pull request to the 3.11 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the needs backport to 3.11 only security fixes label Jan 9, 2024
@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 RHEL7 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/15/builds/6594) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/15/builds/6594

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-RHEL7-x86_64/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 RHEL8 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/185/builds/5849) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/185/builds/5849

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-RHEL8-x86_64/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot aarch64 RHEL8 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/529/builds/5692) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/529/builds/5692

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-RHEL8-aarch64/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@zooba

zooba commented Jan 9, 2024

Copy link
Copy Markdown
Member

Okay, these buildbot failures look related to the new test. Will be a couple hours before I can dig into it myself, so if someone else gets started first, I'd add the actual stdout to the message parameter of the assertTrue call to see what's happening there.

It's possible that the process is failing to start with a mostly empty environment?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@aisk

aisk commented Jan 9, 2024

Copy link
Copy Markdown
Member Author

I'm attempting to address it, I think maybe it's caused by create a new process need some basic environment to start with.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot aarch64 Fedora Stable 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/125/builds/5079) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/125/builds/5079

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-fedora-stable-aarch64/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 RHEL8 FIPS Only Blake2 Builtin Hash 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/469/builds/6966) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/469/builds/6966

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-RHEL8-fips-x86_64.no-builtin-hashes-except-blake2/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 Ubuntu Shared 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/506/builds/6791) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/506/builds/6791

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/srv/buildbot/buildarea/3.x.bolen-ubuntu/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@aisk
aisk deleted the getenvironment branch January 9, 2024 16:35
@aisk

aisk commented Jan 9, 2024

Copy link
Copy Markdown
Member Author

Change the assertTrue to assertEqual will help to address it. Should I open a new PR to do this, or is there some other way to do it?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@aisk

aisk commented Jan 9, 2024

Copy link
Copy Markdown
Member Author

The change will be like #113867

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@aisk aisk mentioned this pull request Jan 9, 2024
gh-87868: Skip some test in test_subprocess if python is configured with --enable-shared #113867
Merged
@bedevere-bot

bedevere-bot commented Jan 9, 2024

Copy link
Copy Markdown

⚠️⚠️⚠️ Buildbot failure ⚠️⚠️⚠️

Hi! The buildbot AMD64 Fedora Stable 3.x has failed when building commit c31be58.

What do you need to do:

  1. Don't panic.
  2. Check the buildbot page in the devguide if you don't know what the buildbots are or how they work.
  3. Go to the page of the buildbot that failed (https://buildbot.python.org/all/#builders/543/builds/5159) and take a look at the build logs.
  4. Check if the failure is related to this commit (c31be58) or if it is a false positive.
  5. If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/all/#builders/543/builds/5159

Failed tests:

  • test_subprocess

Failed subtests:

  • test_one_environment_variable - test.test_subprocess.ProcessTestCase.test_one_environment_variable
  • test_one_environment_variable - test.test_subprocess.ProcessTestCaseNoPoll.test_one_environment_variable

Summary of the results of the build (if available):

==

Click to see traceback logs
Traceback (most recent call last):
  File "/home/buildbot/buildarea/3.x.cstratak-fedora-stable-x86_64/build/Lib/test/test_subprocess.py", line 847, in test_one_environment_variable
    self.assertTrue(stdout.startswith(b"fruit=orange"))
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: False is not true
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

zooba pushed a commit that referenced this pull request Jan 10, 2024
@miss-islington @aisk
@AlexWaygood @eendebakpt @erlend-aasland
gh-87868: Sort and remove duplicates in getenvironment() (GH-102731)
4050a15
(cherry picked from commit c31be58)

Co-authored-by: AN Long <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
zooba pushed a commit that referenced this pull request Jan 11, 2024
@miss-islington @aisk
@AlexWaygood @eendebakpt @erlend-aasland
gh-87868: Sort and remove duplicates in getenvironment() (GH-102731)
d15e1ac
(cherry picked from commit c31be58)

Co-authored-by: AN Long <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
kulikjak pushed a commit to kulikjak/cpython that referenced this pull request Jan 22, 2024
@aisk @AlexWaygood
@eendebakpt @erlend-aasland @kulikjak
pythongh-87868: Sort and remove duplicates in getenvironment() (pytho…
f9c3b4a
…nGH-102731)

Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
aisk added a commit to aisk/cpython that referenced this pull request Feb 11, 2024
@aisk @AlexWaygood
@eendebakpt @erlend-aasland
pythongh-87868: Sort and remove duplicates in getenvironment() (pytho…
aa60e17
…nGH-102731)

Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
@aisk @AlexWaygood
@eendebakpt @erlend-aasland @Glyphack
pythongh-87868: Sort and remove duplicates in getenvironment() (pytho…
888b7c5
…nGH-102731)

Co-authored-by: Alex Waygood <[email protected]>
Co-authored-by: Pieter Eendebak <[email protected]>
Co-authored-by: Erlend E. Aasland <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eendebakpt eendebakpt eendebakpt left review comments
@zooba zooba zooba approved these changes
@erlend-aasland erlend-aasland erlend-aasland approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

OS-windows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[_winapi] correctly sort and remove duplicates in getenvironment()

Uh oh!

There was an error while loading. Please reload this page.

6 participants

@aisk @eendebakpt @bedevere-bot @erlend-aasland @zooba @AlexWaygood
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.