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-109286: Updated macOS SQLite installer to 3.43.1 · Pull Request #110374 · python/cpython

Conversation

@ghost

@ghost ghost commented Oct 4, 2023 •
edited by bedevere-app Bot
Loading

Uh oh!

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

Copy link
Copy Markdown

I updated the macOS SQLite installer, all changes were made in the build-installer.py file. I'll be doing the Windows installer next in a separate PR as requested.

  • Issue: Upgrade to SQLite 3.43 in macOS and Windows installers #109286

Sorry, something went wrong.

Uh oh!

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

All reactions
Julius Tranquilli
Updated macOS SQLite installers to 3.43.1
4052bc9
@ghost
ghost self-requested a review as a code owner October 4, 2023 23:36
@ghost

ghost commented Oct 4, 2023 •
edited by ghost
Loading

Uh oh!

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

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed

All reactions

Sorry, something went wrong.

Uh oh!

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

@bedevere-app

bedevere-app Bot commented Oct 4, 2023

Copy link
Copy Markdown

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

All reactions

Sorry, something went wrong.

Uh oh!

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

@bedevere-app bedevere-app Bot added the awaiting review label Oct 4, 2023
@bedevere-app bedevere-app Bot mentioned this pull request Oct 4, 2023
Upgrade to SQLite 3.43 in macOS and Windows installers #109286
Closed
blurb-it Bot and others added 4 commits October 4, 2023 23:38
@blurb-it
📜🤖 Added by blurb_it.
ddf8d68
Julius Tranquilli
Fixed white space issue
c659e4c
Julius Tranquilli
Merge branch 'macOS-SQLite-installer-upgrade' of github.com:jtranquil…
ef003db
…li/cpython into macOS-SQLite-installer-upgrade
Julius Tranquilli
Fixed trailing white space issue
3e9ae8f
erlend-aasland
erlend-aasland reviewed Oct 5, 2023
View reviewed changes
Comment thread Mac/BuildScript/build-installer.py Resolved

Uh oh!

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

Comment thread Misc/NEWS.d/next/macOS/2023-10-04-23-38-24.gh-issue-109286.1ZLMaq.rst Resolved

Uh oh!

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

erlend-aasland and others added 3 commits October 5, 2023 09:03
@erlend-aasland
Update Misc/NEWS.d/next/macOS/2023-10-04-23-38-24.gh-issue-109286.1ZL…
ee7cf75
…Maq.rst
@erlend-aasland
Update Mac/BuildScript/build-installer.py
aaa2d52
@erlend-aasland
Merge branch 'main' into macOS-SQLite-installer-upgrade
a5b31ac
erlend-aasland
erlend-aasland approved these changes Oct 5, 2023
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting core review and removed awaiting review labels Oct 5, 2023
@erlend-aasland

erlend-aasland commented Oct 5, 2023 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown
Contributor

I updated the macOS SQLite installer, all changes were made in the build-installer.py file. I'll be doing the Windows installer next in a separate PR as requested.

Before doing the Windows PRs, please take a close look at the earlier Windows PRs; your PR should look very similar to these:

  • gh-104623: Update Windows installer to use SQLite 3.42.0 #104625
  • gh-102997: Update Windows installer to SQLite 3.41.2. #102999
All reactions

Sorry, something went wrong.

Uh oh!

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

@erlend-aasland erlend-aasland added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Oct 5, 2023
@ghost

ghost commented Oct 5, 2023

Copy link
Copy Markdown
Author

I updated the macOS SQLite installer, all changes were made in the build-installer.py file. I'll be doing the Windows installer next in a separate PR as requested.

Before doing the Windows PRs, please take a close look at the earlier Windows PRs; your PR should look very similar to these:

  • gh-104623: Update Windows installer to use SQLite 3.42.0 #104625
  • gh-102997: Update Windows installer to SQLite 3.41.2. #102999

Sounds good, sorry about that. I had messed around with those lines because I was getting a trailing white space error, and then in my previous pr someone had asked me to realign the indentation of extra_cflags, so I thought it was accidentally over-indented. Just out of curiosity, why is the extra indentation preferred in that context?

All reactions

Sorry, something went wrong.

Uh oh!

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

@ghost ghost closed this Oct 5, 2023
@ghost
ghost deleted the macOS-SQLite-installer-upgrade branch October 5, 2023 14:40
@erlend-aasland

erlend-aasland commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

Why was this closed?

All reactions

Sorry, something went wrong.

Uh oh!

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

@erlend-aasland

erlend-aasland commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

Just out of curiosity, why is the extra indentation preferred in that context?

It's not that the extra indentation is preferred; the point is that any change should be as isolated as possible. Any style changes (or other unrelated changes) should be left out.

All reactions

Sorry, something went wrong.

Uh oh!

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

@ghost

ghost commented Oct 5, 2023

Copy link
Copy Markdown
Author

Why was this closed?

It was an accident. I was asked to separate the changes for macOS and Windows into two separate PRs, and despite using different branches for each set of changes, GitHub was not allowing me to create separate PRs for each, so I submitted the PR with the macOS changes, then deleted that branch on my GitHub page so that the changes made in that branch would not be included in my PR for the Windows changes, then submitted the PR for the Windows changes. I didn't realize that deleting the branch on my personal page would close this PR and that was the only work around I could think of to isolate the changes for the two operating systems since Github was seemingly not allowing me to isolate the changes made in each branch into two separate PRs. If you merge the PR I made for the Windows SQLite installer upgrade I'll submit a new PR for the macOS installer upgrades, but other than deleting the branch locally I'm not sure how I can isolate those changes as requested. Sorry for the hassle.

erlend-aasland reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

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

@erlend-aasland

erlend-aasland commented Oct 6, 2023

Copy link
Copy Markdown
Contributor

Can you please reopen this PR?

All reactions

Sorry, something went wrong.

Uh oh!

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

@ghost

ghost commented Oct 6, 2023

Copy link
Copy Markdown
Author

Can you please reopen this PR?

I'm prevented from doing so because the branch was deleted, re-uploading it doesn't seem to help either. All the requested changes were saved on my local branch though so I'll just create a new pr. Again, sorry for the hassle.

All reactions

Sorry, something went wrong.

Uh oh!

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

@ghost ghost mentioned this pull request Oct 6, 2023
gh-109286: Update macOS installer to use SQLite 3.43.1 #110482
Merged
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

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

Assignees

No one assigned

Labels

awaiting core review needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

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

1 participant

@erlend-aasland
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.