Skip to content

Conversation

yghjn
Copy link

@yghjn yghjn commented Mar 18, 2024

@ghost
Copy link

ghost commented Mar 18, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Mar 18, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@vstinner
Copy link
Member

cc @erlend-aasland

@python-cla-bot
Copy link

python-cla-bot bot commented Apr 18, 2025

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

@erlend-aasland
Copy link
Contributor

This is not a problem for most folks; we've had this bash code since 1994. IOW, most people have text editors that add new lines. Either we just update the docs to say that Setup.local must contain EOL, or we simply add an implicit newline in the loop:

diff --git a/Modules/makesetup b/Modules/makesetup
index f6cf695b457..104c824b846 100755
--- a/Modules/makesetup
+++ b/Modules/makesetup
@@ -90,6 +90,7 @@ NL='\
 # Main loop
 for i in ${*-Setup}
 do
+       echo ''  # Add a linebreak so we don't choke on files missing EOL.
        case $i in
        -n)     echo '*noobjects*';;
        *)      echo '*doconfig*'; cat "$i";;

@erlend-aasland
Copy link
Contributor

Closing in favour of #139100, and also since the CLA is not signed.

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.

3 participants